greybox 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6403c4bd8f174968087fb1add29ea027ffb2e502
-  data.tar.gz: eaf7e713f00cc65d609a5c80be3f94479da6d42a
+  metadata.gz: 7a780147167dfd89d0dbb4bed3c7fcde9ab6ccf6
+  data.tar.gz: 9176c7343d521e112c2a0b161245853cf8597ed2
 SHA512:
-  metadata.gz: c8ca2cc43f5b623785081aff0918db238fd10e052fe2ac23363dd97539d509afec441fc84b7ddf2c55dd04e7aaf596da0ec21c0c007f60944e0d65f2c6f103c0
-  data.tar.gz: 39f8f4521871ee01435b2a1a7829f8037352aaf42d56414e5f6755bcf63942f3e985d3620202dab9c3a06224680bcf8a7fdbda4fe78c031216d2db08407bf739
+  metadata.gz: f56d4b162b73ccc4de3f4158a6246ab9d398ad9e669507d06db06a8493886e97d7e3700fd2937296bd089bcc285a3da8e9090e2be2619a9943c57e1ba7f06fa7
+  data.tar.gz: 783dd3ee57449158dbfc8e5bb5dbbb4dfff3c8a1d0b74bd2f7053b5c408085b03fcbd4fbac062c8f7b9369f7d3adda38a368f7724064bf953e674605b2afbc06

data/README.md

@@ -30,7 +30,7 @@ end
 ```
 
 This defines a testing setup in which the command
-    
+
     $ ./blackbox_command < INPUT_FILE.input
 
 is used as the reference for correctness.

data/Rakefile

@@ -1,6 +1,8 @@
 require "bundler/gem_tasks"
 require "rake/testtask"
 
+task default: :test
+
 Rake::TestTask.new do |t|
   t.libs << "test"
   t.test_files = ["spec/spec_helper"] + FileList['spec/**/*_spec.rb']

data/greybox.gemspec

@@ -24,4 +24,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "rake"
   spec.add_development_dependency "fakefs"
   spec.add_development_dependency "mocha"
+  spec.add_development_dependency "simplecov"
 end

data/lib/greybox/configurable.rb

@@ -3,7 +3,6 @@ module Greybox
     attr_accessor :properties
 
     module ClassMethods
-      attr_accessor :defaults
       attr_accessor :required_properties
       def def_property(name, args = {})
         setter = "#{name}=".to_sym
@@ -32,6 +31,10 @@ module Greybox
         include mod
       end
 
+      def defaults
+        @defaults || {}
+      end
+
       private
       def new_blank_properties_module
         Module.new do
@@ -50,7 +53,7 @@ module Greybox
     end
 
     def get_prop(prop)
-      (@properties || {}).fetch(prop, self.class.defaults[prop]) 
+      (@properties || {}).fetch(prop, self.class.defaults[prop])
     end
 
     def verify

data/lib/greybox/configuration.rb

@@ -6,7 +6,7 @@ module Greybox
     def_property :test_command, required: true
     def_property :expected, default: ->(input) { input.gsub(/\.input$/, ".output") }
     def_property :comparison, default: ->(actual, expected) { actual == expected }
-    def_property :blackbox_command, required: true
+    def_property :blackbox_command
 
   end
 end

data/lib/greybox/runner.rb

@@ -7,6 +7,7 @@ module Greybox
       @config = config
       @failures = []
     end
+
     def files
       Dir.glob(config.input)
     end
@@ -19,8 +20,11 @@ module Greybox
 
     def expected_for(filename)
       file = expected_filename(filename)
-      File.open(file, 'w') do |f|
-        f.write run_command(config.blackbox_command, filename)
+      unless File.exist?(file)
+        raise "No blackbox_command available to generate #{file}" unless config.blackbox_command
+        File.open(file, 'w') do |f|
+          f.write run_command(config.blackbox_command, filename)
+        end
       end
       File.read(file)
     end

data/lib/greybox/version.rb

@@ -1,3 +1,3 @@
 module Greybox
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/spec/configurable_spec.rb

@@ -0,0 +1,81 @@
+module Greybox
+  describe Configurable do
+
+    def config
+      @config ||= configuration_class.new
+    end
+
+    describe "the inserted module" do
+      it "shows what properties have been defined" do
+        cls = Class.new do
+          include Configurable
+
+          def_property :some_property
+          def_property :some_other_property
+        end
+        cls.ancestors.map(&:to_s).must_include "Properties(some_property, some_other_property)"
+      end
+    end
+
+    describe "setting a property" do
+      def configuration_class
+        Class.new do
+          include Configurable
+
+          def_property :regular
+        end
+      end
+
+      it "lets you set valid properties" do
+        config.regular = "abc"
+        config.regular.must_equal "abc"
+      end
+
+      it "does not let you set invalid properties" do
+        -> do
+          config.unset = "abc"
+        end.must_raise NoMethodError
+      end
+    end
+
+    describe "required properties" do
+      def configuration_class
+        Class.new do
+          include Configurable
+
+          def_property :req, required: true
+        end
+      end
+
+      it "does not complain if the required property is set" do
+        config.req = "abc"
+        config.verify
+      end
+
+      it "complains if you do not set a property" do
+        -> do
+          config.verify
+        end.must_raise RuntimeError
+      end
+    end
+
+    describe "default values" do
+      def configuration_class
+        Class.new do
+          include Configurable
+
+          def_property :with_default, default: 5
+        end
+      end
+
+      it "uses the default value" do
+        config.with_default.must_equal 5
+      end
+
+      it "lets you override the default" do
+        config.with_default = 6
+        config.with_default.must_equal 6
+      end
+    end
+  end
+end

data/spec/greybox_spec.rb

@@ -8,5 +8,31 @@ module Greybox
         end.must_raise RuntimeError
       end
     end
+
+    describe "running the tests" do
+      it "defers to runner to actually run the tests" do
+        Runner.any_instance.expects(:run)
+        Greybox.setup do |c|
+          c.input = "test/*.input"
+          c.test_command = "echo hi"
+        end
+      end
+    end
+
+    describe "printing the failures" do
+      it "prints out each failure" do
+        Diffy::Diff.stubs(:new).returns("-a\n+b\n")
+        $stdout.expects(:puts).with("FAILED:")
+        $stdout.expects(:puts).with("filename: a.input")
+        $stdout.expects(:puts).with("-a\n+b\n")
+        Runner.any_instance.stubs(:failures).returns([{filename: "a.input",
+                                                       expected: "a",
+                                                       actual: "b"}])
+        Greybox.setup do |c|
+          c.input = "test/*.input"
+          c.test_command = "echo hi"
+        end
+      end
+    end
   end
 end

data/spec/runner_spec.rb

@@ -20,6 +20,7 @@ module Greybox
         input: "*.input",
         expected: ->(input) { "output_file" },
         blackbox_command: "echo BLACKBOX_COMMAND",
+        test_command: "echo hi",
         comparison: ->(actual, expected) { actual == expected },
       }
     end
@@ -65,16 +66,32 @@ module Greybox
       end
 
       it "uses the provided blackbox command to find the expected value for cases that don't have one yet" do
-        with_config(test_command: "echo hi",
-                    blackbox_command: "echo output")
+        with_config(blackbox_command: "echo output")
         runner.expects(:`).with "echo hi"
         runner.expects(:`).with "echo output"
         runner.run
       end
 
+      it "does not run the command if the file already exists" do
+        with_config(blackbox_command: "echo output")
+        FileUtils.touch("output_file")
+        runner.expects(:`).with "echo hi"
+        runner.run
+      end
+
+      it "complains if there is no blackbox_command AND the file does not exist" do
+        with_config(blackbox_command: nil)
+        -> { runner.run }.must_raise RuntimeError
+      end
+
+      it "does not complain if there is no blackbox_command and the file DOES exist" do
+        with_config(blackbox_command: nil)
+        FileUtils.touch("output_file")
+        runner.run
+      end
+
       it "inserts output filenames for %'s in the blackbox command" do
-        with_config(test_command: "echo hi",
-                    blackbox_command: "echo output < %")
+        with_config(blackbox_command: "echo output < %")
         runner.expects(:`).with "echo hi"
         runner.expects(:`).with "echo output < inputfile.input"
         runner.run
@@ -83,22 +100,19 @@ module Greybox
 
     describe "asserting output" do
       it "compares for equality by default" do
-        with_config(test_command: "echo hi",
-                    blackbox_command: "echo hi")
+        with_config(blackbox_command: "echo hi")
         runner.run
         runner.failures.must_be_empty
       end
 
       it "has a failure if a test fails" do
-        with_config(test_command: "echo hi",
-                    blackbox_command: "echo hello")
+        with_config(blackbox_command: "echo hello")
         runner.run
         runner.failures.wont_be_empty
       end
 
       it "compares by the provided comparison" do
         with_config(comparison: ->(actual, expected) { actual == "hi\n" },
-                    test_command: "echo hi",
                     blackbox_command: "echo hello")
         runner.run
         runner.failures.must_be_empty

data/spec/spec_helper.rb

@@ -1,3 +1,6 @@
+require 'simplecov'
+SimpleCov.start
+
 require "minitest/autorun"
 require "mocha/setup"
 require "fakefs"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: greybox
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Justin Jaffray
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-30 00:00:00.000000000 Z
+date: 2013-10-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: diffy
@@ -80,6 +80,20 @@ dependencies:
     - - '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Test against a black box.
 email:
 - justin.jaffray@gmail.com
@@ -104,6 +118,7 @@ files:
 - lib/greybox/configuration.rb
 - lib/greybox/runner.rb
 - lib/greybox/version.rb
+- spec/configurable_spec.rb
 - spec/greybox_spec.rb
 - spec/runner_spec.rb
 - spec/spec_helper.rb
@@ -132,6 +147,7 @@ signing_key:
 specification_version: 4
 summary: Easily test school assignments or other projects where you have a reference
 test_files:
+- spec/configurable_spec.rb
 - spec/greybox_spec.rb
 - spec/runner_spec.rb
 - spec/spec_helper.rb