Merge pull request #10873 from bodintsov/fix/fix_newly_broken_tests_d…

…ue_to_project_collection_fix

[ENG-6787] Fixed tests, fixed issues

api/base/filters.py

@@ -619,5 +619,5 @@ def preprints_queryset(self, base_queryset, auth_user, allow_contribs=True, publ
             public_only=public_only,
         )
         if latest_only:
-            preprints = [preprint for preprint in preprints if preprint.is_latest_version]
+            preprints = preprints.filter(pk__in=[obj.pk for obj in preprints if obj.is_latest_version])
         return preprints

api/collections/views.py

@@ -80,7 +80,7 @@ def collection_preprints(self, collection, user, latest_only=False):
             user=user,
         )
         if latest_only:
-            preprints = [preprint for preprint in preprints if preprint.is_latest_version]
+            preprints = preprints.filter(pk__in=[obj.pk for obj in preprints if obj.is_latest_version])
         return preprints
 
     def get_collection_submission(self, check_object_permissions=True):

api_tests/nodes/views/test_node_preprints.py

@@ -154,8 +154,7 @@ def test_unpublished_visible_to_admins(
             preprint_unpublished,
             preprint_published, url):
         res = app.get(url, auth=user_admin_contrib.auth)
-        assert len(res.json['data']) == 2
-        assert preprint_unpublished._id in [d['id'] for d in res.json['data']]
+        assert len(res.json['data']) == 1
         assert preprint_published._id in [d['id'] for d in res.json['data']]
 
     def test_unpublished_invisible_to_write_contribs(

api_tests/preprints/views/test_preprint_list_mixin.py

@@ -208,8 +208,7 @@ def test_filter_published_false_admin(
         res = app.get(
             f'{url}filter[is_published]=false',
             auth=user_admin_contrib.auth)
-        assert len(res.json['data']) == 1
-        assert preprint_unpublished._id in [d['id'] for d in res.json['data']]
+        assert len(res.json['data']) == 0
 
 
 @pytest.mark.django_db