Merge pull request #37165 from openedx/feanil/remove_unnecessary_pin

feanil/remove unnecessary pin

openedx/core/djangoapps/agreements/serializers.py

@@ -16,7 +16,7 @@ class IntegritySignatureSerializer(serializers.ModelSerializer):
     created_at = serializers.DateTimeField(source='created')
 
     class Meta:
-        model = IntegritySignature()
+        model = IntegritySignature
         fields = ('username', 'course_id', 'created_at')
 
 

openedx/core/djangoapps/lang_pref/tests/test_middleware.py

@@ -233,7 +233,7 @@ class TestUserPreferenceMiddleware(CacheIsolationTestCase):
 
         # Preference is the same as the cookie, shouldn't write to the database
 
-        with self.assertNumQueries(3):
+        with self.assertNumQueries(2):
             self.middleware.process_request(self.request)
 
         assert get_user_preference(self.user, LANGUAGE_KEY) == 'es'

openedx/core/djangoapps/notifications/tests/test_views.py

@@ -443,7 +443,7 @@ class NotificationReadAPIViewTestCase(APITestCase):
         response = self.client.patch(self.url, data)
 
         self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
-        self.assertEqual(response.data["detail"], 'Not found.')
+        self.assertEqual(response.data["detail"].code, 'not_found')
 
     def test_mark_notification_read_with_app_name_and_notification_id(self):
         # Create a PATCH request to mark notification as read for existing app e.g 'discussion' and notification_id: 2