Merge pull request #72 from EasyPost/fix_user_tests

Updating user tests and examples

Author: Max Waldt

.gitignore

@@ -1,3 +1,4 @@
+.cache
 .DS_Store
 .AppleDouble
 .LSOverride
@@ -6,6 +7,9 @@ Icon
 .Spotlight-V100
 .Trashes
 
+# virtual environment
+venv
+
 /build
 /dist
 /MANIFEST
@@ -20,4 +24,4 @@ Icon
 test
 virtualenv
 .virtualenv
-.idea
+.idea

CHANGELOG.md

@@ -1,5 +1,9 @@
 ### NEXT (unreleased)
 
+### 3.6.4 2018-04-09
+
+* Update user tests
+
 ### 3.6.3 2018-02-05
 
 * Fix issue with responses that contain an `api_key` field (gh-67)

VERSION

@@ -1 +1 @@
-3.6.3
+3.6.4

examples/user.py

@@ -5,9 +5,7 @@
 
 child_user = easypost.User.create(
     name='Python All-Things-Testing',
-    password='password1',
-    password_confirmation='password1',
-    phone_number='7778675309'
+    phone_number='555-555-5555'
 )
 id = child_user.id
 

tests/test_user.py

@@ -1,15 +1,27 @@
 # Unit tests related to 'Users' (https://www.easypost.com/docs/api#users).
 
 import easypost
+import pytest
 
 
 def test_child_user_create(prod_api_key):
     # Create an address and then verify some fields to test whether it was created just fine.
     child_user = easypost.User.create(
-        name='Python All-Things-Testing',
-        password='password1',
-        password_confirmation='password1',
+        name='Python All-Things-Testing'
     )
+
+    with pytest.raises(easypost.Error) as caught_exception:
+        easypost.User.create(
+            name='Python All-Things-Testing',
+            password='super-secret-password',
+            password_confirmation='super-secret-password'
+        )
+
+    exception = caught_exception.value.json_body
+    error_msg = 'Child users must be created via the API and must not contain an email or password.'
+    assert exception['error']['message'] == error_msg
+    assert exception['error']['code'] == 'USER.INVALID'
+
     child_id = child_user.id
     assert child_id is not None