Merge pull request #35 from asonas/feature/raise_error

raise error when status 4xx and 5xx
asonas · Jan 11, 2018 · 260726c · 260726c
2 parents 5c808b3 + ccad148
commit 260726c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/lib/chatwork/base_client.rb b/lib/chatwork/base_client.rb
@@ -29,7 +29,7 @@ def handle_response(response)
           raise ChatWork::APIConnectionError.new("Response JSON is broken. #{e.message}: #{response.body}", e)
         end
       else
-        ChatWork::ChatWorkError.from_response(response.status, response.body, response.headers)
+        raise ChatWork::ChatWorkError.from_response(response.status, response.body, response.headers)
       end
     end
 

diff --git a/spec/lib/chatwork/me_spec.rb b/spec/lib/chatwork/me_spec.rb
@@ -7,5 +7,13 @@
     end
 
     it_behaves_like :a_chatwork_api, :get, "/me"
+
+    context "when unauthorized" do
+      before do
+        stub_chatwork_request(:get, "/me", "/me", 401)
+      end
+
+      it { expect { subject }.to raise_error(ChatWork::APIError, "Invalid API token") }
+    end
   end
 end