bug: create dialog tests tries to open non-existent enduser tokens file #1402
Assignees
Comments
github-project-automation
bot
moved this to New issues
in ⚠️ Dialogporten / Arbeidsflate - GAMMEL - se https://github.com/orgs/Altinn/projects/146 ⚠️
dagfinno
moved this from New issues
to Doing
in ⚠️ Dialogporten / Arbeidsflate - GAMMEL - se https://github.com/orgs/Altinn/projects/146 ⚠️
4 tasks
dagfinno
added a commit
that referenced
this issue
Nov 6, 2024
return an empty array if testdatafile not found, check for empty testdata in k6 tests <!--- Provide a general summary of your changes in the Title above --> ## Description ## Related Issue(s) - #1402 ## Verification - [ ] **Your** code builds clean without any errors or warnings - [ ] Manual testing done (required) - [ ] Relevant automated test added (if you find this hard, leave it and we'll help out) ## Documentation - [ ] Documentation is updated (either in `docs`-directory, Altinnpedia or a separate linked PR in [altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if applicable) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced error handling in various performance testing scripts to ensure required data arrays are populated before execution. - Added a new function for reading and parsing CSV files, improving error handling during file operations. - **Bug Fixes** - Enhanced robustness of performance tests by preventing execution with empty datasets, which could lead to runtime errors. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
github-project-automation
bot
moved this from Doing
to Done
in ⚠️ Dialogporten / Arbeidsflate - GAMMEL - se https://github.com/orgs/Altinn/projects/146 ⚠️
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
readTestdata.js always tries to read all testdata-files, also the ones not created and not needed for the test (create dialogs does not need enduser tokens)
The text was updated successfully, but these errors were encountered: