Forum: Rails-core (closed, excessive spam) Failing Test in MySQL 5.0.51b

Posted by Ryan Bates (Guest)
on 2008-05-21 17:54
(Received via mailing list)
I'm getting this failing test when running the ActiveRecord mysql
tests on MySQL 5.0.51b (on latest master branch).

  1) Failure:
test_nil_defaults_for_not_null_columns(DefaultTest)
    [./test/cases/defaults_test.rb:17:in
`test_nil_defaults_for_not_null_columns'
     ./test/cases/defaults_test.rb:14:in `each'
     ./test/cases/defaults_test.rb:14:in
`test_nil_defaults_for_not_null_columns'
     ./test/cases/../../lib/../../activesupport/lib/active_support/
testing/setup_and_teardown.rb:67:in `__send__'
     ./test/cases/../../lib/../../activesupport/lib/active_support/
testing/setup_and_teardown.rb:67:in `run']:
name column should be DEFAULT "".
<""> expected but was
<nil>.

It appears MySQL has changed how its defaults work, but before I try
to make a patch, can anyone confirm this problem on their system? Or
is it something wrong with my setup?
Posted by Pratik Naik (pratik)
on 2008-05-21 18:09
(Received via mailing list)
Yup. Rick faced it a while back
http://rails.lighthouseapp.com/projects/8994/ticke...

On Wed, May 21, 2008 at 4:53 PM, Ryan Bates <bates.ryan@gmail.com> 
wrote:
> `test_nil_defaults_for_not_null_columns'
> is it something wrong with my setup?
>
> >
>



--
Cheers!
- Pratik
http://m.onkey.org
Posted by Ryan Bates (Guest)
on 2008-05-21 18:36
(Received via mailing list)
On May 21, 9:08 am, Pratik <pratikn...@gmail.com> wrote:
> Yup. Rick faced it a while backhttp://rails.lighthouseapp.com/projects/8994/tickets/192

Glad to see I'm not the only one. Has anyone come up with a good
solution to this problem? I guess we have to determine the MySQL
version at run time and add a conditional statement in there.
Posted by Ryan Bates (Guest)
on 2008-05-21 18:55
(Received via mailing list)
I added a patch to the ticket which fixes the problem for me. If
someone could please test it and make sure it works for them that
would be great.

http://rails.lighthouseapp.com/projects/8994/ticke...
Posted by Frederick Cheung (Guest)
on 2008-05-22 11:57
(Received via mailing list)
On 21 May 2008, at 17:55, Ryan Bates wrote:

>
> I added a patch to the ticket which fixes the problem for me. If
> someone could please test it and make sure it works for them that
> would be great.
>
Those tests fail on 4.1 (in a different way. sigh) before & after your
patch. I'll have a look at what mysql have changed

Fred
Posted by Jan De Poorter (Guest)
on 2008-05-22 12:09
(Received via mailing list)
This reminds me to ask,

is there still a CI system running (CruiseControl.rb or something)
active for the current Rails repository? And where is it mailing?

rgrds,
Jan
Posted by Pratik Naik (pratik)
on 2008-05-22 12:16
(Received via mailing list)
Don't think it's running anymore.

On Thu, May 22, 2008 at 11:08 AM, Jan De Poorter <jan@openminds.be> 
wrote:
>
>> patch. I'll have a look at what mysql have changed
>>>> solution to this problem? I guess we have to determine the MySQL
>>>> version at run time and add a conditional statement in there.
>>>>
>>
>>
>> >
>
>
> >
>



--
Cheers!
- Pratik
http://m.onkey.org
Posted by Chad Woolley (Guest)
on 2008-05-22 19:14
(Received via mailing list)
On Thu, May 22, 2008 at 3:08 AM, Jan De Poorter <jan@openminds.be> 
wrote:
>
> This reminds me to ask,
>
> is there still a CI system running (CruiseControl.rb or something)
> active for the current Rails repository? And where is it mailing?
>
> rgrds,
> Jan

I'm gonna work with Alexey to get this going again soon.  Sometime
next month, after RailsConf (which is currently taking all my free
time to prepare for)...

Git support is now in ccrb trunk, so theoretically nothing holding is
us back.  Other than somebody doing it ;)
This topic is locked and can not be replied to.