1 files changed, 117 insertions, 3 deletions
diff --git a/spec/unit/resource/chef_gem_spec.rb b/spec/unit/resource/chef_gem_spec.rb
index 480856d19f..7352a8f5fe 100644
--- a/spec/unit/resource/chef_gem_spec.rb
+++ b/spec/unit/resource/chef_gem_spec.rb
@@ -32,16 +32,130 @@ describe Chef::Resource::ChefGem, "initialize" do
 end
 
 describe Chef::Resource::ChefGem, "gem_binary" do
+  let(:resource) { Chef::Resource::ChefGem.new("foo") }
+
   before(:each) do
     expect(RbConfig::CONFIG).to receive(:[]).with('bindir').and_return("/opt/chef/embedded/bin")
-    @resource = Chef::Resource::ChefGem.new("foo")
   end
 
   it "should raise an exception when gem_binary is set" do
-    expect { @resource.gem_binary("/lol/cats/gem") }.to raise_error(ArgumentError)
+    expect { resource.gem_binary("/lol/cats/gem") }.to raise_error(ArgumentError)
+  end
+
+  it "should set the gem_binary based on computing it from RbConfig" do
+    expect(resource.gem_binary).to eql("/opt/chef/embedded/bin/gem")
   end
 
   it "should set the gem_binary based on computing it from RbConfig" do
-    expect(@resource.gem_binary).to eql("/opt/chef/embedded/bin/gem")
+    expect(resource.compile_time).to be nil
+  end
+
+  context "when building the resource" do
+    let(:node) do
+      Chef::Node.new.tap {|n| n.normal[:tags] = [] }
+    end
+
+    let(:run_context) do
+      Chef::RunContext.new(node, {}, nil)
+    end
+
+    let(:recipe) do
+      Chef::Recipe.new("hjk", "test", run_context)
+    end
+
+    let(:chef_gem_compile_time) { nil }
+
+    let(:resource) do
+      Chef::Config[:chef_gem_compile_time] = chef_gem_compile_time
+      Chef::Resource::ChefGem.new("foo", run_context)
+    end
+
+    before do
+      expect(Chef::Resource::ChefGem).to receive(:new).and_return(resource)
+    end
+
+    it "runs the install at compile-time by default", :chef_lt_13_only do
+      expect(resource).to receive(:run_action).with(:install)
+      expect(Chef::Log).to receive(:deprecation).at_least(:once)
+      recipe.chef_gem "foo"
+    end
+
+    # the default behavior will change in Chef-13
+    it "does not runs the install at compile-time by default", :chef_gte_13_only do
+      expect(resource).not_to receive(:run_action).with(:install)
+      expect(Chef::Log).not_to receive(:deprecation)
+      recipe.chef_gem "foo"
+    end
+
+    it "compile_time true installs at compile-time" do
+      expect(resource).to receive(:run_action).with(:install)
+      expect(Chef::Log).not_to receive(:deprecation)
+      recipe.chef_gem "foo" do
+        compile_time true
+      end
+    end
+
+    it "compile_time false does not install at compile-time" do
+      expect(resource).not_to receive(:run_action).with(:install)
+      expect(Chef::Log).not_to receive(:deprecation)
+      recipe.chef_gem "foo" do
+        compile_time false
+      end
+    end
+
+    describe "when Chef::Config[:chef_gem_compile_time] is explicitly true" do
+      let(:chef_gem_compile_time) { true }
+
+      before do
+        expect(Chef::Log).not_to receive(:deprecation)
+      end
+
+      it "by default installs at compile-time" do
+        expect(resource).to receive(:run_action).with(:install)
+        recipe.chef_gem "foo"
+      end
+
+      it "compile_time true installs at compile-time" do
+        expect(resource).to receive(:run_action).with(:install)
+        recipe.chef_gem "foo" do
+          compile_time true
+        end
+      end
+
+      it "compile_time false does not install at compile-time" do
+        expect(resource).not_to receive(:run_action).with(:install)
+        recipe.chef_gem "foo" do
+          compile_time false
+        end
+      end
+    end
+
+    describe "when Chef::Config[:chef_gem_compile_time] is explicitly false" do
+
+      let(:chef_gem_compile_time) { false }
+
+      before do
+        expect(Chef::Log).not_to receive(:deprecation)
+      end
+
+      it "by default does not install at compile-time" do
+        expect(resource).not_to receive(:run_action).with(:install)
+        recipe.chef_gem "foo"
+      end
+
+      it "compile_time true installs at compile-time" do
+        expect(resource).to receive(:run_action).with(:install)
+        recipe.chef_gem "foo" do
+          compile_time true
+        end
+      end
+
+      it "compile_time false does not install at compile-time" do
+        expect(resource).not_to receive(:run_action).with(:install)
+        recipe.chef_gem "foo" do
+          compile_time false
+        end
+      end
+    end
   end
 end