From 39afeb7ac18849e3bf4c22f105d9ce8bbf0fac68 Mon Sep 17 00:00:00 2001 From: d-w-moore Date: Thu, 30 Apr 2026 15:27:32 -0400 Subject: [PATCH] [#600] remove case-sensitive tests entirely. Not all databases are predictable as lexicographical order mixes up case in arbitrarily varying ways. 'A' can be less than, greater than, or equal to 'a' depending on the DB and/or its configuration. --- irods/test/query_test.py | 24 ------------------------ 1 file changed, 24 deletions(-) diff --git a/irods/test/query_test.py b/irods/test/query_test.py index b04c29d87..a4f4ecdb0 100644 --- a/irods/test/query_test.py +++ b/irods/test/query_test.py @@ -213,30 +213,6 @@ def test_files_query_case_sensitive(self): ) self.assertEqual(len(result12), 0) - # BETWEEN tests - - # TODO(#600): Uncomment these lines and/or make a new test when database flavor can be detected. - # The resultset for BETWEEN queries can differ from database to database. - - # result13 = self.sess.query(DataObject.name).filter( - # Collection.name == self.coll_path).filter( - # Between(DataObject.name, [self.case_sensitive_obj_name1, - # self.case_sensitive_obj_name1 + "_"])).all() - # self.assertTrue(result13.has_value(self.case_sensitive_obj_name1)) - # self.assertEqual(len(result13), 1) - - # result14 = self.sess.query(DataObject.name).filter( - # Collection.name == self.coll_path).filter( - # Between(DataObject.name, [str.lower(self.case_sensitive_obj_name1), - # str.lower(self.case_sensitive_obj_name1) + "_"])).all() - # self.assertEqual(len(result14), 0) - - # result15 = self.sess.query(DataObject.name).filter( - # Collection.name == self.coll_path).filter( - # Between(DataObject.name, [str.upper(self.case_sensitive_obj_name1), - # str.upper(self.case_sensitive_obj_name1) + "_"])).all() - # self.assertEqual(len(result15), 0) - def test_files_query_case_insensitive(self): # This tests that GenQueries are case-insensitive when the case_sensitive # option is set to false.