Rails AR/Oracle Unit Test: [5310] failed (but getting better)

"rick" has given AR/Oracle some love, but it's still unhappy...

http://dev.rubyonrails.org/changeset/5310

Michael Schoen wrote:

"rick" has given AR/Oracle some love, but it's still unhappy...

I've submitted a patch to resolve the remaining AR/Oracle test failures; I backed out the still-in-progress bind variable changes, so that this could be applied before the 1.2 release.

  http://dev.rubyonrails.org/ticket/6515

Resending this with a more enticing subject line...

Michael Schoen wrote:

"rick" has given AR/Oracle some love, but it's still unhappy...

I've submitted a patch to resolve the remaining AR/Oracle test failures;
I backed out the still-in-progress bind variable changes, so that this
could be applied before the 1.2 release.

  http://dev.rubyonrails.org/ticket/6515

Resending this with a more enticing subject line…

Thanks for the reminder!

Michael Schoen wrote:

“rick” has given AR/Oracle some love, but it’s still unhappy…

I’ve submitted a patch to resolve the remaining AR/Oracle test failures;
I backed out the still-in-progress bind variable changes, so that this

could be applied before the 1.2 release.

    [http://dev.rubyonrails.org/ticket/6515](http://dev.rubyonrails.org/ticket/6515)

I applied with minor differences but omitted the changes to associations.rb.
See the comments on the ticket.

Thanks,
jeremy

Jeremy Kemper wrote:

I applied with minor differences but omitted the changes to associations.rb.
See the comments on the ticket.

Thanks. Omitting that bit still leaves a failed test. I updated the ticket, please let me know how you want to proceed. I can put together a cleaner patch in a bit if that's ok.

Michael A. Schoen wrote:

Jeremy Kemper wrote:

I applied with minor differences but omitted the changes to associations.rb.
See the comments on the ticket.

Thanks. Omitting that bit still leaves a failed test. I updated the ticket, please let me know how you want to proceed. I can put together a cleaner patch in a bit if that's ok.

I've updated the ticket w/ a patch for the remaining failing test.

Applied to trunk and 1.2rc. Thanks Michael!

jeremy