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

Fix Cypress issues #10

Merged
merged 7 commits into from
Jul 1, 2021
Merged

Fix Cypress issues #10

merged 7 commits into from
Jul 1, 2021

Conversation

ashika01
Copy link
Contributor

@ashika01 ashika01 commented Jun 29, 2021

Description of changes:

  • Add awsui style
  • Fix webpackComplilation error

Note:

  • snapshot added are from name change
=============================== Coverage summary ===============================
Statements   : 89.42% ( 2433/2721 )
Branches     : 84.62% ( 1128/1333 )
Functions    : 86.38% ( 501/580 )
Lines        : 89.06% ( 2321/2606 )
================================================================================

Test Suites: 75 passed, 75 total
Tests:       6 skipped, 1219 passed, 1225 total
Snapshots:   4 passed, 4 total
Time:        92.224s
Ran all test suites.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@diehbria
Copy link
Contributor

make sure to add the yarn test results in the description

some of the new snapshots are locking in style regressions, is there a way we can make this CR with less snapshot image changes so that we ensure that the snapshots are correct?

@ashika01
Copy link
Contributor Author

ashika01 commented Jun 30, 2021

Could you point which snapshots has style regression, I checked them all and they all seem to be from name changes? But I might have might some. I actually think I might have to remove the old ones out.

@ashika01
Copy link
Contributor Author

ashika01 commented Jun 30, 2021

Oh actually I think I could just move some of the old ones to the new folders and rerun the snaps to see the diff. Let me that and I think will point me to style regression.

@ashika01
Copy link
Contributor Author

Okay, I moved the old screenshot over to the proper folder. No new screenshot generated here. I still get couple of 0.02 - 0.05% diff failures will leave it at that for now.


  (Run Finished)


       Spec                                              Tests  Passing  Failing  Pending  Skipped
  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ ✔  charts/alarms.spec.ts                    00:25       18       18        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/bar-chart/empty-status.spec.      00:04        2        2        -        -        - │
  │    ts                                                                                          │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/bar-chart/errors.spec.ts          00:02        1        1        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✖  charts/bar-chart/gestures.spec.ts        00:14        3        1        2        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/bar-chart/loading.spec.ts         00:02        1        1        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✖  charts/bar-chart/sc-webgl-bar-chart      00:46       15       12        3        -        - │
  │    .spec.ts                                                                                    │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/bar-chart/threshold-colorati      00:11        6        6        -        -        - │
  │    on.spec.ts                                                                                  │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/bar-chart/tooltip.spec.ts         00:06        3        3        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/cross-resolution.spec.ts          00:03        1        1        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/line-chart/empty-status.spec      00:04        2        2        -        -        - │
  │    .ts                                                                                         │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/line-chart/errors.spec.ts         00:03        1        1        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✖  charts/line-chart/gestures.spec.ts       00:18        4        1        3        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/line-chart/loading.spec.ts        00:02        1        1        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✖  charts/line-chart/sc-webgl-chart.sp      00:45       20       17        1        2        - │
  │    ec.ts                                                                                       │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/line-chart/threshold-colorat      00:09        5        5        -        -        - │
  │    ion.spec.ts                                                                                 │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/line-chart/tooltip.spec.ts        00:04        2        2        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/performance.spec.ts                27ms        6        -        -        6        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/rendering-meshes.spec.ts          00:27       10       10        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/scatter-chart/empty-status.s      00:04        2        2        -        -        - │
  │    pec.ts                                                                                      │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/scatter-chart/errors.spec.ts      00:02        1        1        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✖  charts/scatter-chart/gestures.spec.      00:06        2        1        1        -        - │
  │    ts                                                                                          │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/scatter-chart/loading.spec.t      00:02        1        1        -        -        - │
  │    s                                                                                           │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/scatter-chart/sc-webgl-scatt      00:08        4        4        -        -        - │
  │    er-chart.spec.ts                                                                            │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/scatter-chart/threshold-colo      00:09        6        6        -        -        - │
  │    ration.spec.ts                                                                              │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/scatter-chart/tooltip.spec.t      00:04        2        2        -        -        - │
  │    s                                                                                           │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/status-timeline/empty-status      00:04        2        2        -        -        - │
  │    .spec.ts                                                                                    │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/status-timeline/errors.spec.      00:03        2        2        -        -        - │
  │    ts                                                                                          │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✖  charts/status-timeline/gestures.spe      00:07        2        1        1        -        - │
  │    c.ts                                                                                        │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/status-timeline/loading.spec      00:02        1        1        -        -        - │
  │    .ts                                                                                         │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/status-timeline/status-timel      00:32       17       17        -        -        - │
  │    ine.spec.ts                                                                                 │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/status-timeline/threshold-co      00:11        7        7        -        -        - │
  │    loration.spec.ts                                                                            │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/status-timeline/threshold-le      00:05        3        3        -        -        - │
  │    gend.spec.ts                                                                                │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  charts/status-timeline/tooltip.spec      00:03        1        1        -        -        - │
  │    .ts                                                                                         │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  expandable-input.spec.ts                 00:01        1        1        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  kpi/kpi.spec.ts                          00:07        5        5        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  size-provider.spec.ts                    00:07        6        6        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  status-grid/status-grid.spec.ts          00:16       14       13        -        1        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  table/table.spec.ts                      00:10        9        9        -        -        - │
  ├────────────────────────────────────────────────────────────────────────────────────────────────┤
  │ ✔  webglContext.spec.ts                     00:14        6        6        -        -        - │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘
    ✖  6 of 39 failed (15%)                     06:39      195      175       11        9        -

@ashika01 ashika01 merged commit 254648e into main Jul 1, 2021
@ashika01 ashika01 deleted the feature/cypress branch July 1, 2021 17:54
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.

3 participants