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

feat: increment unit tests coverage #1044

Merged
merged 71 commits into from
Nov 17, 2022

Conversation

MonikaCat
Copy link
Contributor

@MonikaCat MonikaCat commented Nov 8, 2022

Description

Closes: #XXXX
BDU-580


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • ran linting
  • wrote tests where necessary
  • included the correct type prefix in the PR title
  • targeted the correct branch
  • provided a link to the relevant issue or specification
  • reviewed "Files changed" and left comments if necessary
  • confirmed all CI checks have passed
  • added an entry to the CHANGELOG.md file

@MonikaCat MonikaCat requested a review from teamchong November 8, 2022 10:28
Copy link
Contributor

@teamchong teamchong left a comment

Choose a reason for hiding this comment

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

thanks, @MonikaCat, look good to me. but since the folder structure has changed a lot, I will patch the changes to my PR tonight, and keep this one open.

@teamchong teamchong mentioned this pull request Nov 9, 2022
8 tasks
@teamchong teamchong self-requested a review November 9, 2022 06:40
@huichiaotsou huichiaotsou merged commit a13f442 into base Nov 17, 2022
@huichiaotsou huichiaotsou deleted the BDU-580-increment-unit-tests-coverage branch November 17, 2022 02:47
teamchong added a commit that referenced this pull request Nov 22, 2022
## Description

* Include new unit tests of
#1044
* Move duplicated components / screens into `ui` package
* Update web socket connection, try to fix
#1045
* Various bug fixes
* Setup docker images

### Author Checklist

*All items are required. Please add a note to the item if the item is
not applicable and
please add links to any relevant follow-up issues.*

I have...

- [X] ran linting
- [X] wrote tests where necessary
- [X] included the correct [type
prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json)
in the PR title
- [X] targeted the correct branch
- [X] provided a link to the relevant issue or specification
- [X] reviewed "Files changed" and left comments if necessary
- [X] confirmed all CI checks have passed
- [X] added an entry to the `CHANGELOG.md` file

Co-authored-by: Riccardo <[email protected]>
Co-authored-by: Magic Cat <[email protected]>
Co-authored-by: Ellie <[email protected]>
ankurdotb pushed a commit to cheqd/big-dipper-2.0-cosmos that referenced this pull request Dec 22, 2022
Closes: #XXXX
[BDU-580](https://forbole.atlassian.net/browse/BDU-580)

<!-- Add a description of the changes that this PR introduces and the
files that
are the most critical to review. -->

---

*All items are required. Please add a note to the item if the item is
not applicable and
please add links to any relevant follow up issues.*

I have...

- [x] ran linting
- [x] wrote tests where necessary
- [x] included the correct [type
prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json)
in the PR title
- [x] targeted the correct branch
- [x] provided a link to the relevant issue or specification
- [x] reviewed "Files changed" and left comments if necessary
- [x] confirmed all CI checks have passed
- [x] added an entry to the `CHANGELOG.md` file
ankurdotb pushed a commit to cheqd/big-dipper-2.0-cosmos that referenced this pull request Dec 22, 2022
Closes: #XXXX
[BDU-580](https://forbole.atlassian.net/browse/BDU-580)

<!-- Add a description of the changes that this PR introduces and the
files that
are the most critical to review. -->

---

*All items are required. Please add a note to the item if the item is
not applicable and
please add links to any relevant follow up issues.*

I have...

- [x] ran linting
- [x] wrote tests where necessary
- [x] included the correct [type
prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json)
in the PR title
- [x] targeted the correct branch
- [x] provided a link to the relevant issue or specification
- [x] reviewed "Files changed" and left comments if necessary
- [x] confirmed all CI checks have passed
- [x] added an entry to the `CHANGELOG.md` file
ankurdotb added a commit to cheqd/big-dipper-2.0-cosmos that referenced this pull request Dec 22, 2022
* Change testnet-5 parameters

* Run graphql codegen and disable generation for desmos profiles

* feat: increment unit tests coverage  (forbole#1044)

Closes: #XXXX
[BDU-580](https://forbole.atlassian.net/browse/BDU-580)

<!-- Add a description of the changes that this PR introduces and the
files that
are the most critical to review. -->

---

*All items are required. Please add a note to the item if the item is
not applicable and
please add links to any relevant follow up issues.*

I have...

- [x] ran linting
- [x] wrote tests where necessary
- [x] included the correct [type
prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json)
in the PR title
- [x] targeted the correct branch
- [x] provided a link to the relevant issue or specification
- [x] reviewed "Files changed" and left comments if necessary
- [x] confirmed all CI checks have passed
- [x] added an entry to the `CHANGELOG.md` file

* Update jest snapshots

* Update pull-request.yml

Co-authored-by: Magic Cat <[email protected]>
Co-authored-by: Ankur Banerjee <[email protected]>
boosik pushed a commit to Carina-labs/big-dipper-2.0-cosmos that referenced this pull request Feb 13, 2023
## Description

Closes: #XXXX
[BDU-580](https://forbole.atlassian.net/browse/BDU-580)

<!-- Add a description of the changes that this PR introduces and the
files that
are the most critical to review. -->

---

### Author Checklist

*All items are required. Please add a note to the item if the item is
not applicable and
please add links to any relevant follow up issues.*

I have...

- [x] ran linting
- [x] wrote tests where necessary
- [x] included the correct [type
prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json)
in the PR title
- [x] targeted the correct branch
- [x] provided a link to the relevant issue or specification
- [x] reviewed "Files changed" and left comments if necessary
- [x] confirmed all CI checks have passed
- [x] added an entry to the `CHANGELOG.md` file
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