From 81e4a95c6267b1066f68822692051e946907a155 Mon Sep 17 00:00:00 2001 From: Pedro Rodrigues Date: Fri, 25 Jul 2025 11:20:28 +0100 Subject: [PATCH 1/2] fix starter workspace creation tests --- singlestoredb/tests/test_management.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/singlestoredb/tests/test_management.py b/singlestoredb/tests/test_management.py index 97b0c5ff5..10124ebb5 100755 --- a/singlestoredb/tests/test_management.py +++ b/singlestoredb/tests/test_management.py @@ -390,7 +390,9 @@ def password(self): def setUpClass(cls): cls.manager = s2.manage_workspaces() - us_regions = [x for x in cls.manager.regions if 'US' in x.name] + shared_tier_regions = [ + x for x in cls.manager.shared_tier_regions if 'US' in x.name + ] cls.password = secrets.token_urlsafe(20) + '-x&$' name = clean_name(secrets.token_urlsafe(20)[:20]) @@ -400,7 +402,7 @@ def setUpClass(cls): database_name=f'starter_db_{name}', workspace_group={ 'name': f'starter-wg-test-{name}', - 'cell_id': random.choice(us_regions).id, + 'cell_id': random.choice(shared_tier_regions).id, }, ) From fe86e8ea272a7a966610def54353dda49d4f4f9f Mon Sep 17 00:00:00 2001 From: Pedro Rodrigues Date: Fri, 25 Jul 2025 11:50:36 +0100 Subject: [PATCH 2/2] remove no manager test for regions --- singlestoredb/tests/test_management.py | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/singlestoredb/tests/test_management.py b/singlestoredb/tests/test_management.py index 10124ebb5..cc31e7236 100755 --- a/singlestoredb/tests/test_management.py +++ b/singlestoredb/tests/test_management.py @@ -14,7 +14,6 @@ from singlestoredb.management.job import Status from singlestoredb.management.job import TargetType from singlestoredb.management.region import Region -from singlestoredb.management.region import RegionManager from singlestoredb.management.utils import NamedList @@ -1566,17 +1565,3 @@ def test_str_repr(self): # Test __repr__ assert repr(region) == str(region) - - def test_no_manager(self): - """Test behavior when manager is not available.""" - regions = self.manager.list_regions() - if not regions: - self.skipTest('No regions available for testing') - - region = regions[0] - region._manager = None - - # Verify from_dict class method - with self.assertRaises(s2.ManagementError) as cm: - RegionManager.list_shared_tier_regions(None) - assert 'No workspace manager' in str(cm.exception)