Skip to content

Commit

Permalink
Fix nondeterministic unit tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jnm committed Apr 5, 2022
1 parent 4211d16 commit 6ed4a85
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
4 changes: 2 additions & 2 deletions kpi/fixtures/test_data.json
Original file line number Diff line number Diff line change
Expand Up @@ -112,8 +112,8 @@
"kuid": "def"
}
]},
"date_created": "2015-02-12T19:52:14.406Z",
"date_modified": "2015-02-12T19:52:14.406Z",
"date_created": "2022-04-05T21:00:22.402Z",
"date_modified": "2022-04-05T21:00:33.946Z",
"owner": 2,
"uid": "aPEANgNgTH3xzhJGcctURu"
}
Expand Down
2 changes: 1 addition & 1 deletion kpi/tests/api/v1/test_api_imports.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ class AssetImportTaskTest(BaseTestCase):
def setUp(self):
self.client.login(username='someuser', password='someuser')
self.user = User.objects.get(username='someuser')
self.asset = Asset.objects.first()
self.asset = Asset.objects.get(pk=1)

def _assert_assets_contents_equal(self, a1, a2, sheet='survey'):
def _prep_row_for_comparison(row):
Expand Down
4 changes: 2 additions & 2 deletions kpi/tests/api/v2/test_api_assets.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ def test_asset_list_matches_detail(self):

def test_assets_hash(self):
another_user = User.objects.get(username="anotheruser")
user_asset = Asset.objects.first()
user_asset = Asset.objects.get(pk=1)
user_asset.save()
user_asset.assign_perm(another_user, "view_asset")

Expand Down Expand Up @@ -257,7 +257,7 @@ class AssetVersionApiTests(BaseTestCase):

def setUp(self):
self.client.login(username='someuser', password='someuser')
self.asset = Asset.objects.first()
self.asset = Asset.objects.get(pk=1)
self.asset.save()
self.version = self.asset.asset_versions.first()
self.version_list_url = reverse(self._get_endpoint('asset-version-list'),
Expand Down
2 changes: 1 addition & 1 deletion kpi/tests/api/v2/test_api_imports.py
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ class AssetImportTaskTest(BaseTestCase):
def setUp(self):
self.client.login(username='someuser', password='someuser')
self.user = User.objects.get(username='someuser')
self.asset = Asset.objects.first()
self.asset = Asset.objects.get(pk=1)

def _assert_assets_contents_equal(self, a1, a2, sheet='survey'):
def _prep_row_for_comparison(row):
Expand Down

0 comments on commit 6ed4a85

Please sign in to comment.