rscons 1.3.0 → 1.4.0

data/spec/rscons/environment_spec.rb

@@ -140,7 +140,8 @@ module Rscons
         env.Program("a.out", "main.c")
 
         cache = "cache"
-        Cache.should_receive(:new).and_return(cache)
+        Cache.should_receive(:instance).and_return(cache)
+        cache.should_receive(:clear_checksum_cache!)
         env.should_receive(:run_builder).with(anything, "a.out", ["main.c"], cache, {}).and_return(true)
         cache.should_receive(:write)
 
@@ -153,7 +154,8 @@ module Rscons
         env.Object("main.o", "other.cc")
 
         cache = "cache"
-        Cache.should_receive(:new).and_return(cache)
+        Cache.should_receive(:instance).and_return(cache)
+        cache.should_receive(:clear_checksum_cache!)
         env.should_receive(:run_builder).with(anything, "main.o", ["other.cc"], cache, {}).and_return("main.o")
         env.should_receive(:run_builder).with(anything, "a.out", ["main.o"], cache, {}).and_return("a.out")
         cache.should_receive(:write)
@@ -167,12 +169,28 @@ module Rscons
         env.Object("main.o", "other.cc")
 
         cache = "cache"
-        Cache.should_receive(:new).and_return(cache)
+        Cache.should_receive(:instance).and_return(cache)
+        cache.should_receive(:clear_checksum_cache!)
         env.should_receive(:run_builder).with(anything, "main.o", ["other.cc"], cache, {}).and_return(false)
         cache.should_receive(:write)
 
         expect { env.process }.to raise_error BuildError, /Failed.to.build.main.o/
       end
+
+      it "writes the cache when the Builder raises an exception" do
+        env = Environment.new
+        env.Object("module.o", "module.c")
+
+        cache = "cache"
+        Cache.should_receive(:instance).and_return(cache)
+        cache.should_receive(:clear_checksum_cache!)
+        env.stub(:run_builder) do |builder, target, sources, cache, vars|
+          raise "Ruby exception thrown by builder"
+        end
+        cache.should_receive(:write)
+
+        expect { env.process }.to raise_error RuntimeError, /Ruby exception thrown by builder/
+      end
     end
 
     describe "#clear_targets" do
@@ -249,8 +267,7 @@ module Rscons
     describe "#method_missing" do
       it "calls the original method missing when the target method is not a known builder" do
         env = Environment.new
-        env.should_receive(:orig_method_missing).with(:foobar)
-        env.foobar
+        expect {env.foobar}.to raise_error /undefined method .foobar./
       end
 
       it "records the target when the target method is a known builder" do
@@ -260,7 +277,7 @@ module Rscons
         target = env.instance_variable_get(:@targets)["target"]
         target.should_not be_nil
         target[:builder].is_a?(Builder).should be_true
-        target[:source].should == ["src1", "src2"]
+        target[:sources].should == ["src1", "src2"]
         target[:vars].should == {var: "val"}
         target[:args].should == []
       end

data/spec/rscons/varset_spec.rb

@@ -1,6 +1,6 @@
 module Rscons
   describe VarSet do
-    describe '.initialize' do
+    describe '#initialize' do
       it "initializes variables from a Hash" do
         v = VarSet.new({"one" => 1, "two" => :two})
         v["one"].should == 1
@@ -20,10 +20,10 @@ module Rscons
       end
     end
 
-    describe :[] do
-      v = VarSet.new({"fuz" => "a string", "foo" => 42, "bar" => :baz,
-                      "qax" => [3, 6], "qux" => {a: :b}})
+    describe "#[]" do
       it "allows accessing a variable with its verbatim value if type is not specified" do
+        v = VarSet.new({"fuz" => "a string", "foo" => 42, "bar" => :baz,
+                        "qax" => [3, 6], "qux" => {a: :b}})
         v["fuz"].should == "a string"
         v["foo"].should == 42
         v["bar"].should == :baz
@@ -32,7 +32,7 @@ module Rscons
       end
     end
 
-    describe :[]= do
+    describe "#[]=" do
       it "allows assigning to variables" do
         v = VarSet.new("CFLAGS" => ["-Wall", "-O3"])
         v["CPPPATH"] = ["one", "two"]
@@ -41,40 +41,86 @@ module Rscons
       end
     end
 
-    describe '.append' do
+    describe "#include?" do
+      it "returns whether the variable is in the VarSet" do
+        v = VarSet.new("CFLAGS" => [], :foo => :bar)
+
+        expect(v.include?("CFLAGS")).to be_true
+        expect(v.include?(:CFLAGS)).to be_false
+        expect(v.include?(:foo)).to be_true
+        expect(v.include?("foo")).to be_false
+        expect(v.include?("bar")).to be_false
+
+        v2 = v.clone
+        v2.append("bar" => [])
+
+        expect(v2.include?("CFLAGS")).to be_true
+        expect(v2.include?(:CFLAGS)).to be_false
+        expect(v2.include?(:foo)).to be_true
+        expect(v2.include?("foo")).to be_false
+        expect(v2.include?("bar")).to be_true
+      end
+    end
+
+    describe '#append' do
       it "adds values from a Hash to the VarSet" do
         v = VarSet.new("LDFLAGS" => "-lgcc")
         v.append("LIBS" => "gcc", "LIBPATH" => ["mylibs"])
-        v.vars.keys.should =~ ["LDFLAGS", "LIBS", "LIBPATH"]
+        expect(v["LDFLAGS"]).to eq("-lgcc")
+        expect(v["LIBS"]).to eq("gcc")
+        expect(v["LIBPATH"]).to eq(["mylibs"])
       end
+
       it "adds values from another VarSet to the VarSet" do
         v = VarSet.new("CPPPATH" => ["mydir"])
         v2 = VarSet.new("CFLAGS" => ["-O0"], "CPPPATH" => ["different_dir"])
         v.append(v2)
-        v.vars.keys.should =~ ["CPPPATH", "CFLAGS"]
-        v["CPPPATH"].should == ["different_dir"]
+        expect(v["CFLAGS"]).to eq(["-O0"])
+        expect(v["CPPPATH"]).to eq(["different_dir"])
+      end
+
+      it "does not pick up subsequent variable changes from a given VarSet" do
+        v = VarSet.new("dirs" => ["a"])
+        v2 = VarSet.new
+        v2.append(v)
+        v["dirs"] << "b"
+        expect(v["dirs"]).to eq(["a", "b"])
+        expect(v2["dirs"]).to eq(["a"])
       end
     end
 
-    describe '.merge' do
+    describe '#merge' do
       it "returns a new VarSet merged with the given Hash" do
         v = VarSet.new("foo" => "yoda")
         v2 = v.merge("baz" => "qux")
-        v.vars.keys.should == ["foo"]
-        v2.vars.keys.should =~ ["foo", "baz"]
+        expect(v["foo"]).to eq("yoda")
+        expect(v2["foo"]).to eq("yoda")
+        expect(v2["baz"]).to eq("qux")
       end
+
       it "returns a new VarSet merged with the given VarSet" do
         v = VarSet.new("foo" => ["a", "b"], "bar" => 42)
         v2 = v.merge(VarSet.new("bar" => 33, "baz" => :baz))
         v2["foo"] << "c"
-        v["foo"].should == ["a", "b"]
-        v["bar"].should == 42
-        v2["foo"].should == ["a", "b", "c"]
-        v2["bar"].should == 33
+        expect(v["foo"]).to eq ["a", "b"]
+        expect(v["bar"]).to eq 42
+        expect(v2["foo"]).to eq ["a", "b", "c"]
+        expect(v2["bar"]).to eq 33
+      end
+
+      it "does not pick up subsequent variable changes from a given VarSet" do
+        v = VarSet.new("var" => ["a", "b"], "var2" => ["1", "2"])
+        v["var2"] << "3"
+        v2 = v.clone
+        v["var"] << "c"
+        expect(v["var"]).to eq(["a", "b", "c"])
+        expect(v["var2"]).to eq(["1", "2", "3"])
+        expect(v2["var"]).to eq(["a", "b"])
+        expect(v2["var2"]).to eq(["1", "2", "3"])
       end
     end
 
-    describe '.expand_varref' do
+    describe '#expand_varref' do
       v = VarSet.new("CFLAGS" => ["-Wall", "-O2"],
                      "CC" => "gcc",
                      "CPPPATH" => ["dir1", "dir2"],

data/spec/rscons_spec.rb

@@ -2,7 +2,7 @@ describe Rscons do
   describe ".clean" do
     it "removes all build targets and created directories" do
       cache = "cache"
-      Rscons::Cache.should_receive(:new).and_return(cache)
+      Rscons::Cache.should_receive(:instance).and_return(cache)
       cache.should_receive(:targets).and_return(["build/a.out", "build/main.o"])
       FileUtils.should_receive(:rm_f).with("build/a.out")
       FileUtils.should_receive(:rm_f).with("build/main.o")
@@ -18,7 +18,7 @@ describe Rscons do
       Dir.should_receive(:rmdir).with("build")
       File.should_receive(:directory?).with("other").and_return(true)
       Dir.should_receive(:entries).with("other").and_return([".", "..", "other.file"])
-      Rscons::Cache.should_receive(:clear)
+      cache.should_receive(:clear)
 
       Rscons.clean
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rscons
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.4.0
 platform: ruby
 authors:
 - Josh Holtrop
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-25 00:00:00.000000000 Z
+date: 2014-04-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec-core
@@ -166,8 +166,11 @@ files:
 - build_tests/two_sources/two.c
 - lib/rscons.rb
 - lib/rscons/builder.rb
+- lib/rscons/builders/cfile.rb
+- lib/rscons/builders/disassemble.rb
 - lib/rscons/builders/library.rb
 - lib/rscons/builders/object.rb
+- lib/rscons/builders/preprocess.rb
 - lib/rscons/builders/program.rb
 - lib/rscons/cache.rb
 - lib/rscons/environment.rb
@@ -175,6 +178,7 @@ files:
 - lib/rscons/version.rb
 - rscons.gemspec
 - spec/build_tests_spec.rb
+- spec/rscons/builders/cfile_spec.rb
 - spec/rscons/cache_spec.rb
 - spec/rscons/environment_spec.rb
 - spec/rscons/varset_spec.rb
@@ -206,6 +210,7 @@ specification_version: 4
 summary: Software construction library inspired by SCons and implemented in Ruby
 test_files:
 - spec/build_tests_spec.rb
+- spec/rscons/builders/cfile_spec.rb
 - spec/rscons/cache_spec.rb
 - spec/rscons/environment_spec.rb
 - spec/rscons/varset_spec.rb