Skip to content

chore: add integ tests for storage field#1011

Open
bombguy wants to merge 6 commits intomainfrom
cshin/storage-field-integ-test
Open

chore: add integ tests for storage field#1011
bombguy wants to merge 6 commits intomainfrom
cshin/storage-field-integ-test

Conversation

@bombguy
Copy link
Copy Markdown
Contributor

@bombguy bombguy commented May 17, 2023

Problem

To add integration test for storage field in generated form

Verification

Manual tests

Automated tests

  • E2E tests added/updated

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

@bombguy bombguy requested a review from a team as a code owner May 17, 2023 20:56
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 17, 2023

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.71%. Comparing base (1fbf44d) to head (2339668).
⚠️ Report is 121 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1011   +/-   ##
=======================================
  Coverage   93.71%   93.71%           
=======================================
  Files         122      122           
  Lines        5428     5428           
  Branches     1620     1620           
=======================================
  Hits         5087     5087           
  Misses        323      323           
  Partials       18       18           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1fbf44d...2339668. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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