Skip to content

Commit 678fa8d

Browse files
committed
double con close
get rid of failing test on app.dev without push
1 parent e1675cc commit 678fa8d

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

mergin/test/test_client.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1480,7 +1480,6 @@ def test_push_gpkg_schema_change(mc):
14801480
# open a connection and keep it open (qgis does this with a pool of connections too)
14811481
acon2 = AnotherSqliteConn(test_gpkg)
14821482
acon2.run("select count(*) from simple;")
1483-
acon2.close()
14841483

14851484
# add a new table to ensure that geodiff will fail due to unsupported change
14861485
# (this simulates an independent reader/writer like GDAL)
@@ -3071,6 +3070,9 @@ def test_validate_auth(mc: MerginClient):
30713070
def test_uploaded_chunks_cache(mc):
30723071
"""Create a new project, download it, add a file and then do sync - it should not fail"""
30733072

3073+
# This test does not make sense on servers without v2 push
3074+
if not mc.server_features().get("v2_push_enabled"):
3075+
return
30743076
test_project = "test_uploaded_chunks_cache"
30753077
project = API_USER + "/" + test_project
30763078
project_dir = os.path.join(TMP_DIR, test_project) # primary project dir for updates

0 commit comments

Comments
 (0)