Skip to content

Add integration test for empty feature name fallback path#58

Open
micheleh wants to merge 1 commit intomasterfrom
test/empty-feature-name-it-case
Open

Add integration test for empty feature name fallback path#58
micheleh wants to merge 1 commit intomasterfrom
test/empty-feature-name-it-case

Conversation

@micheleh
Copy link
Copy Markdown
Collaborator

@micheleh micheleh commented Apr 5, 2026 •
edited
Loading

Summary

  • Adds an integration test case that exercises the scenario-name lookup
    fallback in Bdd2Octane.run(), triggered when getFeatureName()
    returns empty (classname="EMPTY_NAME")
  • Uses existing test data (TEST-empty-feature-name.xml and
    feature_name_is_empty.feature) that was previously only used in a
    handler unit test
  • Covers FeatureFileLocator.getOctaneFeatureByScenarioNameFromFile and
    OctaneFeatureLocator.getFeatureByScenarioName two-phase lookup

Test plan

  • New IT case passes (test index 36)
  • Full IT suite passes (38 tests, 0 failures)

@micheleh @claude
Add integration test for empty-feature-name scenario-name fallback
b713998 
The fallback path in Bdd2Octane.run() that resolves features by scenario
name when the feature name is empty was not covered by any integration
test. Wire the existing empty-feature-name test data into
Bdd2OctaneHandlerITCase to exercise this path end-to-end.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@micheleh micheleh requested a review from nissimshitrit April 5, 2026 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nissimshitrit nissimshitrit Awaiting requested review from nissimshitrit

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@micheleh