[CLN] Remove prints and add documentation

chromadb/test/property/strategies.py

@@ -239,6 +239,10 @@ def embedding_function_strategy(
 class ExternalCollection:
     """
     An external view of a collection.
+    
+    This strategy only contains information about a collection that a client of Chroma
+    sees -- that is, it contains none of Chroma's internal bookkeeping. It should
+    be used to test the API and client code.
     """
     name: str
     metadata: Optional[types.Metadata]
@@ -249,6 +253,10 @@ class ExternalCollection:
 class Collection(ExternalCollection):
     """
     An internal view of a collection.
+
+    This strategy contains all the information Chroma uses internally to manage a
+    collection. It is a superset of ExternalCollection and should be used to test
+    internal Chroma logic.
     """
     id: uuid.UUID
     dimension: int

chromadb/test/property/test_collections.py

@@ -199,8 +199,6 @@ def modify_coll(
             return multiple()
 
         c = self.api.get_collection(name=coll.name)
-        print("before API:", c)
-        print("model:", self.model)
         _metadata: Optional[Mapping[str, Any]] = self.model[coll.name]
         _name: str = coll.name
         if new_metadata is not None:
@@ -228,8 +226,6 @@ def modify_coll(
         self.set_model(_name, _metadata)
         c.modify(metadata=_metadata, name=_name)
         c = self.api.get_collection(name=coll.name)
-        print("after API:", c)
-        print("model:", self.model)
 
         assert c.name == coll.name
         assert c.metadata == self.model[coll.name]