Added tests

gratipay/models/participant.py

@@ -596,7 +596,6 @@ def get_verification_link(self):
         link = "%s://%s/%s/verify-email.html?hash=%s" % (gratipay.canonical_scheme, gratipay.canonical_host, username, hash_string)
         return link
 
-
     def update_goal(self, goal):
         typecheck(goal, (Decimal, None))
         with self.db.get_cursor() as c:

tests/py/test_participant.py

@@ -209,22 +209,27 @@ def test_john_is_plural(self):
         assert actual == expected
 
     def test_can_change_email(self):
-        self.alice.update_email('[email protected]')
+        self.alice.change_email('[email protected]')
         expected = '[email protected]'
         actual = self.alice.email.address
         assert actual == expected
 
-    def test_cannot_confirm_email_in_one_step(self):
-        self.alice.update_email('[email protected]', True)
-        actual = self.alice.email.confirmed
-        assert actual == False
-
-    def test_can_confirm_email_in_second_step(self):
+    def test_can_verify_email(self):
         self.alice.update_email('[email protected]')
-        self.alice.update_email('[email protected]', True)
-        actual = self.alice.email.confirmed
+        hash_string = Participant.from_username('alice').email.hash
+        self.alice.verify_email(hash_string)
+        actual = Participant.from_username('alice').email.confirmed
         assert actual == True
 
+    def test_cannot_verify_email_with_wrong_hash(self):
+        self.alice.update_email('[email protected]')
+        hash_string = "some wrong hash"
+        self.alice.verify_email(hash_string)
+        actual = Participant.from_username('alice').email.confirmed
+        assert actual == False
+
+    # TODO - Add a test for expired hashes. (We don't have control over the ctime of emails)
+
     def test_cant_take_over_claimed_participant_without_confirmation(self):
         with self.assertRaises(NeedConfirmation):
             self.alice.take_over(('twitter', str(self.bob.id)))