[Ironruby-core] Code Review: mspec-12

John Lam (IRONRUBY) jflam at microsoft.com
Thu Jun 26 23:15:44 EDT 2008

Jim Deville:

> Test issues:
> * Instead of testing for ENV["HOME"] with
>     unless ENV["HOME"].nil?
>   why not use
>     if ENV["HOME"]
>   it's a little more idiomatic.


> * All of the _spec.rb.txt files have been renamed to _tags.txt. You'll
> need to fix your changeset for that.


> * The new rakefile doesn't allow testing methods directly. Any reason?
> Also, I'd prefer to keep the test tasks namespaced for organization
> sake. Can you at least move them into the mspec namespace with the rest
> of the mspec tasks? I'll remove redundant tasks later.

It's done to keep typing to a minimum. I don't like having to type more than I have to (and rake regression is already quite long). I replaced rake test to keep things short as well. I realize that folks can define aliases for these, but I'd rather the out of box experience be better than that.


More information about the Ironruby-core mailing list