Skip to content

Rspec tests fail due to tilt dependency #248

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 21, 2015
Merged

Rspec tests fail due to tilt dependency #248

merged 1 commit into from
Apr 21, 2015

Conversation

gbanis
Copy link
Contributor

@gbanis gbanis commented Apr 16, 2015

This is related to #243

I am using react-rails as dependency in a gem I created and running rspec in that gem throws the following error:

$ rspec
/Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react/jsx/template.rb:1:in require': cannot load such file -- tilt (LoadError) from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react/jsx/template.rb:1:in<top (required)>'
from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react/jsx.rb:3:in require' from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react/jsx.rb:3:in<top (required)>'
from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react-rails.rb:1:in require' from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react-rails.rb:1:in<top (required)>'
...
This happens because react-rails requires tilt here.

I've noticed that adding tilt as a dependency in the gemspec solves this issue and all other tests run without a problem.

This is related to [#243](https://github.com/reactjs/react-rails/pull/243/files)

I am using react-rails as dependency in a gem I created and running rspec in that gem throws the following error:

$ rspec
/Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react/jsx/template.rb:1:in `require': cannot load such file -- tilt (LoadError)
    from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react/jsx/template.rb:1:in `<top (required)>'
    from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react/jsx.rb:3:in `require'
    from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react/jsx.rb:3:in `<top (required)>'
    from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react-rails.rb:1:in `require'
    from /Users/gbanis/.rvm/gems/ruby-2.0.0-p598/gems/react-rails-0.13.0.0/lib/react-rails.rb:1:in `<top (required)>'
    ...
This happens because react-rails requires tilt here.

I've noticed that adding tilt as a dependency in the gemspec solves this issue and all other tests run without a problem.
@zpao
Copy link
Member

zpao commented Apr 16, 2015

cc @rmosolgo just a heads up here, looks like this makes the tests actually pass again :)

@rmosolgo
Copy link
Member

👍 weird that this affects the tests but I'll take it! good catch!

(I thought tilt was a dependency of rails, I guess not so!)

rmosolgo pushed a commit that referenced this pull request Apr 21, 2015
Rspec tests fail due to `tilt` dependency
@rmosolgo rmosolgo merged commit 0faa0f6 into reactjs:master Apr 21, 2015
gbanis-fiksu added a commit to gbanis-fiksu/react-rails that referenced this pull request May 27, 2015
For more info look at PRs [248](reactjs#248) and [243](reactjs#243).
gbanis-fiksu added a commit to gbanis-fiksu/react-rails that referenced this pull request May 27, 2015
For more info look at PRs [248](reactjs#248) and [243](reactjs#243).
This was referenced May 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants