address comments from Jerome

Author: baileympearson

source/index-management/index-management.rst

@@ -1128,3 +1128,4 @@ Changelog
 :2023-05-10:  Merge index enumeration and index management specs and get rid of references 
              to legacy server versions.
 :2023-05-18:  Add the search index management API.
+:2023-07-17:  Add search index management clarifications.

source/index-management/tests/README.rst

@@ -80,7 +80,7 @@ Case 1: Driver can successfully create and list search indexes
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
 #. Create a collection with a randomly generated name (referred to as ``coll0``).
-#. Create a new search index on ``coll0`` with the following definition:
+#. Create a new search index on ``coll0`` with the ``createSearchIndex`` helper.  Use the following definition:
 
   .. code:: typescript
 
@@ -91,17 +91,17 @@ Case 1: Driver can successfully create and list search indexes
       }
     }
 
-#. Assert that the command does not error and the server responds with a success.
+#. Assert that the command returns the name of the index: ``"test-search-index"``.
 #. Run ``coll0.listSearchIndexes()`` repeatedly every 5 seconds until the following condition is satisfied:
-   1. An index with the ``name`` of ``test-search-index`` is present.  This index is referred to as ``index``.
-   2. The ``latestDefinition`` property of the search index has the field ``queryable`` with a value of ``true``.
+   1. An index with the ``name`` of ``test-search-index`` is present and index has the field ``queryable`` with a value of ``true``.
+
 #. Assert that ``index`` has a property ``mappings`` whose value is ``{ dynamic: false }``
 
 Case 2: Driver can successfully create multiple indexes in batch
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
 #. Create a collection with a randomly generated name (referred to as ``coll0``).
-#. Create two new search indexes on ``coll0`` with the ``createSearchIndexes`` command.  Use the following
+#. Create two new search indexes on ``coll0`` with the ``createSearchIndexes`` helper.  Use the following
    definitions when creating the indexes.  These definitions are referred to as ``indexDefinitions``.
 
   .. code:: typescript
@@ -119,11 +119,11 @@ Case 2: Driver can successfully create multiple indexes in batch
       }
     }
 
-#. Assert that the command does not error and the server responds with a success.
+#. Assert that the command returns an array containing the new indexes' names: ``["test-search-index-1", "test-search-index-2"]``.
 #. Run ``coll0.listSearchIndexes()`` repeatedly every 5 seconds until the following condition is satisfied.  Store
    the result in ``indexes``.
-   1. An index with the ``name`` of ``test-search-index-1`` is present.  The ``latestDefinition`` property of the search index has the field ``queryable`` with a value of ``true``.
-   2. An index with the ``name`` of ``test-search-index-2`` is present.  The ``latestDefinition`` property of the search index has the field ``queryable`` with a value of ``true``.
+   1. An index with the ``name`` of ``test-search-index-1`` is present and index has the field ``queryable`` with a value of ``true``.
+   2. An index with the ``name`` of ``test-search-index-2`` is present and index has the field ``queryable`` with a value of ``true``.
 #. For each ``index`` in ``indexDefinitions``
    1. Find the matching index definition in ``indexes`` by matching on ``index.name``.  If no index exists, raise an error.
    2. Assert that the matching index ``mappings``, whose value is ``{ dynamic: false }``
@@ -143,10 +143,9 @@ Case 3: Driver can successfully drop search indexes
       }
     }
 
-#. Assert that the command does not error and the server responds with a success.
+#. Assert that the command returns the name of the index: ``"test-search-index"``.
 #. Run ``coll0.listSearchIndexes()`` repeatedly every 5 seconds until the following condition is satisfied:
-   1. An index with the ``name`` of ``test-search-index`` is present.  This index is referred to as ``index``.
-   2. The ``latestDefinition`` property of the search index has the field ``queryable`` with a value of ``true``.
+   1. An index with the ``name`` of ``test-search-index`` is present and index has the field ``queryable`` with a value of ``true``.
 
 #. Run a ``dropSearchIndexes`` on ``coll0``, using ``test-search-index`` for the name.
 #. Run ``coll0.listSearchIndexes()`` repeatedly every 5 seconds until ``listSearchIndexes`` returns an empty array.
@@ -168,10 +167,9 @@ Case 4: Driver can update a search index
       }
     }
 
-#. Assert that the command does not error and the server responds with a success.
+#. Assert that the command returns the name of the index: ``"test-search-index"``.
 #. Run ``coll0.listSearchIndexes()`` repeatedly every 5 seconds until the following condition is satisfied:
-   1. An index with the ``name`` of ``test-search-index`` is present.  This index is referred to as ``index``.
-   2. The ``latestDefinition`` property of the search index has the field ``queryable`` with a value of ``true``.
+   1. An index with the ``name`` of ``test-search-index`` is present and index has the field ``queryable`` with a value of ``true``.
 
 #. Run a ``updateSearchIndex`` on ``coll0``, using the following definition.
 
@@ -187,8 +185,8 @@ Case 4: Driver can update a search index
 #. Assert that the command does not error and the server responds with a success.
 #. Run ``coll0.listSearchIndexes()`` repeatedly every 5 seconds until the following condition is satisfied:
    1. An index with the ``name`` of ``test-search-index`` is present.  This index is referred to as ``index``.
-   2. The ``latestDefinition`` property of the search index has the field ``queryable`` with a value of ``true``.
-   3. The ``latestDefinition`` property of the search index has the field ``status`` with the value of ``READY``.
+   2. The index has a field ``queryable`` with a value of ``true``.
+   3. The index has a ``latestDefinition`` property.  The ``latestDefinition`` field has a property ``status`` with the value of ``READY``.
 
 #. Assert that an index is present with the name ``test-search-index`` and the definition has a
   property ``mappings``, whose value is ``{ dynamic: true }``