Skip to content

Inconsistencies: File upload tests should use query_string for params #1501

Description

@dcblundell
  • Upload tests use f-string query-param concatenation. Most upload tests in backend/ttnn_visualizer/tests/test_file_uploads.py (e.g. :328, :366, :395, :425, :457, :497, :535, :577) build the URL as f"/api/local/upload/...?instanceId={instance_id}" rather than the query_string={...} form. The instanceId value is UUID-like so this happens to be encoding-safe, but the form drifts from the Testing-section rule. Pre-existing; new tests should use query_string={...}.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type
No fields configured for issues without a type.

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions