ChiliProject is not maintained anymore. Please be advised that there will be no more updates.

We do not recommend that you setup new ChiliProject instances and we urge all existing users to migrate their data to a maintained system, e.g. Redmine. We will provide a migration script later. In the meantime, you can use the instructions by Christian Daehn.

Test errors on 1.8.6 after acts_as_journalized merge (Bug #413)

Added by Gregor Schmidt at 2011-05-13 10:08 pm. Updated at 2011-05-13 11:39 pm.

Status:Closed Start date:2011-05-13
Priority:Normal Due date:
Assignee:- % Done:

0%
Category:-
Target version:2.0.0
Remote issue URL: Affected version:unstable

Description

There is an incompatibility with an API change from 1.8.6 to 1.8.7. Some tests in the IssueControllerTest rely on Hash#first which was only introduced with Ruby 1.8.7.

I know, that 1.8.6's support status is rather vague, but since the needed changes are rather minimal, I am proposing to fix this issue.

Associated revisions

Revision 9c9dc6e8
Added by Jean-Philippe Lang at 2009-05-17 11:55 am

Adds email notification on wiki changes (#413). It's disabled by default and can be enabled in application settings.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2749 e93f8b46-1217-0410-a6f0-8f06a7374b81

Revision 85ce903c
Added by Jean-Philippe Lang at 2009-05-25 09:02 pm

Ability to watch a wiki or a single wiki page (#413).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2765 e93f8b46-1217-0410-a6f0-8f06a7374b81

Revision 300bc07b
Added by Gregor Schmidt at 2011-05-14 12:09 am

[#413] using Hash#to_a.first since Hash#first is not present in 1.8.6

Revision 0957e478
Added by Felix Schäfer at 2011-05-14 01:38 am

Merge pull request #54 from schmidt/b/413-test-errors-in-1.8.6

Fix test errors on 1.8.6 after acts_as_journalized merge. #413

History

#1
Updated by Gregor Schmidt at 2011-05-13 10:25 pm

The pull request is at https://github.com/chiliproject/chiliproject/pull/54.

Thanks for having a look.

  • Assignee deleted (Gregor Schmidt)
  • Status changed from Open to Ready for review

#2
Updated by Felix Schäfer at 2011-05-13 11:39 pm

As this is only in the tests, I don't mind the performance hit if any. Merged in 0957e478abbc12cf9ff36cdefb455dd0c5e663a1, thanks.

  • Status changed from Ready for review to Closed

Also available in: Atom PDF