remove self imposed complexity of the method

Author: tdstein

src/posit/connect/resources.py

@@ -103,19 +103,6 @@ def _create_instance(self, path: str, /, **kwargs: Any) -> T:
         """Create an instance of 'T'."""
         raise NotImplementedError()
 
-    def cached(self) -> bool:
-        """Returns True if the collection is cached.
-
-        Returns
-        -------
-        bool
-
-        See Also
-        --------
-        reload
-        """
-        return self._cache is not None
-
     def reload(self) -> Self:
         """Reloads the collection from Connect.
 
@@ -194,7 +181,7 @@ def find(self, uid) -> T:
         """
         Find a record by its unique identifier.
 
-        If the cache is already populated, it is checked first for matching record. If not, a conventional GET request is made to the Connect server.
+        Fetches the record from Connect by it's identifier.
 
         Parameters
         ----------
@@ -205,12 +192,6 @@ def find(self, uid) -> T:
         -------
         T
         """
-        if self.cached():
-            conditions = {self._uid: uid}
-            result = self.find_by(**conditions)
-            if result:
-                return result
-
         endpoint = self._ctx.url + self._path + uid
         response = self._ctx.session.get(endpoint)
         result = response.json()

tests/posit/connect/test_jobs.py

@@ -46,25 +46,6 @@ def test(self):
         job = content.jobs.find("tHawGvHZTosJA2Dx")
         assert job["key"] == "tHawGvHZTosJA2Dx"
 
-    @responses.activate
-    def test_cached(self):
-        responses.get(
-            "https://connect.example/__api__/v1/content/f2f37341-e21d-3d80-c698-a935ad614066",
-            json=load_mock("v1/content/f2f37341-e21d-3d80-c698-a935ad614066.json"),
-        )
-
-        responses.get(
-            "https://connect.example/__api__/v1/content/f2f37341-e21d-3d80-c698-a935ad614066/jobs",
-            json=load_mock("v1/content/f2f37341-e21d-3d80-c698-a935ad614066/jobs.json"),
-        )
-
-        c = Client("https://connect.example", "12345")
-        content = c.content.get("f2f37341-e21d-3d80-c698-a935ad614066")
-
-        assert content.jobs
-        job = content.jobs.find("tHawGvHZTosJA2Dx")
-        assert job["key"] == "tHawGvHZTosJA2Dx"
-
     @responses.activate
     def test_miss(self):
         responses.get(