Additional updates based on comments, primarily in prose tests.

Author: caseyclements

source/bson-binary-vector/bson-binary-vector.md

@@ -180,7 +180,7 @@ End Function
 
 #### Validation
 
-Drivers MUST validate vector metadata and raise an error if any invariant is violated:
+Drivers MUST validate vector metadata and raise an exception if any invariant is violated:
 
 - When unpacking binary data into a FLOAT32 Vector structure, the length of the binary data following the dtype and
     padding MUST be a multiple of 4 bytes.
@@ -189,8 +189,8 @@ Drivers MUST validate vector metadata and raise an error if any invariant is vio
     - For a PACKED_BIT vector with non-zero padding, ignored bits SHOULD be zero.
         - When encoding, if ignored bits aren't zero, drivers SHOULD raise an exception, but drivers MAY leave them as-is if
             backwards-compatibility is a concern.
-        - When decoding, drivers SHOULD raise an error if decoding non-zero ignored bits, but drivers MAY choose not to for
-            backwards compatibility.
+        - When decoding, drivers SHOULD raise an exception if decoding non-zero ignored bits, but drivers MAY choose not to
+            for backwards compatibility.
         - Drivers SHOULD use the next major release to conform to ignored bits being zero.
 
 Drivers MUST perform this validation when a numeric vector and padding are provided through the API, and when unpacking

source/bson-binary-vector/tests/README.md

@@ -1,5 +1,3 @@
-from xmlrpc.client import Binary
-
 # Testing Binary subtype 9: Vector
 
 The JSON files in this directory tree are platform-independent tests that drivers can use to prove their conformance to
@@ -70,7 +68,9 @@ as all of its bits are ones.
 
 #### 1. Encoding
 
-- Test that an exception is raised when one attempts to encode a vector with non-zero ignored bits.
+- Test encoding with non-zero ignored bits. Use the driver API that validates vector metadata.
+- If the driver validates ignored bits are zero (preferred), expect an error. Otherwise expect the ignored bits are
+    preserved.
 
 ```python
 with pytest.raises(ValueError):
@@ -79,20 +79,17 @@ with pytest.raises(ValueError):
 
 ### 2. Decoding
 
-- Test that one can manually create a BSON Binary object following the vector structure (dtype + padding + data).
-- Test that this can be inserted and found
 - Test the behaviour of your driver when one attempts to decode from binary to vector.
     - e.g. As of pymongo 4.14, a warning is raised. From 5.0, it will be an exception.
 
 ```python
-v = Binary(b'\x10\x07\xff', subtype=9)
-clxn.insert_one({"v": v})
-found = clxn.find_one({"v": v})["v"]
-assert isinstance(found, Binary)
+b = Binary(b'\x10\x07\xff', subtype=9)
 with pytest.warns():
-    Binary.as_vector(found)
+    Binary.as_vector(b)
 ```
 
+Drivers MAY skip this test if they choose not to implement a `Vector` type.
+
 ### 3. Comparison
 
 Once we can guarantee that all ignored bits are non-zero, then equality can be tested on the binary subtype. Until then,
@@ -116,6 +113,8 @@ assert b1 != b2  # Unequal at naive Binary level
 assert v2 !=v1  # Also chosen to be unequal at BinaryVector level as [255] != [128]
 ```
 
+Drivers MAY skip this test if they choose not to implement a `Vector` type.
+
 ## FAQ
 
 - What MongoDB Server version does this apply to?