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

Update unit tests for PSP-7039 #3576

Merged
merged 2 commits into from
Nov 7, 2023
Merged

Conversation

asanchezr
Copy link
Collaborator

No description provided.

@asanchezr asanchezr added the unit-tests Adding testing coverage label Nov 7, 2023
@asanchezr asanchezr self-assigned this Nov 7, 2023
Copy link
Contributor

github-actions bot commented Nov 7, 2023

✅ No secrets were detected in the code.

Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #3576 (c0eb999) into dev (3839b91) will increase coverage by 4.98%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #3576      +/-   ##
==========================================
+ Coverage   69.56%   74.55%   +4.98%     
==========================================
  Files        1370      898     -472     
  Lines       33716    18575   -15141     
  Branches     6270     5244    -1026     
==========================================
- Hits        23454    13848    -9606     
+ Misses      10011     4727    -5284     
+ Partials      251        0     -251     
Flag Coverage Δ
unittests 74.55% <ø> (+4.98%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...Bar/property/tabs/takes/detail/TakesDetailView.tsx 92.85% <ø> (ø)
...Bar/property/tabs/takes/update/TakesUpdateForm.tsx 100.00% <ø> (ø)

... and 473 files with indirect coverage changes

Copy link
Contributor

github-actions bot commented Nov 7, 2023

✅ No secrets were detected in the code.

@asanchezr asanchezr merged commit e1d63fb into bcgov:dev Nov 7, 2023
9 checks passed
@asanchezr asanchezr deleted the psp-7039-tests branch November 7, 2023 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
unit-tests Adding testing coverage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants