Red Hat Bugzilla – Bug 588442
Review Request: rubygem-heckle - unit test sadism(tm) at it's core
Last modified: 2010-12-01 13:14:34 EST
Spec URL: http://github.com/admiyo/MySpecs/blob/master/rubygem-heckle.spec
SRPM URL: http://admiyo.fedorapeople.org/buildr-repo/rubygem-heckle-1.4.3-1.young.src.rpm
Description: Heckle is unit test sadism(tm) at it's core. Heckle is a mutation tester. It modifies your code and runs your tests to make sure they fail. The idea is that if code can be changed and your tests don't notice, either that code isn't being covered or it doesn't do anything. It's like hiring a white-hat hacker to try to break into your server and making sure you detect it. You learn the most by trying to break things and watching the outcome in an act of unit test sadism.
warning: File listed twice: /usr/share/doc/usr/lib/ruby/gems/1.8-1.4.3
warning: File listed twice: /usr/share/doc/usr/lib/ruby/gems/1.8-1.4.3/History.txt
warning: File listed twice: /usr/share/doc/usr/lib/ruby/gems/1.8-1.4.3/Manifest.txt
warning: File listed twice: /usr/share/doc/usr/lib/ruby/gems/1.8-1.4.3/README.txt
rubygem-heckle.src: W: patch-not-applied Patch0: heckle-local.patch
rubygem-heckle.noarch: E: non-executable-script /usr/lib/ruby/gems/1.8/gems/heckle-1.4.3/lib/test_unit_heckler.rb 0444L /usr/bin/env
rubygem-heckle.noarch: E: non-standard-executable-perm /usr/lib/ruby/gems/1.8/gems/heckle-1.4.3/bin/heckle 0555L
rubygem-heckle.noarch: W: hidden-file-or-dir /usr/lib/ruby/gems/1.8/gems/heckle-1.4.3/.autotest
rubygem-heckle.noarch: E: non-executable-script /usr/lib/ruby/gems/1.8/gems/heckle-1.4.3/sample/test/test_heckled.rb 0444L /usr/bin/env
Again, ruby_sitelib macro isn't used. Remove that line.
No need for the /usr/share/doc setup again. Please remove, if you do, the Rakefile will work for rake test in %check. You will also need to add a BR on parse tree for the check.
Rather than patching, you could simply run sed to modify the path for ruby.
This initially started as a dependency for rubygem-buildr. It is no longer
needed for that. We can cancel this review.