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

[ui, ci] UI test look-back and comparison for PRs #24604

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

philrenaud
Copy link
Contributor

Description

Phase 2 of the project to fix and improve our ember test audits. Phase 1 can be found here.

@philrenaud philrenaud self-assigned this Dec 4, 2024
.github/workflows/test-ui.yml Fixed Show fixed Hide fixed
.github/workflows/test-ui.yml Fixed Show fixed Hide fixed
.github/workflows/test-ui.yml Fixed Show fixed Hide fixed
.github/workflows/test-ui.yml Fixed Show fixed Hide fixed

# Try to list available artifacts first
- name: List artifacts
uses: actions/github-script@v7

Check warning

Code scanning / GitHub Actions Scanner

Missing pinned commit hash for GitHub Actions configuration Warning test

found external action "actions/github-script@v7" without pinned version hash
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from 6aacf40 to 0d96528 Compare December 4, 2024 16:01
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from 0d96528 to 9ddf7c6 Compare December 4, 2024 16:09
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from 9ddf7c6 to 3226226 Compare December 4, 2024 16:20
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from 3226226 to b71f943 Compare December 4, 2024 16:25
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from b71f943 to b59eb3b Compare December 4, 2024 16:27
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from b59eb3b to 5b76906 Compare December 4, 2024 16:54
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from 5b76906 to a7b089e Compare December 4, 2024 16:56
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from a7b089e to 9155550 Compare December 4, 2024 16:58
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from 9155550 to b792afc Compare December 4, 2024 16:59
@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from f828286 to 90eae68 Compare December 5, 2024 15:31
Copy link

github-actions bot commented Dec 5, 2024

Test Time Analysis

  • Total Tests: 1606
  • Significantly Slower: 64
  • Significantly Faster: 55

Most Significant Changes:

Integration | Helper | trim-path: it will remove a suffix slash

  • Current: 215ms
  • Historical Avg: 39.84615384615385ms
  • Change: 439.6%

Integration | Component | list pagination: when there is less pages than the total spread amount

  • Current: 215ms
  • Historical Avg: 43ms
  • Change: 400.0%

Unit | Ability | token: A non-management user can do nothing with tokens

  • Current: 120ms
  • Historical Avg: 25.923076923076923ms
  • Change: 362.9%

Integration | Component | das/recommendation-card: it renders diff calculations in a sentence with no aggregation for one allocatio

  • Current: 315ms
  • Historical Avg: 92.84615384615384ms
  • Change: 239.3%

Integration | Component | multi-select dropdown: list options have a zero tabindex and are therefore sequentially navigable

  • Current: 168ms
  • Historical Avg: 55.46153846153846ms
  • Change: 202.9%

Copy link

github-actions bot commented Dec 5, 2024

Test Time Analysis

  • Total Tests: 1606
  • Significantly Slower: 25
  • Significantly Faster: 10

Most Significant Changes:

Integration | Helper | trim-path: it will remove a suffix slash

  • Current: 215ms
  • Historical Avg: 39.42857142857143ms
  • Change: 445.3%

Integration | Component | das/recommendation-card: it renders diff calculations in a sentence with no aggregation for one allocatio

  • Current: 315ms
  • Historical Avg: 94.28571428571429ms
  • Change: 234.1%

Integration | Component | TopoViz::Datacenter: args get passed down to the TopViz::Node children

  • Current: 317ms
  • Historical Avg: 125.35714285714286ms
  • Change: 152.9%

Integration | Component | das/recommendation-card: it disables a resource’s toggle all toggle when there are no recommendations for it

  • Current: 144ms
  • Historical Avg: 59ms
  • Change: 144.1%

Integration | Component | line-chart: annotations will have staggered heights when too close to be positioned side-by-side

  • Current: 550ms
  • Historical Avg: 261.42857142857144ms
  • Change: 110.4%

Copy link

github-actions bot commented Dec 5, 2024

Test Time Analysis

  • Total Tests: 1606
  • Significantly Slower: 65
  • Significantly Faster: 56

Most Significant Changes:

Integration | Helper | trim-path: it will remove a suffix slash

  • Current: 215ms
  • Historical Avg: 39.42857142857143ms
  • Change: 445.3%

Integration | Component | list pagination: when there is less pages than the total spread amount

  • Current: 215ms
  • Historical Avg: 43.07142857142857ms
  • Change: 399.2%

Unit | Ability | token: A non-management user can do nothing with tokens

  • Current: 120ms
  • Historical Avg: 25.071428571428573ms
  • Change: 378.6%

Integration | Component | das/recommendation-card: it renders diff calculations in a sentence with no aggregation for one allocatio

  • Current: 315ms
  • Historical Avg: 94.28571428571429ms
  • Change: 234.1%

Integration | Component | multi-select dropdown: list options have a zero tabindex and are therefore sequentially navigable

  • Current: 168ms
  • Historical Avg: 54.57142857142857ms
  • Change: 207.9%

Copy link

github-actions bot commented Dec 5, 2024

Test Time Analysis

  • Total Tests: 1606
  • Significantly Slower: 59
  • Significantly Faster: 59

Most Significant Changes:

Integration | Helper | trim-path: it will remove a suffix slash

  • Current: 215ms
  • Historical Avg: 39.2ms
  • Change: 448.5%

Unit | Ability | token: A non-management user can do nothing with tokens

  • Current: 120ms
  • Historical Avg: 26.2ms
  • Change: 358.0%

Integration | Component | list pagination: when there is less pages than the total spread amount

  • Current: 215ms
  • Historical Avg: 56.93333333333333ms
  • Change: 277.6%

Integration | Component | trigger > Asynchronous Interactions: it can trigger an asynchronous action

  • Current: 546ms
  • Historical Avg: 158.46666666666667ms
  • Change: 244.6%

Integration | Component | das/recommendation-card: it renders diff calculations in a sentence with no aggregation for one allocatio

  • Current: 315ms
  • Historical Avg: 99.6ms
  • Change: 216.3%

Copy link

github-actions bot commented Dec 5, 2024

Test Time Analysis

  • Total Tests: 1606
  • Significantly Slower: 59
  • Significantly Faster: 59

Most Significant Changes:

Integration | Helper | trim-path: it will remove a suffix slash

  • Current: 215ms
  • Historical Avg: 39.2ms
  • Change: 448.5%

Unit | Ability | token: A non-management user can do nothing with tokens

  • Current: 120ms
  • Historical Avg: 26.2ms
  • Change: 358.0%

Integration | Component | list pagination: when there is less pages than the total spread amount

  • Current: 215ms
  • Historical Avg: 56.93333333333333ms
  • Change: 277.6%

Integration | Component | trigger > Asynchronous Interactions: it can trigger an asynchronous action

  • Current: 546ms
  • Historical Avg: 158.46666666666667ms
  • Change: 244.6%

Integration | Component | das/recommendation-card: it renders diff calculations in a sentence with no aggregation for one allocatio

  • Current: 315ms
  • Historical Avg: 99.6ms
  • Change: 216.3%

@philrenaud philrenaud force-pushed the b-ui/ember-test-lookback-and-compare branch from b123921 to aed33ae Compare December 5, 2024 16:12
Copy link

github-actions bot commented Dec 5, 2024

Test Time Analysis

  • Total Tests: 1606
  • Significantly Slower: 58
  • Significantly Faster: 62

Most Significant Changes:

Integration | Helper | trim-path: it will remove a suffix slash

  • Current: 215ms
  • Historical Avg: 39.625ms
  • Change: 442.6%

Unit | Ability | token: A non-management user can do nothing with tokens

  • Current: 120ms
  • Historical Avg: 25.375ms
  • Change: 372.9%

Integration | Component | list pagination: when there is less pages than the total spread amount

  • Current: 215ms
  • Historical Avg: 55.8125ms
  • Change: 285.2%

Integration | Component | trigger > Asynchronous Interactions: it can trigger an asynchronous action

  • Current: 546ms
  • Historical Avg: 157.8125ms
  • Change: 246.0%

Integration | Component | das/recommendation-card: it renders diff calculations in a sentence with no aggregation for one allocatio

  • Current: 315ms
  • Historical Avg: 96.375ms
  • Change: 226.8%

Copy link

github-actions bot commented Dec 5, 2024

Test Time Analysis

  • Total Tests: 1606
  • Significantly Slower: 61
  • Significantly Faster: 64

Most Significant Changes:

Unit | Ability | token: A non-management user can do nothing with tokens

  • Current: 120ms

  • Historical Avg: 24.11111111111111ms

  • Change: 397.7%

  • Trend: ▁▁▁▂▁▁▁█▁▁▁▁▁▁▁▁▁▁ ↑ 🔴

Integration | Helper | trim-path: it will remove a suffix slash

  • Current: 215ms

  • Historical Avg: 47.5ms

  • Change: 352.6%

  • Trend: ▁▁▂▁▁▁▁▁▁▁▁▁▁▁▁▁█▁ ↑ 🔴

Integration | Component | list pagination: when there is less pages than the total spread amount

  • Current: 215ms

  • Historical Avg: 53.94444444444444ms

  • Change: 298.6%

  • Trend: ▁▁▁█▁▁▁▁▁▁▁▁▁▁▁▁▁▁ ↑ 🔴

Integration | Component | trigger > Asynchronous Interactions: it can trigger an asynchronous action

  • Current: 546ms

  • Historical Avg: 152.22222222222223ms

  • Change: 258.7%

  • Trend: ▁▁▁▁▂▁█▁▂▁▅▁▄▁▂▁▂▁ ↑ 🔴

Integration | Component | das/recommendation-card: it renders diff calculations in a sentence with no aggregation for one allocatio

  • Current: 315ms

  • Historical Avg: 93.44444444444444ms

  • Change: 237.1%

  • Trend: ▃▁▁▂▁▁▁▂▂▁█▁▂▁▁▃▂▂ ↑ 🔴

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant