Handling non-existing validSince case on UserRecord (#220)

* Handling non-existing validSince case on UserRecord

* Merged with master

Author: hiranya911

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 - [fixed] Fixing error handling in FCM. The SDK now checks the key
   type.googleapis.com/google.firebase.fcm.v1.FcmError to set error code.
+- [fixed] Ensuring that `UserRecord.tokens_valid_after_time` always
+  returns an integer, and never returns `None`.
 
 # v2.13.0
 

firebase_admin/_user_mgt.py

@@ -215,7 +215,7 @@ def tokens_valid_after_timestamp(self):
         valid_since = self._data.get('validSince')
         if valid_since is not None:
             return 1000 * int(valid_since)
-        return None
+        return 0
 
     @property
     def user_metadata(self):

tests/test_user_mgt.py

@@ -151,6 +151,14 @@ def test_invalid_provider(self, data):
         with pytest.raises(ValueError):
             _user_mgt.ProviderUserInfo(data)
 
+    def test_tokens_valid_after_time(self):
+        user = auth.UserRecord({'localId' : 'user', 'validSince' : 100})
+        assert user.tokens_valid_after_timestamp == 100000
+
+    def test_no_tokens_valid_after_time(self):
+        user = auth.UserRecord({'localId' : 'user'})
+        assert user.tokens_valid_after_timestamp is 0
+
 
 class TestGetUser(object):