roust 1.4.0 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b2f1667d37f6c04f786ddb9bd4c437290151f549
-  data.tar.gz: e65c73bab11bd66ec7c904445de4aa7573440f69
+  metadata.gz: d1a2f356ca5640d7d394f07ced73cc03abcd6e34
+  data.tar.gz: 41196d852637d8c592068c6e1faf993183349a5a
 SHA512:
-  metadata.gz: 0aa451f3c531b9981ecd88f47d738122ed38684a3ec41ac27eb743b1bdb528c9377b42feb9e352f65039bc24a8458faa94d3717df009ebf0681da230ff77efc8
-  data.tar.gz: fdcf546e35a39bd594c9f2d5fa42d1232675a9019f6f019d77caf52e04fef5254bf4a15b452aad381c38feaa8a575941586256e30dc15e98674da7a65eff9d55
+  metadata.gz: 197a802ad243cafcff1965016d544b1722dd83ef9d633410ede414422a0ffba1de3e63f4d91f548ac546501f4c898bda05f459f50c255495c3a11252ee326089
+  data.tar.gz: 5a68904147845e22f3ff138eb0bf1d0a6d5f64da303d7cc877706ac57a132d7bea1ef7ef5122ea4bbe128a5ed44b8f77adc990a6283334a9461b04cda8ad2725

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    roust (1.4.0)
+    roust (1.4.1)
       activesupport (>= 4.1.0)
       httparty (>= 0.13.1)
       mail (>= 2.5.4)

data/lib/roust/version.rb

@@ -1,3 +1,3 @@
 class Roust
-  VERSION = '1.4.0'
+  VERSION = '1.4.1'
 end

data/lib/roust.rb

@@ -17,6 +17,10 @@ class Roust
     username = credentials[:username]
     password = credentials[:password]
 
+    if server =~ /REST\/1\.0/
+      raise ArgumentError, "The supplied :server has REST in the URL. You only need to specify the base, e.g. http://rt.example.org/"
+    end
+
     self.class.base_uri(server)
 
     response = self.class.post(

data/spec/roust/authentication_spec.rb

@@ -13,21 +13,27 @@ describe Roust do
     it 'errors when credentials are incorrect' do
       mocks_path = Pathname.new(__FILE__).parent.parent.join('mocks')
 
-      stub_request(:post, 'http://rt.example.org/index.html').
-        with(:body => {
-              'user'=>'admin',
-              'pass'=>'incorrect',
-             }).
-        to_return(:status => 200, :body => '', :headers => {})
-
-      stub_request(:get, 'http://rt.example.org/REST/1.0/ticket/1/show').
-        to_return(:status  => 200,
-                  :body    => mocks_path.join('ticket-1-show-unauthenticated.txt').read,
-                  :headers => {})
+      stub_request(:post, 'http://rt.example.org/index.html')
+        .with(:body => {
+                'user'=>'admin',
+                'pass'=>'incorrect',
+              })
+        .to_return(:status => 200, :body => '', :headers => {})
+
+      stub_request(:get, 'http://rt.example.org/REST/1.0/ticket/1/show')
+        .to_return(:status  => 200,
+                   :body    => mocks_path.join('ticket-1-show-unauthenticated.txt').read,
+                   :headers => {})
 
       credentials.merge!({:username => 'admin', :password => 'incorrect'})
 
       expect { Roust.new(credentials) }.to raise_error(Unauthenticated)
     end
+
+    it 'errors when API root is supplied in server url' do
+      credentials[:server] = 'http://rt.example.org/REST/1.0'
+
+      expect { Roust.new(credentials) }.to raise_error(ArgumentError)
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: roust
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 1.4.1
 platform: ruby
 authors:
 - Lindsay Holmwood