Skip to content

Commit e5da7cd

Browse files
committed
Test that replication method was not called when there are no principals in group to migrate
1 parent 1bd5879 commit e5da7cd

File tree

1 file changed

+13
-2
lines changed

1 file changed

+13
-2
lines changed

tests/migration_tool/tests_migrate.py

+13-2
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,13 @@
2323
from uuid import uuid4
2424

2525
from django.test import TestCase, override_settings
26+
2627
from django.utils import timezone
2728

2829
from api.models import CrossAccountRequest, Tenant
2930

3031
from management.models import *
31-
from management.role.definer import seed_roles
32+
3233
from management.tenant_service.tenant_service import BootstrappedTenant
3334
from management.tenant_service.v2 import V2TenantBootstrapService
3435
from migration_tool.in_memory_tuples import (
@@ -43,7 +44,9 @@
4344
subject,
4445
subject_type,
4546
)
46-
from migration_tool.migrate import migrate_data
47+
48+
from migration_tool.migrate import migrate_data, migrate_groups_for_tenant
49+
4750
from management.group.definer import seed_group, clone_default_group_in_public_schema
4851
from tests.management.role.test_dual_write import RbacFixture
4952

@@ -161,6 +164,14 @@ def setUp(self):
161164
)
162165
self.cross_account_request.roles.add(self.system_role_2)
163166

167+
@override_settings(REPLICATION_TO_RELATION_ENABLED=True, PRINCIPAL_USER_DOMAIN="redhat", READ_ONLY_API_MODE=True)
168+
@patch("migration_tool.migrate.RelationApiDualWriteGroupHandler.replicate")
169+
def test_migration_of_data_no_replication_event_to_migrate_groups(self, replicate_method):
170+
"""Test that we get the correct access for a principal."""
171+
kwargs = {"exclude_apps": ["app1"], "orgs": ["7654321"]}
172+
migrate_data(**kwargs)
173+
replicate_method.assert_not_called()
174+
164175
@override_settings(REPLICATION_TO_RELATION_ENABLED=True, PRINCIPAL_USER_DOMAIN="redhat", READ_ONLY_API_MODE=True)
165176
@patch("management.relation_replicator.logging_replicator.logger")
166177
def test_migration_of_data(self, logger_mock):

0 commit comments

Comments
 (0)