Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Connect Koswat workflow with infrastructure export #161

Open
8 tasks
ArdtK opened this issue Sep 19, 2024 · 0 comments · May be fixed by #167
Open
8 tasks

Connect Koswat workflow with infrastructure export #161

ArdtK opened this issue Sep 19, 2024 · 0 comments · May be fixed by #167
Assignees
Labels
enhancement New feature or request

Comments

@ArdtK
Copy link
Contributor

ArdtK commented Sep 19, 2024

(Related to #153)

Assuming Koswat can generate a .csv file representing the InfrastructureCosts per type ReinforcementProfileProtocol and Point, we need to ensure that this report is correctly generated from the KoswatSummary and taken during any type of analysis when the property infrastructuur = True is set in the koswat_general.ini file (section [Infrastructuur]).

What needs to be done?

  • Adapt one of the existing acceptance tests to run with / without infrastructure surroundings. Ensure for both cases the test is run successfully.
    • Update and correct Koswat logic so that the tests can be run and generate valid data.
    • Tests without infrastructure should give back the same results as the ones being generated now. (Perhaps with the addition of the extra column that should just be 0).
  • Extend running acceptance test data to all available cases. This will generate twice as many data as earlier, so needs to be done carefully.
  • Update test bench cases so that no "slow" tests are running in the "fast" category.

Acceptance criteria

  • There are sufficient test cases running with (infrastructure) surroundings.
  • The results are validated by the Product Owner.
  • The test reference data is updated with the validated results.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants