Update test_filter_removed_access_documents for JS (#307)

* Update test_filter_removed_access_documents for JS

Not checking for the filtered access removed doc notification for JS. Based on the discussion, we will not implement document ended notification for the docs rejected by the pull filter in JS. We may implement it based on demand.

From Jens, this feature doesn’t make sense as the rejection from the user’s replication filters shouldn’t be treat as an error. Also it happens on the platform as a side effect due to an error thrown by LiteCore. I have created CBL-7645 for changing the behavior on the other platforms in the future release.

* Update per feedback

Author: pasin

spec/tests/dev_e2e/003-replication-auto-purge.md

@@ -182,7 +182,10 @@ Test that when the removed access documents are filtered, the removed access doc
    * credentials: user1/pass
 8. Check document replications:
    * `post_1` has access-removed flag set with no error.
-   * `post_2` has access-removed flag set with WebSocket/403 ("CBL, 10403) error.
+   * Note: JS doesn't notify document ended error notifications when documents 
+      are rejected by pull replication filters. The other platforms plan to align 
+      this behavior. So we are not checking the error from the filtered removed 
+      revision here. See CBL-7246 and CBL-7645 for more details. 
 9. Check local documents:
    * `post_1` was purged.
    * `post_2` still exists.

tests/dev_e2e/test_replication_auto_purge.py

@@ -732,7 +732,10 @@ async def test_filter_removed_access_documents(
         self.mark_test_step("""
             Check document replications
                * `post_1` has access-removed flag set with no error.
-               * `post_2` has access-removed flag set with WebSocket/403 ("CBL, 10403) error.
+               * Note: JS doesn't notify document ended error notifications when documents 
+                 are rejected by pull replication filters. The other platforms plan to align 
+                 this behavior. So we are not checking the error from the filtered removed 
+                 revision here. See CBL-7246 and CBL-7645 for more details. 
         """)
         await repl2.wait_for_all_doc_events(
             {
@@ -741,15 +744,7 @@ async def test_filter_removed_access_documents(
                     "post_1",
                     ReplicatorType.PULL,
                     ReplicatorDocumentFlags.ACCESS_REMOVED,
-                ),
-                WaitForDocumentEventEntry(
-                    "_default.posts",
-                    "post_2",
-                    ReplicatorType.PULL,
-                    ReplicatorDocumentFlags.ACCESS_REMOVED,
-                    "CBL",
-                    10403,
-                ),
+                )
             }
         )