feat: extend content.get() to automatically use include param (#420)

Adds `include` param support for `content.get()`

- Add the param support, opting to hardcode it so it is always used
- Add unit test
- Add integration test
- Also manually verified by testing locally against a live Connect
instance

Replaces previous fork PR:
#416

Author: cgraham-rs

integration/tests/posit/connect/test_content.py

@@ -23,7 +23,17 @@ def test_count(self):
         assert self.client.content.count() == 1
 
     def test_get(self):
-        assert self.client.content.get(self.content["guid"]) == self.content
+        item = self.client.content.get(self.content["guid"])
+        # Check that essential fields match instead of exact equality
+        for key in self.content:
+            assert key in item
+            assert item[key] == self.content[key]
+        if CONNECT_VERSION >= version.parse("2024.06.0"):
+            # get() always includes owner, tags, and vanity_url. Owner data is always present in
+            # all content, tags and vanity_url are only present if explicitly set in the content.
+            assert "owner" in item
+            assert "tags" not in item
+            assert "vanity_url" not in item
 
     def test_find(self):
         assert self.client.content.find()

src/posit/connect/content.py

@@ -989,10 +989,14 @@ def get(self, guid: str) -> ContentItem:
         Parameters
         ----------
         guid : str
+            The unique identifier of the content item.
 
         Returns
         -------
         ContentItem
         """
-        response = self._ctx.client.get(f"v1/content/{guid}")
+        # Always request all available optional fields for the content item
+        params = {"include": "owner,tags,vanity_url"}
+
+        response = self._ctx.client.get(f"v1/content/{guid}", params=params)
         return ContentItem(self._ctx, **response.json())

tests/posit/connect/test_content.py

@@ -169,7 +169,7 @@ def test(self):
         assert content[2]["name"] == "My-Streamlit-app"
 
     @responses.activate
-    def test_params_include(self):
+    def test_params_include_string(self):
         # behavior
         mock_get = responses.get(
             "https://connect.example/__api__/v1/content",
@@ -326,7 +326,7 @@ def test_name(self):
         assert content_item["name"] == name
 
     @responses.activate
-    def test_params_include(self):
+    def test_params_include_string(self):
         # behavior
         mock_get = responses.get(
             "https://connect.example/__api__/v1/content",
@@ -365,9 +365,11 @@ def test_params_include_none(self):
 class TestContentsGet:
     @responses.activate
     def test(self):
+        # All calls to get() should automatically include all available optional fields
         responses.get(
             "https://connect.example/__api__/v1/content/f2f37341-e21d-3d80-c698-a935ad614066",
             json=load_mock("v1/content/f2f37341-e21d-3d80-c698-a935ad614066.json"),
+            match=[matchers.query_param_matcher({"include": "owner,tags,vanity_url"})],
         )
         con = Client("https://connect.example", "12345")
         get_one = con.content.get("f2f37341-e21d-3d80-c698-a935ad614066")