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

STCOR-729 upgrade to react v18 #1324

Merged
merged 11 commits into from
Jul 17, 2023
Merged

STCOR-729 upgrade to react v18 #1324

merged 11 commits into from
Jul 17, 2023

Conversation

zburke
Copy link
Member

@zburke zburke commented Jul 12, 2023

Upgrade test infrastructure to support React v18. Jettison @bigtest/react.

Refs STCOR-729

@zburke zburke requested a review from JohnC-80 July 12, 2023 21:17
@github-actions
Copy link

github-actions bot commented Jul 12, 2023

Jest Unit Test Statistics

64 tests  ±0   64 ✔️ ±0   19s ⏱️ +3s
10 suites ±0     0 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 466232e. ± Comparison against base commit 406ad6f.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 12, 2023

BigTest Unit Test Statistics

    1 files  ±0      1 suites  ±0   11s ⏱️ ±0s
271 tests ±0  266 ✔️ ±0  5 💤 ±0  0 ±0 
274 runs  ±0  269 ✔️ ±0  5 💤 ±0  0 ±0 

Results for commit 466232e. ± Comparison against base commit 406ad6f.

This pull request removes 273 and adds 271 tests. Note that renamed tests count towards both.
      equal to check email label in english translation
      equal to check email precautions label in english translation
      equal to sent email precautions label in english translation
Chrome_114_0_0_0_(Linux_x86_64).About ‑ About viewing the about page has one installed app
Chrome_114_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Passing a string to the tag-prop Should render an AppIcon with a HTML tag of "div"
Chrome_114_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Passing a string using the children-prop Should render an AppIcon with a label
Chrome_114_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Rendering an AppIcon using Stripes-context Should render an <img>
Chrome_114_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Rendering an AppIcon using Stripes-context Should render an img with an alt-attribute
Chrome_114_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Rendering an AppIcon using an icon-object Should render an <img>
Chrome_114_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Rendering an AppIcon using an icon-object Should render an img with an alt-attribute
…
Chrome_113_0_0_0_(Linux_x86_64).About ‑ About viewing the about page has one installed app
Chrome_113_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Passing a string to the tag-prop Should render an AppIcon with a HTML tag of "div"
Chrome_113_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Passing a string using the children-prop Should render an AppIcon with a label
Chrome_113_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Rendering an AppIcon using Stripes-context Should render an <img>
Chrome_113_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Rendering an AppIcon using Stripes-context Should render an img with an alt-attribute
Chrome_113_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Rendering an AppIcon using an icon-object Should render an <img>
Chrome_113_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Rendering an AppIcon using an icon-object Should render an img with an alt-attribute
Chrome_113_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Rendering an AppIcon using an icon-object Should render with a className of "My className"
Chrome_113_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Size tests Passing a size of "large" Should render an icon into a large-sized container
Chrome_113_0_0_0_(Linux_x86_64).AppIcon ‑ AppIcon Size tests Passing a size of "medium" Should render an icon into a medium-sized container
…
This pull request removes 5 skipped tests and adds 5 skipped tests. Note that renamed tests count towards both.
Chrome_114_0_0_0_(Linux_x86_64).PasswordValidationField ‑ PasswordValidationField with an invalid password shows a validation error
Chrome_114_0_0_0_(Linux_x86_64).useCustomFields hook ‑ useCustomFields hook requests for existing custom fields should have custom fields
Chrome_114_0_0_0_(Linux_x86_64).useCustomFields hook ‑ useCustomFields hook requests for non-existing custom fields should have error
Chrome_114_0_0_0_(Linux_x86_64).useCustomFields hook ‑ useCustomFields hook requests for version-compatible custom fields should have custom fields
Chrome_114_0_0_0_(Linux_x86_64).useCustomFields hook ‑ useCustomFields hook requests for version-incompatible custom fields should have error
Chrome_113_0_0_0_(Linux_x86_64).PasswordValidationField ‑ PasswordValidationField with an invalid password shows a validation error
Chrome_113_0_0_0_(Linux_x86_64).useCustomFields hook ‑ useCustomFields hook requests for existing custom fields should have custom fields
Chrome_113_0_0_0_(Linux_x86_64).useCustomFields hook ‑ useCustomFields hook requests for non-existing custom fields should have error
Chrome_113_0_0_0_(Linux_x86_64).useCustomFields hook ‑ useCustomFields hook requests for version-compatible custom fields should have custom fields
Chrome_113_0_0_0_(Linux_x86_64).useCustomFields hook ‑ useCustomFields hook requests for version-incompatible custom fields should have error

♻️ This comment has been updated with latest results.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@zburke zburke merged commit d1b9efc into master Jul 17, 2023
@zburke zburke deleted the STCOR-729 branch July 17, 2023 23:25
@svc-github-gopac-jenkins
Copy link

svc-github-gopac-jenkins commented Jul 17, 2023

❌ Build of commit 466232e failed. See details at http://10.23.33.46:8080/job/stripes-core/74/

@svc-github-gopac-jenkins
Copy link

svc-github-gopac-jenkins commented Jul 18, 2023

❌ Build of commit 466232e failed. See details at http://10.23.33.46:8080/job/stripes-core/76/

zburke added a commit that referenced this pull request Jul 19, 2023
Missed a few test-related deps in #1324.

Refs STCOR-729
zburke added a commit that referenced this pull request Jul 27, 2023
Missed a few test-related deps in #1324. Additionally, convert from
local deps/config to importing from jest-config-stripes.

Refs STCOR-729
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