paulczar / meez

rubygem to set up opinionated TDD chef cookbook
73 stars 17 forks source link

ruby error in chefspec part of strainer test #6

Closed stackedsax closed 10 years ago

stackedsax commented 10 years ago

When I run bundle exec strainer test, I get this error:

/Users/ascammon/.rvm/gems/ruby-1.9.3-p327@meez/gems/json-1.7.7/lib/json/common.rb:285:in `generate': 810: NaN not allowed in JSON (JSON::GeneratorError)

Here's the chefspec excerpt:

chefspec             | bundle exec rspec /Users/ascammon/dev/meez/tmp/test/test/unit/spec
chefspec             | *
chefspec             | Pending:
chefspec             | test::default ubuntu writes some chefspec code
chefspec             | # todo
chefspec             | # /Users/ascammon/dev/meez/tmp/test/test/unit/spec/default_spec.rb:10
chefspec             | Finished in 0.03139 seconds
chefspec             | 1 example, 0 failures, 1 pending
chefspec             | /Users/ascammon/.rvm/gems/ruby-1.9.3-p327@meez/gems/json-1.7.7/lib/json/common.rb:285:in `generate': 810: NaN not allowed in JSON (JSON::GeneratorError)
chefspec             | from /Users/ascammon/.rvm/gems/ruby-1.9.3-p327@meez/gems/json-1.7.7/lib/json/common.rb:285:in `pretty_generate'
chefspec             | from /Users/ascammon/.rvm/gems/ruby-1.9.3-p327@meez/gems/chefspec-3.1.4/lib/chefspec/coverage.rb:88:in `block in report!'
chefspec             | from /Users/ascammon/.rvm/gems/ruby-1.9.3-p327@meez/gems/chefspec-3.1.4/lib/chefspec/coverage.rb:87:in `open'
chefspec             | from /Users/ascammon/.rvm/gems/ruby-1.9.3-p327@meez/gems/chefspec-3.1.4/lib/chefspec/coverage.rb:87:in `report!'
chefspec             | from /private/Users/ascammon/dev/meez/tmp/test/test/unit/spec/spec_helper.rb:19:in `block in <top (required)>'
chefspec             | Terminated with a non-zero exit status. Strainer assumes this is a failure.
chefspec             | FAILURE!

I'll try and take a look at it later, but it's late and I just wanted to report it.

flaccid commented 10 years ago

I ran into it to, its fixed via this pull request I raised, https://github.com/paulczar/meez/pull/4.

paulczar commented 10 years ago

the fix for this was merged in.