#4797 AR tests suite warnings - Ruby on Rails

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

This project is archived and is in readonly mode.

#4797 ✓resolved

AR tests suite warnings

Reported by Emilio Tagua | June 8th, 2010 @ 07:24 PM | in 3.x

when running tests in AR you get these warnings:

./test/cases/adapter_test.rb:148: warning: ambiguous first argument; put parentheses or even spaces ./test/cases/adapter_test.rb:149: warning: ambiguous first argument; put parentheses or even spaces ./test/cases/adapter_test.rb:154: warning: ambiguous first argument; put parentheses or even spaces ./test/cases/adapter_test.rb:155: warning: ambiguous first argument; put parentheses or even spaces

patch attached to fix this.

warnings_fix.diff 1.6 KB

Comments and changes to this ticket

Prem Sichanugrist (sikachu) June 9th, 2010 @ 02:47 AM
- State changed from “new” to “resolved”
This one is already fixed in the master. Thank you :)

[ab2877cbe89e266ee986fc12e603abd93ac017ad]
You flagged this item as spam.
Andrea Campi October 11th, 2010 @ 07:19 AM
- Tag changed from “patch activerecord” to “activerecord, patch”
- Importance changed from “” to “Low”
bulk tags cleanup