From de661e7987192bc9deb131e49686af610171ac82 Mon Sep 17 00:00:00 2001 From: Anthony Mahanna Date: Fri, 28 Jul 2023 15:16:18 -0400 Subject: [PATCH 1/3] initial commit --- tests/test_analyzer.py | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/tests/test_analyzer.py b/tests/test_analyzer.py index af4b91cb..219ea653 100644 --- a/tests/test_analyzer.py +++ b/tests/test_analyzer.py @@ -1,3 +1,5 @@ +from packaging import version + from arango.exceptions import ( AnalyzerCreateError, AnalyzerDeleteError, @@ -7,7 +9,7 @@ from tests.helpers import assert_raises, generate_analyzer_name -def test_analyzer_management(db, bad_db, cluster): +def test_analyzer_management(db, bad_db, cluster, enterprise, db_version): analyzer_name = generate_analyzer_name() full_analyzer_name = db.name + "::" + analyzer_name bad_analyzer_name = generate_analyzer_name() @@ -56,3 +58,11 @@ def test_analyzer_management(db, bad_db, cluster): # Test delete missing analyzer with ignore_missing set to True assert db.delete_analyzer(analyzer_name, ignore_missing=True) is False + + # Test create geo_s2 analyzer (EE only) + if enterprise and db_version >= version.parse("3.10.5"): + analyzer_name = generate_analyzer_name() + result = db.create_analyzer(analyzer_name, "geo_s2", {}) + assert result["type"] == "geo_s2" + assert result["properties"]["format"] == "latLngDouble" + assert db.delete_analyzer(analyzer_name) From 6e2b00a461dc0072517c4e1c9d0a934da8763d85 Mon Sep 17 00:00:00 2001 From: Anthony Mahanna Date: Mon, 31 Jul 2023 09:10:42 -0400 Subject: [PATCH 2/3] fix: test --- tests/test_analyzer.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tests/test_analyzer.py b/tests/test_analyzer.py index 219ea653..fa0c29a7 100644 --- a/tests/test_analyzer.py +++ b/tests/test_analyzer.py @@ -63,6 +63,10 @@ def test_analyzer_management(db, bad_db, cluster, enterprise, db_version): if enterprise and db_version >= version.parse("3.10.5"): analyzer_name = generate_analyzer_name() result = db.create_analyzer(analyzer_name, "geo_s2", {}) - assert result["type"] == "geo_s2" - assert result["properties"]["format"] == "latLngDouble" + assert result["features"] == [] + assert result["properties"] == { + "options": {"maxCells": 20, "minLevel": 4, "maxLevel": 23}, + "type": "shape", + "format": "latLngDouble", + } assert db.delete_analyzer(analyzer_name) From 9e00d2050f585902c62c0eb5170605607677db85 Mon Sep 17 00:00:00 2001 From: Anthony Mahanna Date: Mon, 31 Jul 2023 09:12:11 -0400 Subject: [PATCH 3/3] add type assert --- tests/test_analyzer.py | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/test_analyzer.py b/tests/test_analyzer.py index fa0c29a7..63627251 100644 --- a/tests/test_analyzer.py +++ b/tests/test_analyzer.py @@ -63,6 +63,7 @@ def test_analyzer_management(db, bad_db, cluster, enterprise, db_version): if enterprise and db_version >= version.parse("3.10.5"): analyzer_name = generate_analyzer_name() result = db.create_analyzer(analyzer_name, "geo_s2", {}) + assert result["type"] == "geo_s2" assert result["features"] == [] assert result["properties"] == { "options": {"maxCells": 20, "minLevel": 4, "maxLevel": 23},