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

Add tests to Address Book #1657

Merged
merged 3 commits into from
Sep 12, 2023
Merged

Add tests to Address Book #1657

merged 3 commits into from
Sep 12, 2023

Conversation

buberdds
Copy link
Contributor

No description provided.

@cloudflare-pages
Copy link

cloudflare-pages bot commented Sep 12, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 05a0460
Status: ✅  Deploy successful!
Preview URL: https://a448ee22.oasis-wallet.pages.dev
Branch Preview URL: https://mz-addrtests.oasis-wallet.pages.dev

View logs

@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Merging #1657 (17afdc4) into master (9932308) will increase coverage by 1.19%.
The diff coverage is 100.00%.

❗ Current head 17afdc4 differs from pull request most recent head 05a0460. Consider uploading reports for the commit 05a0460 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1657      +/-   ##
==========================================
+ Coverage   81.31%   82.50%   +1.19%     
==========================================
  Files         171      171              
  Lines        4345     4345              
  Branches      777      777              
==========================================
+ Hits         3533     3585      +52     
+ Misses        812      760      -52     
Flag Coverage Δ
cypress 49.64% <0.00%> (ø)
jest 77.74% <100.00%> (+1.18%) ⬆️

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

Files Changed Coverage Δ
...mponents/Toolbar/Features/SettingsButton/index.tsx 91.66% <ø> (ø)
...pp/components/Toolbar/Features/Account/Account.tsx 92.10% <100.00%> (+2.63%) ⬆️

... and 7 files with indirect coverage changes

@buberdds buberdds force-pushed the mz/addrTests branch 2 times, most recently from 7556486 to 14e90bf Compare September 12, 2023 09:49
@buberdds buberdds changed the title Add tests to Address Book components Add tests to Address Book Sep 12, 2023
renderComponent(store)
await userEvent.click(screen.getByRole('button', { name: 'toolbar.contacts.save' }))
expect(screen.getAllByText('toolbar.contacts.validation.required').length).toEqual(2)
expect(onSave).not.toHaveBeenCalled()
Copy link
Member

Choose a reason for hiding this comment

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

👍

.fill('oasis1qq2vzcvxn0js5unsch5me2xz4kr43vcasv0d5eq4')
await tab2.getByRole('button', { name: 'Save' }).click()
await expect(tab2.getByTestId('account-choice')).toHaveCount(1)
await expect(page.getByTestId('account-choice')).toHaveCount(1)
Copy link
Member

Choose a reason for hiding this comment

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

(potential additional e2e: expect not to sync if user changes name and cancels)

@buberdds buberdds merged commit dae6efe into master Sep 12, 2023
10 checks passed
@buberdds buberdds deleted the mz/addrTests branch September 12, 2023 18:44
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.

None yet

2 participants