Skip to content

Comments

Fix test_bluesky_sim_air_traffic_data incorrect type handling#97

Merged
RomanPszonka merged 1 commit intomainfrom
fix_test_bluesky_sim_air_traffic_data
Feb 12, 2026
Merged

Fix test_bluesky_sim_air_traffic_data incorrect type handling#97
RomanPszonka merged 1 commit intomainfrom
fix_test_bluesky_sim_air_traffic_data

Conversation

@seslash
Copy link
Contributor

@seslash seslash commented Feb 12, 2026

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a bug in the test_bluesky_sim_air_traffic_data scenario where the wrong attribute was being accessed on the StepResult object returned by generate_bluesky_sim_air_traffic_data(). The method is decorated with @scenario_step, which wraps its return value in a StepResult object with a result attribute containing the actual data.

Changes:

  • Corrected attribute access from result.details to result.result to properly extract air traffic observations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@RomanPszonka RomanPszonka merged commit 577bad4 into main Feb 12, 2026
8 checks passed
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