Merge pull request #824 from CitrineInformatics/remove-deprecated-deserialization-except

Remove unnecessary deserialization protection.

Author: anoto-moniz

src/citrine/__version__.py

@@ -1 +1 @@
-__version__ = '2.2.0'
+__version__ = '2.2.1'

src/citrine/_rest/collection.py

@@ -125,10 +125,4 @@ def _build_collection_elements(self,
 
         """
         for element in collection:
-            try:
-                yield self.build(element)
-            except(KeyError, ValueError) as e:
-                # TODO(PLA-9109): This is a patch to handle deprecated predictors client side
-                # Remove when predictors are migrated
-                logger.warning(f"Building element skipped due to error: {e}")
-                pass
+            yield self.build(element)

tests/resources/test_project.py

@@ -350,19 +350,6 @@ def test_list_no_team(session):
     assert 5 == len(projects)
 
 
-def test_list_projects_filters_non_projects(collection, session, caplog):
-    # Given
-    projects_data = ProjectDataFactory.create_batch(5)
-    projects_data.append({'foo': 'not a project'})
-    session.set_response({'projects': projects_data})
-
-    # Then
-    with caplog.at_level(WARNING):
-        caplog.clear()
-        assert len(list(collection.list())) == 5 # Skip the bad one
-        assert any(r.levelno == WARNING for r in caplog.records)
-
-
 def test_list_projects_with_page_params(collection, session):
     # Given
     project_data = ProjectDataFactory()