diff --git a/ansible_wisdom/ai/api/telemetry/tests/test_api_telemetry_settings_views.py b/ansible_wisdom/ai/api/telemetry/tests/test_api_telemetry_settings_views.py index 9994ed837..5671807be 100644 --- a/ansible_wisdom/ai/api/telemetry/tests/test_api_telemetry_settings_views.py +++ b/ansible_wisdom/ai/api/telemetry/tests/test_api_telemetry_settings_views.py @@ -42,15 +42,6 @@ def test_permission_classes(self, *args): for permission in required_permissions: self.assertTrue(permission in view.permission_classes) - @override_settings(LAUNCHDARKLY_SDK_KEY='dummy_key') - @patch.object(feature_flags, 'LDClient') - def test_get_settings_when_feature_disabled(self, LDClient, *args): - LDClient.return_value.variation.return_value = False - self.user.organization = Organization.objects.get_or_create(id=123)[0] - self.client.force_authenticate(user=self.user) - r = self.client.get(reverse('telemetry_settings')) - self.assertEqual(r.status_code, HTTPStatus.SERVICE_UNAVAILABLE) - @override_settings(SEGMENT_WRITE_KEY='DUMMY_KEY_VALUE') def test_get_settings_without_org_id(self, *args): self.client.force_authenticate(user=self.user) @@ -95,15 +86,6 @@ def test_set_settings_authentication_error(self, *args): r = self.client.post(reverse('telemetry_settings')) self.assertEqual(r.status_code, HTTPStatus.UNAUTHORIZED) - @override_settings(LAUNCHDARKLY_SDK_KEY='dummy_key') - @patch.object(feature_flags, 'LDClient') - def test_set_settings_when_feature_disabled(self, LDClient, *args): - LDClient.return_value.variation.return_value = False - self.user.organization = Organization.objects.get_or_create(id=123)[0] - self.client.force_authenticate(user=self.user) - r = self.client.get(reverse('telemetry_settings')) - self.assertEqual(r.status_code, HTTPStatus.SERVICE_UNAVAILABLE) - @override_settings(SEGMENT_WRITE_KEY='DUMMY_KEY_VALUE') def test_set_settings_without_org_id(self, *args): self.client.force_authenticate(user=self.user) diff --git a/ansible_wisdom/ai/api/tests/test_serializers.py b/ansible_wisdom/ai/api/tests/test_serializers.py index fe5ae1473..8289abb4d 100644 --- a/ansible_wisdom/ai/api/tests/test_serializers.py +++ b/ansible_wisdom/ai/api/tests/test_serializers.py @@ -188,22 +188,6 @@ def test_commercial_user_raises_exception_on_inlineSuggestion(self): with self.assertRaises(serializers.ValidationError): serializer.is_valid(raise_exception=True) - def test_commercial_user_not_telemetry_enabled_raises_exception_on_inlineSuggestion(self): - org = Mock(telemetry_opt_out=False) - user = Mock(rh_user_has_seat=True, organization=org) - request = Mock(user=user) - serializer = FeedbackRequestSerializer( - context={'request': request}, - data={ - "inlineSuggestion": { - "latency": 1000, - "userActionTime": 5155, - "action": "0", - "suggestionId": "a1b2c3d4-e5f6-a7b8-c9d0-e1f2a3b4c5d6", - } - }, - ) - # inlineSuggestion feedback raises exception when seat and not telemetry enabled with self.assertRaises(serializers.ValidationError): serializer.is_valid(raise_exception=True)