Skip to content

test(api): rename clear_worker_metrics test to satisfy FT003 naming rule#8

Merged
AAshGray merged 1 commit into
mainfrom
fix/test-name-clear-worker-metrics
May 21, 2026
Merged

test(api): rename clear_worker_metrics test to satisfy FT003 naming rule#8
AAshGray merged 1 commit into
mainfrom
fix/test-name-clear-worker-metrics

Conversation

@SketchRudy
Copy link
Copy Markdown
Collaborator

Renames the only FT003-violating test in api/tests/unit/metrics/test_unit_worker_metrics.py
so flagsmith-lint-tests passes on the upstream PR (Flagsmith#7546).

Adds the missing condition segment per test_{subject}{condition}{expected outcome}:

  • before: test_clear_worker_metrics__removes_gauge_label
  • after: test_clear_worker_metrics__label_present__removes_gauge_label

@AAshGray AAshGray merged commit 96cd27d into main May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants