locriani / magmawiki

A wiki.
magmawiki.com
Other
13 stars 2 forks source link

rspec and shoulda? #5

Closed NobbZ closed 13 years ago

NobbZ commented 13 years ago

Why are there rspec and shoulda in the Gemfile?

So far as I can tell both are used for the same purpose, they are both a replacement for unit tests, or do I missunderstand things?

Additionally it seems that shoulda is not really used in the tests, so can we remove it safely?

NobbZ commented 13 years ago

Also I think a cleanup of features, specs or/and tests is quite necessary. I think this cleanup fits very well into the fact that there are two testing frameworks required in the Gemfile...

epitron commented 13 years ago

Agreed. I just committed what I was working on yesterday -- removing extraneous things from the Gemfile.

The tests are out of date in other ways too -- Factory Girl deprecated the use of the 'Factory' module.

Nothing complicated... just a little maintenance work. :)

locriani commented 13 years ago

Shoulda extends RSpec functionality, it's not a separate test / test harness framework.

Sent from my iPhone

On Aug 11, 2011, at 3:31, NobbZreply@reply.github.com wrote:

Why are there rspec and shoulda in the Gemfile?

So far as I can tell both are used for the same purpose, they are both a replacement for unit tests, or do I missunderstand things?

Additionally it seems that shoulda is not really used in the tests, so can we remove it safely?

Reply to this email directly or view it on GitHub: https://github.com/locriani/magmawiki/issues/5

epitron commented 13 years ago

Whoops, sorry. I was confusing it with bacon or something.