Skip to content

Commit

Permalink
Fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
romartin committed Mar 25, 2024
1 parent 189f161 commit 7220c8a
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 34 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down Expand Up @@ -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)
Expand Down
16 changes: 0 additions & 16 deletions ansible_wisdom/ai/api/tests/test_serializers.py
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down

0 comments on commit 7220c8a

Please sign in to comment.