8000 add extra check to unit test · firebase/firebase-admin-python@8dd4af3 · GitHub
[go: up one dir, main page]

Skip to content

Commit 8dd4af3

Browse files
committed
add extra check to unit test
1 parent 7245194 commit 8dd4af3

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

tests/test_db.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -180,6 +180,8 @@ class TestReference:
180180
500: exceptions.InternalError,
181181
}
182182

183+
duplicate_timestamp = time.time()
184+
183185
@classmethod
184186
def setup_class(cls):
185187
firebase_admin.initialize_app(testutils.MockCredential(), {'databaseURL' : cls.test_url})
@@ -413,14 +415,18 @@ def test_push_default(self):
413415
assert len(recorder) == 0
414416

415417
@pytest.mark.parametrize('data', valid_values)
416-
@mock.patch('time.time', mock.MagicMock(return_value=1700497750.2549))
418+
@mock.patch('time.time', mock.MagicMock(return_value=duplicate_timestamp))
417419
def test_push_duplicate_timestamp(self, data):
418420
ref = db.reference('/test')
419421
recorder = self.instrument(ref, json.dumps({}))
420422
child = []
421423
child.append(ref.push(data))
422424
child.append(ref.push(data))
423-
assert child[1].key > child[0].key
425+
key1 = child[0].key
426+
key2 = child[1].key
427+
# First 8 digits are the encoded timestamp
428+
assert key1[:8] == key2[:8]
429+
assert key2 > key1
424430
assert len(recorder) == 2
425431
for index, record in enumerate(recorder):
426432
assert record.method == 'PUT'

0 commit comments

Comments
 (0)
0