bamboo-client 0.0.2 → 0.0.3

data/features/remote.feature

@@ -15,4 +15,7 @@ Feature: Bamboo Remote client
   Scenario: Fetch a build result
     When I fetch a build result
     Then the build result should have a key
-    And the build result should have a state
+    And the build result should have a state
+
+  Scenario: Log out
+    Then I should be able to log out

data/features/step_definitions/remote_steps.rb

@@ -33,3 +33,7 @@ end
 Then /^I should be able to get the latest result$/ do
   @builds.first.latest_results.should be_kind_of(Bamboo::Client::Remote::BuildResult)
 end
+
+Then /^I should be able to log out$/ do
+  client.logout
+end

data/lib/bamboo-client/http/xml.rb

@@ -12,6 +12,7 @@ module Bamboo
           def initialize(doc)
             @doc = doc
             puts doc.to_s if $DEBUG
+            assert_no_error
           end
 
           def text_for(css)
@@ -28,6 +29,15 @@ module Bamboo
 
             klass.new node, *extra_args
           end
+
+          private
+
+          def assert_no_error
+            errors = @doc.css("errors error").map { |e| e.text }
+            unless errors.empty?
+              raise Error, "#{errors.join ' '}"
+            end
+          end
         end # Doc
 
         def post(path, data = {})

data/lib/bamboo-client/remote.rb

@@ -24,7 +24,7 @@ module Bamboo
       end
 
       def logout
-        post :logout, :token => token
+        post :logout, :auth => token
         @token = nil
       end
 

data/lib/bamboo-client/version.rb

@@ -1,5 +1,5 @@
 module Bamboo
   module Client
-    VERSION = "0.0.2"
+    VERSION = "0.0.3"
   end
 end

data/spec/bamboo-client/http/xml_spec.rb

@@ -19,7 +19,11 @@ module Bamboo
       end
 
       describe Xml::Doc do
-        let(:wrapped) { mock("nokogiri document") }
+        let(:wrapped) { 
+          m = mock("nokogiri document") 
+          m.stub!(:css).with("errors error").and_return []
+          m
+        }
         let(:doc) { Xml::Doc.new(wrapped)}
 
         it "returns the text of the given CSS selector" do

data/spec/bamboo-client/remote_spec.rb

@@ -6,7 +6,12 @@ module Bamboo
       let(:url)  { "http://bamboo.example.com" }
       let(:http) { mock(Http::Xml) }
       let(:client) { Remote.new(http) }
-      let(:document) { mock(Http::Xml::Doc) }
+      let(:document) {
+        m = mock(Http::Xml::Doc)
+        m.stub(:css).with("errors error").and_return []
+
+        m
+      }
 
       context "authorization" do
         it "logs in" do
@@ -29,7 +34,7 @@ module Bamboo
         it "logs out" do
           http.should_receive(:post).with(
             "/api/rest/logout.action",
-            :token => "foo"
+            :auth => "foo"
           )
 
           client.instance_variable_set "@token", "foo"

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 0
-  - 2
-  version: 0.0.2
+  - 3
+  version: 0.0.3
 platform: ruby
 authors: 
 - Jari Bakken