Skip to content

Centralize wire-protocol namespace validation tests (TEST_COVERAGE.md §19) #607

Description

@imforster

Problem

TEST_COVERAGE.md §19 specifies that wire-protocol namespace validation should have a single centralized test site, with each command running only one representative case. Currently every command (compact, drop, count, aggregate, renameCollection, insert, and others) duplicates a full 15-case BSON-type rejection matrix for the collection-name field, asserting INVALID_NAMESPACE_ERROR.

What to do

  1. Create the centralized namespace validation test site (location TBD per §19).
  2. Trim each command's collection-name rejection matrix to 1-2 representative cases that delegate to the central site.

Scope

Commands known to duplicate the matrix:

  • insert (test_insert_argument_handling.py:21-122)
  • compact
  • drop
  • count
  • aggregate
  • renameCollection
  • Others

Context

Identified during PR #597 review. Left as-is in that PR to stay consistent with sibling commands. Should be addressed as a single sweep across all commands.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions