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

provide various missing unit tests #1474

Merged
merged 4 commits into from
Jun 17, 2024
Merged

provide various missing unit tests #1474

merged 4 commits into from
Jun 17, 2024

Conversation

zburke
Copy link
Member

@zburke zburke commented May 20, 2024

Provide tests for hideEmail, loginServices, okapiActions, okapiReducer, and processBadResponse.

Tiny refactor of some loginServices functions to:

  • consistently test for success, else handle an error
  • consistently test for success in responses by checking response.ok rather than response.status. consistency is nice. it's predictable.

Provide tests for hideEmail, loginServices, okapiActions, okapiReducer,
and processBadResponse.

Tiny refactor of some loginServices functions to:
* consistently test for success, else handle an error
* consistently test for success in responses by checking `response.ok`
  rather than `response.status`. consistency is nice. it's predictable.
@zburke zburke requested a review from JohnC-80 May 20, 2024 20:31
Copy link

github-actions bot commented May 20, 2024

Jest Unit Test Statistics

    1 files  ±  0    33 suites  +2   47s ⏱️ -1s
258 tests +51  258 ✔️ +51  0 💤 ±0  0 ±0 
259 runs  +52  259 ✔️ +52  0 💤 ±0  0 ±0 

Results for commit a3a63d3. ± Comparison against base commit 142ff27.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 20, 2024

BigTest Unit Test Statistics

    1 files  ±0      1 suites  ±0   10s ⏱️ ±0s
270 tests ±0  264 ✔️ ±0  6 💤 ±0  0 ±0 
273 runs  ±0  267 ✔️ ±0  6 💤 ±0  0 ±0 

Results for commit a3a63d3. ± Comparison against base commit 142ff27.

This pull request removes 5 and adds 3 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_125_0_0_0_(Linux_x86_64).Forgot username form test check email status page tests ‑ Forgot username form test check email status page tests should have the header with an appropriate text content
Chrome_125_0_0_0_(Linux_x86_64).Forgot username form test check email status page tests ‑ Forgot username form test check email status page tests should have the paragraph with an appropriate text content
Chrome_125_0_0_0_(Linux_x86_64).Forgot username form test check email status page tests ‑ Forgot username form test check email status page tests should have the header with an appropriate text content
      equal to check email label in english translation
Chrome_125_0_0_0_(Linux_x86_64).Forgot username form test check email status page tests ‑ Forgot username form test check email status page tests should have the paragraph with an appropriate text content
      equal to check email precautions label in english translation
Chrome_125_0_0_0_(Linux_x86_64).Forgot username form test check email status page tests ‑ Forgot username form test check email status page tests should have the paragraph with an appropriate text content
      equal to sent email precautions label in english translation

♻️ This comment has been updated with latest results.

Copy link

Jest Unit Test Results

  1 files  ± 0   38 suites  +2   46s ⏱️ ±0s
279 tests +51  279 ✅ +51  0 💤 ±0  0 ❌ ±0 
280 runs  +52  280 ✅ +52  0 💤 ±0  0 ❌ ±0 

Results for commit d23abf6. ± Comparison against base commit 9d7cee9.

Copy link

github-actions bot commented May 31, 2024

Jest Unit Test Results

  1 files  ± 0   38 suites  +2   47s ⏱️ ±0s
283 tests +51  283 ✅ +51  0 💤 ±0  0 ❌ ±0 
284 runs  +52  284 ✅ +52  0 💤 ±0  0 ❌ ±0 

Results for commit a9594c6. ± Comparison against base commit e93a5af.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 31, 2024

Bigtest Unit Test Results

193 tests  ±0   188 ✅ ±0   6s ⏱️ ±0s
  1 suites ±0     5 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit a9594c6. ± Comparison against base commit e93a5af.

This pull request removes 5 and adds 3 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_125_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the header with an appropriate text content
Chrome_125_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the paragraph with an appropriate text content
Chrome_125_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the header with an appropriate text content
      equal to check email label in english translation
Chrome_125_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the paragraph with an appropriate text content
      equal to check email precautions label in english translation
Chrome_125_0_0_0_(Linux_x86_64).Forgot username/password status test check email status page tests ‑ Forgot username/password status test check email status page tests should have the paragraph with an appropriate text content
      equal to sent email precautions label in english translation

♻️ This comment has been updated with latest results.

Copy link

sonarcloud bot commented Jun 17, 2024

@zburke zburke merged commit ac49090 into master Jun 17, 2024
26 checks passed
@zburke zburke deleted the testy-mctestface branch June 17, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant