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

Mk 1988 add more beacon wallet tests #1991

Merged
merged 16 commits into from
Oct 27, 2022

Conversation

michaelkernaghan
Copy link
Collaborator

add more tests for covering beacon wallet

CLOSE #1988

@netlify
Copy link

netlify bot commented Oct 5, 2022

Deploy Preview for taquito-test-dapp canceled.

Name Link
🔨 Latest commit 778a05c
🔍 Latest deploy log https://app.netlify.com/sites/taquito-test-dapp/deploys/635aad3b5a5dd7000938d714

@michaelkernaghan michaelkernaghan self-assigned this Oct 5, 2022
@michaelkernaghan michaelkernaghan added the testing🧪 Work related to testing, manual or automated label Oct 5, 2022
@github-actions
Copy link

github-actions bot commented Oct 6, 2022

A new deploy preview is available on Netlify at https://72932f3--tezostaquito.netlify.app

@github-actions
Copy link

github-actions bot commented Oct 6, 2022

New packages have been deployed to the preview repository at https://npm.preview.tezostaquito.io/.

Published packages:

npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/
npm i @taquito/[email protected] --registry https://npm.preview.tezostaquito.io/

@codecov-commenter
Copy link

Codecov Report

Base: 81.62% // Head: 81.72% // Increases project coverage by +0.09% 🎉

Coverage data is based on head (62811af) compared to base (11c55d4).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1991      +/-   ##
==========================================
+ Coverage   81.62%   81.72%   +0.09%     
==========================================
  Files         222      222              
  Lines       12371    12371              
  Branches     2936     2936              
==========================================
+ Hits        10098    10110      +12     
+ Misses       2272     2260      -12     
  Partials        1        1              
Impacted Files Coverage Δ
...taquito-beacon-wallet/src/taquito-beacon-wallet.ts 49.35% <0.00%> (+15.58%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@roxaneletourneau roxaneletourneau merged commit 571f5da into master Oct 27, 2022
@roxaneletourneau roxaneletourneau deleted the mk-1988-beacon-wallet-mocks branch October 27, 2022 17:37
egarson pushed a commit that referenced this pull request Nov 16, 2022
* edits

* more tests

* added tests

* test(beacon wallet): add unit tests for bacon wallet

* test: fix PR

* test(fix the pr): fix the PR

* test(fix package problem): fix package problem

* test: fix merge conflict

* changes per review

* add package locks
egarson added a commit that referenced this pull request Nov 16, 2022
* Re-work leading section

* Compress SemVer discussion and break into sections

* Incorporate feedback as solicited on hackmd.io/aYu2xvrFRbyiYtvgMpeBtw

* Mk 1988 add more beacon wallet tests  (#1991)

* edits

* more tests

* added tests

* test(beacon wallet): add unit tests for bacon wallet

* test: fix PR

* test(fix the pr): fix the PR

* test(fix package problem): fix package problem

* test: fix merge conflict

* changes per review

* add package locks

* Multisig documentation (#2033)

* Added documentation for multisig and reordered the doc chapters in the sidebar

* Pushing uncommitted changes

* Added mention of MANAGER_LAMBDA

* Corrected error in text

* Corrected error in doc

* Corrected typo in yarn.lock file

* Updated boilerplate section of the doc

* Removed conflicting yarn.lock files

* Updated .gitignore in /website

* fix: upgrade @svgr/webpack from 6.3.1 to 6.4.0

Snyk has created this PR to upgrade @svgr/webpack from 6.3.1 to 6.4.0.

See this package in npm:
https://www.npmjs.com/package/@svgr/webpack

See this project in Snyk:
https://app.snyk.io/org/roxane/project/ad4c74dc-f97a-4a6b-8c68-4c305b03e490?utm_source=github&utm_medium=referral&page=upgrade-pr

* removed tests with references to timelock feature (#2083)

* 1630 ballot operation support in contract API (#2050)

* added estimate and contract ballot operation support

* added rpc contract provider for ballot op

* updated implementation and added docs

* added property in OperationContentsAndResultBallot and added unit test

* remove metadata property from ballot

* addressed PR comments

* updated yarn and package-lock

* cleaned up comments and switched out async for promise in op emitter

* removed unnecessary line break

* 660 support customize parser options (#2061)

* feat: exposed setParserProvider configuration through TezosToolkit

re #660

* test: added integration tests using setParserProvider with TezosToolkit

re #660

* docs: updated starting production server commands with taquito-test-dapp readme.md

* test: removed unnecessory console.log

re #660

* chore: update vite to latest version

* docs: fixed some typos in doc to address comments

re #660

* contract-security-non-existent-KT-address: use rpc (#2091)

* revert changes from ballot operation

* Bump @airgap/beacon-dapp to v 3.3.0

* build(deps): bump loader-utils from 1.4.0 to 1.4.1 in /packages/taquito

Bumps [loader-utils](https://github.com/webpack/loader-utils) from 1.4.0 to 1.4.1.
- [Release notes](https://github.com/webpack/loader-utils/releases)
- [Changelog](https://github.com/webpack/loader-utils/blob/v1.4.1/CHANGELOG.md)
- [Commits](webpack/loader-utils@v1.4.0...v1.4.1)

---
updated-dependencies:
- dependency-name: loader-utils
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

* build(deps): bump loader-utils from 1.4.0 to 1.4.1 in /website

Bumps [loader-utils](https://github.com/webpack/loader-utils) from 1.4.0 to 1.4.1.
- [Release notes](https://github.com/webpack/loader-utils/releases)
- [Changelog](https://github.com/webpack/loader-utils/blob/v1.4.1/CHANGELOG.md)
- [Commits](webpack/loader-utils@v1.4.0...v1.4.1)

---
updated-dependencies:
- dependency-name: loader-utils
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

* chore(releng) bump version to 14.1.0-beta-RC.0

* Commit package-lock file for taquito-test-dapp

* Fix unit test related to version in ContractsLibrary

* Fix unit test related to version in ContractsLibrary

* Update taquito-test-dapp with taquito version 14.1.0-beta-RC.0

* chore(releng) bump version to 14.1.0

* Created version 14.1.0 on the Taquito Website

* 1630 ballot (#2107)

* added estimate and contract ballot operation support

* added rpc contract provider for ballot op

* updated implementation and added docs

* added property in OperationContentsAndResultBallot and added unit test

* remove metadata property from ballot

* addressed PR comments

* updated yarn and package-lock

* cleaned up comments and switched out async for promise in op emitter

* removed unnecessary line break

* renamed ballotenum to BallotVote

* update BallotEnum to BallotVote

* Add proposals operation support (#2112)

* add proposals operation

* added tests for proposals operation

* updated documentation to combine ballot and proposals op

* remove yarn lock files

* addressed PR comments and updated docs

* Fix typo (repeat of "breaking changes") in Release Timing section

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Michael Kernaghan <[email protected]>
Co-authored-by: Claude Barde <[email protected]>
Co-authored-by: snyk-bot <[email protected]>
Co-authored-by: Davis Sawali <[email protected]>
Co-authored-by: hui-an-yang <[email protected]>
Co-authored-by: Arvid Jakobsson <[email protected]>
Co-authored-by: roxaneletourneau <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing🧪 Work related to testing, manual or automated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

extend test coverage for beacon-wallet package using mocks
3 participants