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

feat: remove feature flag and msw mocks for operations panel status on batch modifications #18194

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

vitorwtavares
Copy link
Contributor

@vitorwtavares vitorwtavares commented May 2, 2024

Description

  • Removes feature flag
  • Removes options for skipping tests
  • Removes msw mocks

Related issues

relates to camunda/operate#6354
is blocked by #18171

@github-actions github-actions bot added the component/operate Related to the Operate component/team label May 2, 2024
@vitorwtavares vitorwtavares self-assigned this May 2, 2024
Copy link
Contributor

github-actions bot commented May 2, 2024

Operate Test Results

801 tests  ±0   786 ✅ ±0   14m 8s ⏱️ +49s
135 suites ±0    15 💤 ±0 
135 files   ±0     0 ❌ ±0 

Results for commit 2fcfcb5. ± Comparison against base commit 51a9d1a.

♻️ This comment has been updated with latest results.

@vitorwtavares vitorwtavares force-pushed the feat-operation-panel-batch-status branch from 735abcb to a1e01b4 Compare May 3, 2024 13:37
@vitorwtavares vitorwtavares force-pushed the feat-operation-panel-batch-status branch from 2fcfcb5 to c71717a Compare May 16, 2024 18:18
Copy link
Contributor

github-actions bot commented May 16, 2024

Operate Unit Tests Results

299 tests  ±0   298 ✅ ±0   5m 56s ⏱️ - 1m 16s
 45 suites ±0     1 💤 ±0 
 45 files   ±0     0 ❌ ±0 

Results for commit 7e82a28. ± Comparison against base commit e6a1994.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented May 16, 2024

Operate Integration Tests Results

523 tests  ±0   521 ✅ ±0   7m 21s ⏱️ +7s
 88 suites ±0     2 💤 ±0 
 88 files   ±0     0 ❌ ±0 

Results for commit 7e82a28. ± Comparison against base commit e6a1994.

♻️ This comment has been updated with latest results.

@vitorwtavares vitorwtavares force-pushed the feat-operation-panel-batch-status branch from c71717a to 9f0bb49 Compare May 21, 2024 20:41
Copy link
Contributor

github-actions bot commented May 21, 2024

Operate Opensearch ITs Results

182 tests  ±0   182 ✅ ±0   1m 0s ⏱️ ±0s
 19 suites ±0     0 💤 ±0 
 19 files   ±0     0 ❌ ±0 

Results for commit 7e82a28. ± Comparison against base commit e6a1994.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@pedesen pedesen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need to wait until #18171 is merged. As you can see in the updated visual regression snapshots, the success/failed count is not visible due to the old API endpoint used.

@vitorwtavares vitorwtavares requested a review from Szik May 27, 2024 14:34
@vitorwtavares vitorwtavares force-pushed the feat-operation-panel-batch-status branch from 85095a8 to 2d3b979 Compare May 28, 2024 20:23
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