Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Ben Orenstein

Ben Orenstein

Member of 1 project: Ruby on Rails

Ben Orenstein's latest activity

Monday, February 28 2011
6483 ActiveSupport::OrderedOptions calls to_sym redundantly was created in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein created the ticket at 1:31 AM

    There's no need to call to_sym on the matched method name. The [] method already does this.
Thursday, February 17 2011
6442 Remove unused code in test was created in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein created the ticket at 5:13 AM

    Patch removes a line in test setup that never gets used.
Wednesday, February 16 2011
6437 Remove unused code in test was created in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein created the ticket at 2:35 AM

    Looks like an empty class got created for testing purposes and never got used. Attached patch removes it.
Friday, February 04 2011
6375 Fix incorrect regex in test was created in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein created the ticket at 7:11 PM

    The regex for test_existing_migration_is_removed_on_force is incorrect (it searches for create_foos instead of create_accounts).

    The included patch fixes this.
Tuesday, May 25 2010
4691 Use new assertion methods in test was updated in Ruby on Rails
  • Ben Orenstein
    • Tag changed from patched to patch
    by Ben Orenstein at 8:26 PM
  • Ben Orenstein
    • Tag changed from assertions, patched to patched
    by Ben Orenstein at 8:25 PM
  • Ben Orenstein
    • Tag changed from assertions to assertions, patched
    by Ben Orenstein at 8:25 PM
  • Ben Orenstein
    Ben Orenstein commented at 7:15 PM
  • Ben Orenstein
    Ben Orenstein created the ticket at 7:13 PM

    This previous patch added two new assertions: assert_present and assert_blank.

    My patch refactors several tests to use these new methods.

    assert_blank foo is su...
Sunday, May 16 2010
3675 form_for doesn't work with new singleton nested resource was updated in Ruby on Rails
  • Ben Orenstein
    Ben Orenstein commented at 8:50 PM

    I've got a failing test for this in rails3 beta 3. Working on a patch.