Skip to content
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

Peg rails dependency to 4.0.0 #7

Closed
wants to merge 5 commits into from
Closed

Peg rails dependency to 4.0.0 #7

wants to merge 5 commits into from

Conversation

tiegz
Copy link
Contributor

@tiegz tiegz commented Jul 12, 2013

WHAT

  • Set rails in gem to at least 4.0.0
  • Also fix some deprecation warnings in the tests

WHY

bundle didn't work for me when I initially checked out the gem:

Bundler could not find compatible versions for gem "activerecord":
  In Gemfile:
    rails (>= 0) ruby depends on
      activerecord (= 4.1.0.beta) ruby

    activerecord (4.0.0.rc1)

Could not find gem 'activerecord-deprecated_finders (~> 0.0.3) ruby', which is required by gem 'activerecord (~> 4.0.0.beta) ruby', in any of the sources.

Now that 4.0.0 we might as well point to that. After I did this, I get one failing test in SaveFromAfterCommitBlockTest#test_after_commit_in_save, but I'm not sure if it existed before these commits? (can't find travis build)

gem 'activerecord-deprecated_finders', github: 'rails/activerecord-deprecated_finders', branch: 'master'
gem 'rails', github: 'rails/rails', branch: 'master'
gem 'activerecord-deprecated_finders'
gem 'rails'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe you can remove this line altogether.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, thx! will do

@tiegz
Copy link
Contributor Author

tiegz commented Jul 12, 2013

@rafaelfranca the test failure I'm getting is:

assert_equal true, topic.cached

  1) Failure:
SaveFromAfterCommitBlockTest#test_after_commit_in_save [test/transaction_callbacks_test.rb:275]:
Expected: true
  Actual: false

This test broke in the switch from 4.0.0.beta1 to 4.0.0 (the test originated at rails/rails@44d1804)

I can't figure out what changed, but I'm also not sure why that test file is in rails-observers. Is it ok to remove it and/or return it to rails?

@rafaelfranca
Copy link
Member

I need to take a look better on this one. Will do later

@guilleiguaran
Copy link
Member

Oops, didn't noticed this and I fixed rake test, still getting the failing test mentioned here and some deprecation warnings.

Can you update this to handle only the deprecation warnings?

@guilleiguaran
Copy link
Member

Deprecation warnings fixed, thanks for your contribution!!!

@guilleiguaran
Copy link
Member

btw, the test still failing 😁

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