fix tests

Author: RulaKhaled

graphql_api/tests/test_repository.py

@@ -438,6 +438,7 @@ def test_repository_not_activated(self, try_auto_activate, is_activated):
             private=True,
             coverage_enabled=True,
             bundle_analysis_enabled=True,
+            test_analytics_enabled=True,
         )
 
         is_activated.return_value = False
@@ -461,6 +462,7 @@ def test_repository_not_activated_self_hosted(self, try_auto_activate):
             private=True,
             coverage_enabled=True,
             bundle_analysis_enabled=True,
+            test_analytics_enabled=True,
         )
 
         data = self.gql_request(

graphql_api/types/owner/owner.py

@@ -162,19 +162,23 @@ async def resolve_repository(
         return NotFoundError()
 
     current_owner = info.context["request"].current_owner
-    has_products_enabled = (
-        repository.bundle_analysis_enabled
-        or repository.coverage_enabled
-        or repository.test_analytics_enabled
-    )
+    product_flags = [
+        repository.bundle_analysis_enabled,
+        repository.coverage_enabled,
+        repository.test_analytics_enabled,
+    ]
+    has_products_enabled = any(product_flags)
+    all_products_enabled = all(product_flags)
 
     if repository.private and has_products_enabled:
         await sync_to_async(activation.try_auto_activate)(owner, current_owner)
-        is_owner_activated = await sync_to_async(activation.is_activated)(
-            owner, current_owner
-        )
-        if not is_owner_activated:
-            return OwnerNotActivatedError()
+
+    is_owner_activated = await sync_to_async(activation.is_activated)(
+        owner, current_owner
+    )
+
+    if all_products_enabled and not is_owner_activated:
+        return OwnerNotActivatedError()
 
     info.context["profiling_summary"] = ProfilingSummary(repository)
     return repository

services/activation.py

@@ -71,7 +71,7 @@ def _get_activator(org: Owner, owner: Owner) -> BaseActivator:
 
 def try_auto_activate(org: Owner, owner: Owner) -> bool:
     """
-    Returns true iff the user was able to be activated, false otherwise.
+    Returns true if the user was able to be activated, false otherwise.
     """
     activator = _get_activator(org, owner)