diff --git a/chromadb/test/property/strategies.py b/chromadb/test/property/strategies.py index 785d2775723..9196e6b22ec 100644 --- a/chromadb/test/property/strategies.py +++ b/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 diff --git a/chromadb/test/property/test_collections.py b/chromadb/test/property/test_collections.py index 18ef6f609fe..3aa2e454c07 100644 --- a/chromadb/test/property/test_collections.py +++ b/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]