Add tests for https://gitlab.com/caosdb/caosdb-server/-/issues/145

Summary

Tests for caosdb-server!62 (merged) (https://gitlab.com/caosdb/caosdb-server/-/issues/145) and also https://gitlab.com/caosdb/caosdb-server/-/issues/147

Please merge together with caosdb-server!62 (merged)

Focus

The MR caosdb-server!62 (merged) proposed to remove the "casting to integer". These tests show that that would be a bug and also test the fix for https://gitlab.com/caosdb/caosdb-server/-/issues/145

Check List for the Author

Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines

• All automated tests pass
• Reference related issues
• Up-to-date CHANGELOG.md (or not necessary)
• Annotations in code (Gitlab comments)
  • Intent of new code
  • Problems with old code
  • Why this implementation?

Check List for the Reviewer

• I understand the intent of this MR
• All automated tests pass
• Up-to-date CHANGELOG.md (or not necessary)
• The test environment setup works and the intended behavior is reproducible in the test environment
• In-code documentation and comments are up-to-date.
• Check: Are there specifications? Are they satisfied?

For further good practices have a look at our review guidelines.

tests/test_issues_server.py

@@ -848,3 +848,44 @@ def test_141():
     # switch back to admin user
     db.configure_connection()
     assert db.execute_query("FIND TestRT", unique=True).description == "Desc2"
+
+
+def test_145():
+    """Searching for large numbers results in wrong results if integer values
+    are used.
+
+    https://gitlab.com/caosdb/caosdb-server/-/issues/145
+    """
+    db.Property("TestProp", datatype=db.TEXT).insert()
+    db.Property("TestPropInt", datatype=db.INTEGER).add_parent("TestProp").insert()
+    db.Property("TestPropDouble", datatype=db.DOUBLE).add_parent("TestProp").insert()
+
+    db.RecordType("TestRT").insert()
+    rec1 = db.Record("TestRec1").add_parent("TestRT").add_property(
+        "TestPropInt", 1_000_000_000).insert()
+    assert rec1.get_property("TestPropInt").value == 1_000_000_000
+    assert isinstance(rec1.get_property("TestPropInt").value, int)
+    rec2 = db.Record("TestRec2").add_parent("TestRT").add_property(
+        "TestPropDouble", 20_000_000_000).insert()
+    assert rec2.get_property("TestPropDouble").value == 20_000_000_000
+    assert isinstance(rec2.get_property("TestPropDouble").value, float)
+
+    assert db.execute_query("FIND TestRT WITH TestProp < 1000000000.2",
+                            unique=True).id == rec1.id  # this is the integer property
+    assert db.execute_query("FIND TestRT WITH TestProp > 1000000000.2", unique=True).id == rec2.id
+
+    assert db.execute_query("FIND TestRT WITH TestProp = 1000000000", unique=True).id == rec1.id
+    assert db.execute_query("FIND TestRT WITH TestProp = 1000000000.0", unique=True).id == rec1.id
+
+    assert db.execute_query("FIND TestRT WITH TestProp > 1000000000", unique=True).id == rec2.id
+    assert db.execute_query("FIND TestRT WITH TestProp > 1000000000.0", unique=True).id == rec2.id
+
+    assert db.execute_query("FIND TestRT WITH TestProp = 20000000000", unique=True).id == rec2.id
+    assert db.execute_query("FIND TestRT WITH TestProp = 20000000000.0", unique=True).id == rec2.id
+
+    assert db.execute_query("FIND TestRT WITH TestProp < 20000000000", unique=True).id == rec1.id
+    assert db.execute_query("FIND TestRT WITH TestProp < 20000000000.0", unique=True).id == rec1.id
+
+    assert db.execute_query(
+        "FIND TestRT WITH TestPropInt < 10000000000000000000000000000000000000000000000000000000000",
+        unique=True).id == rec1.id