celestiaorg / knuu

Integration Test Framework
Apache License 2.0
38 stars 31 forks source link

bug: Scope/Namespace is not sanitized before printed #449

Closed smuu closed 1 week ago

smuu commented 1 week ago

When using upper-case characters as a scope, the resulting namespace is lower-case as we sanitize the name before creating the namespace. But the log shows the scope with the upper-case characters, which creates confusion as the scope != namespace.

I see two options:

  1. Sanitize the scope name if initializing knuu with a scope
  2. Require the scope to be already sanitized when initializing knuu with a scope -> this would be a small breaking change