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

test: refactor operate authorization and statistics integration tests #18470

Merged
merged 4 commits into from
May 14, 2024

Conversation

matt-whiteman
Copy link
Contributor

@matt-whiteman matt-whiteman commented May 13, 2024

Description

  • Removed DecisionOldIT - covered by existing tests
  • Renamed ProcessStatisticsIT to reflect actual code under test, removed some unnecessary tests
  • Removed ProcessStatisticsAfterArchivingIT - archive tests addressed later
  • Refactored AuthorizationIT to be faster
  • Moved validation code for FlowNodeStatistics into ProcessInstanceRequestValidator and added unit tests

Related issues

related to #17650

@matt-whiteman matt-whiteman changed the title test: refactor operate authorization and statistics integratio ntests test: refactor operate authorization and statistics integration tests May 13, 2024
@github-actions github-actions bot added the component/operate Related to the Operate component/team label May 13, 2024
Copy link
Contributor

github-actions bot commented May 13, 2024

Operate Test Results

798 tests   795 ✅  10m 36s ⏱️
132 suites    3 💤
132 files      0 ❌

Results for commit f1ef198.

♻️ This comment has been updated with latest results.

@matt-whiteman matt-whiteman requested a review from a team May 13, 2024 22:22
@matt-whiteman matt-whiteman marked this pull request as ready for review May 13, 2024 22:22
@matt-whiteman matt-whiteman added this pull request to the merge queue May 14, 2024
Merged via the queue into main with commit 8b5a1dd May 14, 2024
21 of 22 checks passed
@matt-whiteman matt-whiteman deleted the refactor-operate-authorize-it branch May 14, 2024 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/operate Related to the Operate component/team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants