rake spec:extensions EXT=multi_site exits with 0 although there are failing specs

14 views
Skip to first unread message

Johannes Fahrenkrug

unread,
May 8, 2012, 11:25:45 AM5/8/12
to Radiant CMS: Dev
Could this be a radiant bug or a multi_site bug?

See: http://travis-ci.org/#!/radiant/radiant-multi-site-extension/jobs/1276398

Jim Gay

unread,
May 8, 2012, 11:28:33 AM5/8/12
to radiant...@googlegroups.com
There's something wrong. The specs return 0 when they fail.
I've seen this in other extensions as well.

This commit in RSpec is related, but only mentions the before :all
scenario https://github.com/rspec/rspec-core/commit/79273a74a83ea3210c7abc219d840da4f1fdb0d0
> --
> Radiant CMS Dev Mailing List
> Post:        radiant...@googlegroups.com
> Unsubscribe: radiantcms-de...@googlegroups.com
> Group Site:  http://groups.google.com/group/radiantcms-dev/



--
Write intention revealing code #=> http://www.clean-ruby.com

Jim Gay
Saturn Flyer LLC
571-403-0338

Johannes Fahrenkrug

unread,
May 8, 2012, 11:30:16 AM5/8/12
to radiant...@googlegroups.com
I see, so it might be an RSpec bug?
springenwerk.com | github.com/jfahrenkrug | twitter.com/jfahrenkrug

Jim Gay

unread,
May 8, 2012, 11:35:27 AM5/8/12
to radiant...@googlegroups.com
Yes.

And since we're still on an old version of both rails and rspec, we
might be stuck with it... I haven't taken the time to track it down to
see if we could fix it.

On Tue, May 8, 2012 at 11:30 AM, Johannes Fahrenkrug

Johannes Fahrenkrug

unread,
May 8, 2012, 11:38:00 AM5/8/12
to radiant...@googlegroups.com
Ugh. That would be ugly. Maybe we can find a way (monkey patch if
worse comes to worse).
Springenwerk Consulting
Johannes Fahrenkrug
Muehlenredder 21c, 24582 Bordesholm, Germany
+49 (0)151 22 634 651

Johannes Fahrenkrug

unread,
May 9, 2012, 6:05:20 AM5/9/12
to radiant...@googlegroups.com
This is also related: https://github.com/rspec/rspec-core/pull/410

Do you know since when this is happening in radiant projects?

Johannes Fahrenkrug

unread,
May 9, 2012, 6:57:21 AM5/9/12
to radiant...@googlegroups.com
Hi,

this seems to be exactly what we need:
http://spectator.in/2011/02/06/rspecs-1-dot-3-1-exit-codes/

Where exactly should we put the monkey patch?

- Johannes

On Wed, May 9, 2012 at 12:05 PM, Johannes Fahrenkrug
--
springenwerk.com | github.com/jfahrenkrug | twitter.com/jfahrenkrug

Jim Gay

unread,
May 9, 2012, 3:20:12 PM5/9/12
to radiant...@googlegroups.com
Does this actually fix the error on Travis?

I'm confused by the comment at the end of that article "This issue has
been fixed in Ruby 1.9.2-p180."
Is this a Ruby problem? Why would the exit codes be incorrect on 1.9.3 if it is?

On Wed, May 9, 2012 at 6:57 AM, Johannes Fahrenkrug

Johannes Fahrenkrug

unread,
May 10, 2012, 3:09:41 AM5/10/12
to radiant...@googlegroups.com
That message confused me too. But this (more elaborate) ticket doesn't
mention Ruby 1.9.2: https://github.com/rspec/rspec-core/pull/410

I haven't tried it with Travis yet: Monkey-patching RSpec proves to be
difficult.
Reply all
Reply to author
Forward
0 new messages