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

frontend: Fix bug in Cypress plugin #2486

Merged

Conversation

gorankokin
Copy link
Contributor

Fix bug in Cypress custom plugin that provides executions for Data Jobs e2e tests.
Array was used as integer in one calculation, and because of it there were some unexpected results in some scenarios.
Applied fix uses Array length for calculate the number of left executions that should be executed serial.

Fix bug in Cypress custom plugin that provides executions.
Array was used as integer in one calculation, that
was provided unexpected results in some scenarios.
Fix uses Array length for calculation of left exec.

Signed-off-by: Goran Kokinovski <[email protected]>
@gorankokin gorankokin added the area/frontend Related to changes in the folder projects/frontend for details about ui please see readme label Jul 26, 2023
@gorankokin gorankokin self-assigned this Jul 26, 2023
@gorankokin gorankokin requested a review from hzhristova as a code owner July 26, 2023 13:50
@gorankokin gorankokin merged commit e479386 into main Jul 27, 2023
@gorankokin gorankokin deleted the person/gkokinovski/fix-bug-for-executions-in-data-jobs-plugin branch July 27, 2023 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to changes in the folder projects/frontend for details about ui please see readme cla-not-required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants