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

177343947 add offline storage tests #235

Merged
merged 2 commits into from
Apr 7, 2021

Conversation

knowuh
Copy link
Contributor

@knowuh knowuh commented Apr 7, 2021

Just adding a few tests.

Also moved some storage-facade related type declarations to the top of the file.

[#177343947] Adding spec tests for storage facade
https://www.pivotaltracker.com/story/show/177343947
[#177343947] Adding spec tests for storage facade
https://www.pivotaltracker.com/story/show/177343947
@codecov
Copy link

codecov bot commented Apr 7, 2021

Codecov Report

Merging #235 (03188d1) into offline-mode-dev (0d77921) will increase coverage by 0.50%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                 Coverage Diff                  @@
##           offline-mode-dev     #235      +/-   ##
====================================================
+ Coverage             74.04%   74.54%   +0.50%     
====================================================
  Files                    92       92              
  Lines                  3471     3473       +2     
  Branches                757      757              
====================================================
+ Hits                   2570     2589      +19     
+ Misses                  895      877      -18     
- Partials                  6        7       +1     
Flag Coverage Δ
cypress 59.84% <100.00%> (-0.01%) ⬇️
jest 61.50% <100.00%> (+0.94%) ⬆️

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

Impacted Files Coverage Δ
src/portal-api.ts 38.46% <ø> (ø)
src/storage/storage-facade.ts 28.21% <100.00%> (+8.13%) ⬆️
...components/activity-page/activity-page-content.tsx 95.76% <0.00%> (-0.85%) ⬇️

Copy link
Member

@scytacki scytacki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@scytacki scytacki merged commit d2ff9cb into offline-mode-dev Apr 7, 2021
@scytacki scytacki deleted the 177343947-add-offline-storage-tests branch April 7, 2021 21:00
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.

2 participants