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

rubyonrails-core
1

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

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

2

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

3

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

4

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

5

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.

6

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.

7

Applied to trunk and 1.2rc. Thanks Michael!

jeremy