Skip to content

test: cover wildcard namespace disable round-trip#1042

Open
vibhor-aggr wants to merge 1 commit into
debug-js:masterfrom
vibhor-aggr:test/disable-enable-wildcard-roundtrip
Open

test: cover wildcard namespace disable round-trip#1042
vibhor-aggr wants to merge 1 commit into
debug-js:masterfrom
vibhor-aggr:test/disable-enable-wildcard-roundtrip

Conversation

@vibhor-aggr

Copy link
Copy Markdown

Summary

  • add regression coverage for debug.disable() namespace strings being accepted by debug.enable()
  • cover wildcard namespace patterns that start with * and include skip patterns
  • verify the re-enabled namespace still matches and skips the expected debug names

Closes #918.

Verification

  • npx mocha test.js --grep 'round trips wildcard namespaces through enable'
  • npm run test:node
  • touched-file lint: xo test.js with Node 24 compatibility shims for this repo's old lint dependencies
  • git diff --check

npm run test:browser was attempted, but the local environment has no ChromeHeadless binary and Karma exits with Please, set "CHROME_BIN" env variable.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

namespaces returned by debug.disable are not always accepted by debug.enable

1 participant