Merge pull request #24 from NohSeho/improve/return-error-code-when-http-not-200

Improve/return error code when http not 200

Author: perhapsspy

README.rst

@@ -103,7 +103,9 @@ Python 사용자를 위한 아임포트 REST API 연동 모듈입니다.
     except Iamport.ResponseError as e:
         print e.code
         print e.message  # 에러난 이유를 알 수 있음
-
+    except Iamport.HttpError as http_error:
+        print http_error.code
+        print http_error.reason # HTTP not 200 에러난 이유를 알 수 있음
 
 비인증 결제
 -------------
@@ -129,7 +131,9 @@ Python 사용자를 위한 아임포트 REST API 연동 모듈입니다.
     except Iamport.ResponseError as e:
         # 응답 에러 처리
         pass
-
+    except Iamport.HttpError as http_error:
+        # HTTP not 200 응답 에러 처리
+        pass
 
 저장된 빌링키로 재결제합니다.
 
@@ -149,7 +153,9 @@ Python 사용자를 위한 아임포트 REST API 연동 모듈입니다.
     except Iamport.ResponseError as e:
         # 응답 에러 처리
         pass
-
+    except Iamport.HttpError as http_error:
+        # HTTP not 200 응답 에러 처리
+        pass
 
 정기 예약 결제
 ----------------
@@ -190,6 +196,9 @@ Python 사용자를 위한 아임포트 REST API 연동 모듈입니다.
     except Iamport.ResponseError as e:
         # 응답 에러 처리
         pass
+    except Iamport.HttpError as http_error:
+        # HTTP not 200 응답 에러 처리
+        pass
 
 정기 결제 예약을 취소합니다.
 
@@ -208,7 +217,9 @@ Python 사용자를 위한 아임포트 REST API 연동 모듈입니다.
     except Iamport.ResponseError as e:
         # 응답 에러 처리
         pass
-
+    except Iamport.HttpError as http_error:
+        # HTTP not 200 응답 에러 처리
+        pass
 
 결제 사전 검증
 ----------------
@@ -225,7 +236,9 @@ Python 사용자를 위한 아임포트 REST API 연동 모듈입니다.
     except Iamport.ResponseError as e:
         # 응답 에러 처리
         pass
-
+    except Iamport.HttpError as http_error:
+        # HTTP not 200 응답 에러 처리
+        pass
 
 등록된 사전정보를 확인합니다
 
@@ -239,6 +252,9 @@ Python 사용자를 위한 아임포트 REST API 연동 모듈입니다.
     except Iamport.ResponseError as e:
         # 응답 에러 처리
         pass
+    except Iamport.HttpError as http_error:
+        # HTTP not 200 응답 에러 처리
+        pass
 
 
 개발환경 및 테스트 설정

iamport/client.py

@@ -23,10 +23,15 @@ def __init__(self, code=None, message=None):
             self.code = code
             self.message = message
 
+    class HttpError(Exception):
+        def __init__(self, code=None, reason=None):
+            self.code = code
+            self.reason = reason
+
     @staticmethod
     def get_response(response):
         if response.status_code != requests.codes.ok:
-            return {}
+            raise Iamport.HttpError(response.status_code, response.reason)
         result = response.json()
         if result['code'] != 0:
             raise Iamport.ResponseError(result.get('code'), result.get('message'))

tests/test_find.py

@@ -5,7 +5,7 @@
 def test_find(iamport):
     with pytest.raises(KeyError):
         iamport.find()
-    result = iamport.find(imp_uid='test')
-    assert dict == type(result)
-    result = iamport.find(merchant_uid='test')
-    assert dict == type(result)
+    with pytest.raises(iamport.HttpError):
+        iamport.find(imp_uid='test')
+    with pytest.raises(iamport.HttpError):
+        iamport.find(merchant_uid='âàáaā')