Skip to content

Backfill step definitions should populate or remove unused parameter_ids tracker #568

Description

@kbighorse

Context

Identified during review of #558 (558-radionuclides-backfill branch).

Problem

_ensure_backfill_tracking in chemistry-backfill.py initializes a parameter_ids list in _backfill_created, but nothing ever appends to it. This creates confusion about whether Parameter cleanup is handled.

Fix

Either populate parameter_ids when _get_or_create_parameter creates new Parameters and clean them up in after_scenario, or remove the unused key to avoid misleading future developers.

Files

  • tests/features/steps/chemistry-backfill.py (line ~52, _ensure_backfill_tracking)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions