Project

General

Profile

Actions

Task #2930

closed

Feature #2428: upgrade to ruby 1.9.3

Make all tests pass and without deprecation warnings in 1.8.7

Added by Phil Hodgson almost 10 years ago. Updated over 9 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Phil Hodgson
Actions #1

Updated by Phil Hodgson almost 10 years ago

  • Subject changed from Get rid of deprecation warnings in all tests to Make all tests pass and without deprecation warnings
Actions #2

Updated by Phil Hodgson over 9 years ago

  • Status changed from New to In Progress
  • Start date set to 06/30/2014
Actions #3

Updated by Phil Hodgson over 9 years ago

  • Subject changed from Make all tests pass and without deprecation warnings to Make all tests pass and without deprecation warnings in 1.8.7
Actions #4

Updated by Phil Hodgson over 9 years ago

Annoying deprecation warning on should_allow_values_for and should_not_allow_values_for: we should switch to should allow_value(*values).for(attribute) but if there is more than one value we get a too many arguments error! This must be because of where shoulda is frozen in the Gemfile currently (see commit 89c0d5a02686bbd8b9746ce1849d880a7cd2b0a7). I can't get rid of the deprecation "properly" without separating these calls onto many lines (as I had to for validates_presence_of), and since I believe the problem will disappear after we upgrade Ruby and the shoulda gem, I'm going to leave these in and fix them when Ruby and shoulda gets upgraded. As long as there aren't too many exceptions like this, we should be able to keep things under control.

Actions #5

Updated by Phil Hodgson over 9 years ago

There are a series of tests related to User#in_screening_eligibility_group but it seems as if Tapestry grew out of using such groupings. There is no longer this method in the User model. However, there seem to be a residual controller and a lot of "waitlist" logic in the code base, and it's not yet clear to me whether all of this has been abandoned and should be removed entirely (which would help clean things up a bit, to be sure!), or whether this code should be maintained in the event that similar Enrollment Steps and conditions will be applicable again in the future.

In the meantime I am going to at least remove the in_screening_eligibility_group tests, since it can't make sense to test a method that was removed.

Actions #6

Updated by Phil Hodgson over 9 years ago

  • Status changed from In Progress to Resolved
  • Remaining (hours) set to 0.0
Actions

Also available in: Atom PDF