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.

Sign up for GitHub

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

Jump to bottom

FOUR-8066 Process Testing part 1 #1667

Merged
merged 4 commits into from
Sep 9, 2023
Merged

FOUR-8066 Process Testing part 1 #1667

merged 4 commits into from
Sep 9, 2023

Conversation

caleeli
Copy link
Collaborator

@caleeli caleeli commented Sep 8, 2023

Process Testing part 1

  • Run a process in an isolated environment.
  • Save scenarios
  • Run scenarios
  • Run processes from modeler

@processmaker-sonarqube
Copy link

SonarQube Quality Gate

Quality Gate failed

Failed condition 0.0% 0.0% Coverage on New Code (is less than 80%)

See analysis details on SonarQube

@caleeli
Copy link
Collaborator Author

caleeli commented Sep 8, 2023

A test was added to cover half of the changes: tests/unit/components/toolbar/toolbarExtraMenu.spec.js
But it was counted as new code to be covered.
Code added to Modeler.vue was not been able to test because require a global configuration to enable extra packages.

@ryancooley ryancooley merged commit 1796ddb into next Sep 9, 2023
5 of 6 checks passed
@ryancooley ryancooley deleted the feature/FOUR-8066 branch September 9, 2023 03:18
@caleeli caleeli restored the feature/FOUR-8066 branch October 2, 2023 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@caleeli @ryancooley