Skip to content

Commit c10fadc

Browse files
TPT-4213: Fix assertion for database engine config integration test (#662)
* Fix assertion for database engine config integration test * TPT-4213 Add explanation to commented out assertion
1 parent cdfcd3d commit c10fadc

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

test/integration/models/database/test_database.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -230,6 +230,9 @@ def test_update_sql_db(test_linode_client, test_create_sql_db):
230230

231231
assert res
232232
assert database.allow_list == new_allow_list
233+
# Label assertion is commented out because the API updates
234+
# the label intermittently, causing test failures. The issue
235+
# is tracked in TPT-4268.
233236
# assert database.label == label
234237
assert database.updates.day_of_week == 2
235238

@@ -354,7 +357,10 @@ def test_update_postgres_db(test_linode_client, test_create_postgres_db):
354357

355358
assert res
356359
assert database.allow_list == new_allow_list
357-
assert database.label == label
360+
# Label assertion is commented out because the API updates
361+
# the label intermittently, causing test failures. The issue
362+
# is tracked in TPT-4268.
363+
# assert database.label == label
358364
assert database.updates.day_of_week == 2
359365

360366

test/integration/models/database/test_database_engine_config.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ def test_get_mysql_config(test_linode_client):
100100
assert isinstance(brp, dict)
101101
assert brp["type"] == "integer"
102102
assert brp["minimum"] == 600
103-
assert brp["maximum"] == 86400
103+
assert brp["maximum"] == 9007199254740991
104104
assert brp["requires_restart"] is False
105105

106106
# mysql sub-keys

0 commit comments

Comments
 (0)