rubolite 0.0.4 → 0.0.5

data/lib/rubolite/client.rb

@@ -1,7 +1,11 @@
 module Rubolite
   class Client
+    extend Forwardable
+
     attr_reader :admin, :ssh_keys
 
+    def_delegators :@admin, :git
+
     def initialize(admin)
       @admin = admin
       @ssh_keys = {}
@@ -40,8 +44,9 @@ module Rubolite
     end
 
     def commit!
-      admin.git.add('.')
-      admin.git.commit_all("Modified configuration by rubolite")
+      return unless commitable?
+      admin.git.add(".")
+      admin.git.commit "Modified configuration by rubolite"
     end
 
     def push!
@@ -61,5 +66,9 @@ module Rubolite
       @groups = nil
       @ssh_keys = {}
     end
+
+    def commitable?
+      admin.git.status.changed.size > 0
+    end
   end
 end

data/lib/rubolite/version.rb

@@ -1,3 +1,3 @@
 module Rubolite
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/spec/lib/rubolite/client_spec.rb

@@ -39,6 +39,10 @@ describe Rubolite::Client do
   end
 
   context "saving" do
+    let(:git) { stub status: status, add: nil }
+    let(:status) { stub changed: changed }
+    let(:changed) { Hash.new }
+
     it "saves changes" do
       admin.writer.should_receive(:write!)
       subject.save!
@@ -55,7 +59,14 @@ describe Rubolite::Client do
     end
 
     it "commits the changes to the repo" do
-      admin.git.should_receive(:commit_all)
+      admin.git.should_receive(:commit)
+      subject.commit!
+    end
+
+    it "commits only if things have changed" do
+      subject.admin.stub(git: git)
+
+      admin.git.should_not_receive(:commit_all)
       subject.commit!
     end
 
@@ -108,4 +119,19 @@ describe Rubolite::Client do
       expect(subject.ssh_keys.object_id).not_to eq(ssh_keys)
     end
   end
+
+  context "status reporting" do
+    let(:git) { stub status: status }
+    let(:status) { stub changed: changed }
+    let(:changed) { Hash.new }
+
+    before(:each) do
+      subject.admin.stub(git: git)
+    end
+
+    it "reports that it is commitable" do
+      changed.should_receive(:size).and_return(1)
+      expect(subject).to be_commitable
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubolite
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-19 00:00:00.000000000 Z
+date: 2012-11-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec