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

test: refactor OptionCoder tests #1006

Conversation

danielbate
Copy link
Member

@danielbate danielbate commented May 17, 2023

Related to #753

This PR will:

  • Introduce a seperate test file for the service under test
  • Seperate test cases for encode and decode
  • Remove redundant tests to more clearly report on code coverage

@danielbate danielbate requested review from arboleya, camsjams, Torres-ssf, Dhaiwat10 and a team May 17, 2023 11:29
@danielbate danielbate self-assigned this May 17, 2023
@github-actions
Copy link
Contributor

github-actions bot commented May 17, 2023

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
95.48% (+0.11% 🔼)
5261/5510
🟢 Branches
84.9% (+0.39% 🔼)
866/1020
🟢 Functions
87.18% (+0.09% 🔼)
959/1100
🟢 Lines
95.48% (+0.11% 🔼)
5032/5270

Test suite run success

921 tests passing in 153 suites.

Report generated by 🧪jest coverage report action from 550ab68

@danielbate danielbate merged commit 41faf45 into db/test/refactor-abi-coder-coders-tests May 17, 2023
@danielbate danielbate deleted the db/test/refactor-option-coder-tests branch May 17, 2023 12:38
danielbate added a commit that referenced this pull request May 17, 2023
* test: add individual unit test files for coders

* test: implemented abi coder test utils

* Revert "test: add individual unit test files for coders"

This reverts commit 9d77f9d.

* test: refactor `NumberCoder` tests (#949)

* test: refactor and increase verbosity of number coder tests

* chore: changeset

* chore: force rebuild

---------

Co-authored-by: danielbate <--global>

* test: refactor `B256Coder` tests (#946)

* test: refactor and increase verbosity of b256 coder tests

* chore: changeset

* test: add test for missing throw condition

---------

Co-authored-by: danielbate <--global>

* test: refactor `B512Coder` tests (#945)

* test: refactor and increase verbosity of b512 coder tests

* chore: changeset

* chore: linting

* test: add test case for missing throw condition in b512 coder

* test: correct imported class name in b512 coder test

Co-authored-by: Sérgio Torres <[email protected]>

---------

Co-authored-by: danielbate <--global>
Co-authored-by: Sérgio Torres <[email protected]>

* test: refactor `BooleanCoder` tests (#944)

* test: refactor and increase verbosity of boolean coder tests

* chore: changeset

* test: add missing test coverage to boolean coder throw conditions

---------

Co-authored-by: danielbate <--global>

* test: refactor `ByteCoder` tests (#943)

* test: refactored byte coder tests

* chore: changeset

---------

Co-authored-by: danielbate <--global>

* test: refactor `TupleCoder` tests (#939)

* test: refactor and increase verbosity of tuple coder tests

* chore: changeset

---------

Co-authored-by: danielbate <--global>

* test: refactor `StructCoder` tests (#940)

* test: refactor and increase verbosity of struct test

* test: fix test number inputs in struct coder test

* chore: changeset

* test: use template literals for struct name in struct coder test

Co-authored-by: Anderson Arboleya <[email protected]>

---------

Co-authored-by: danielbate <--global>
Co-authored-by: Anderson Arboleya <[email protected]>

* test: refactor `ArrayCoder` tests (#937)

* test: refactor and increase verbosity of array coder tests

* chore: changeset

* test: remove redundant describe block from array coder test

* test: add missing throw condition testcase to array coder

---------

Co-authored-by: danielbate <--global>

* test: refactor `U64Coder` tests (#941)

* test: refactor and increase verbosity of u64 coder tests

* test: remove redundant imports and comments

* chore: changeset

* chore: force rebuild

---------

Co-authored-by: danielbate <--global>

* test: refactor `VecCoder` tests (#936)

* test: refactor and increase verbosity of vec test

* chore: changeset

* test: add missing test cases for encoding non array inputs in vec coder

---------

Co-authored-by: danielbate <--global>

* test: refactor `StringCoder` tests (#972)

* test: refactor and increase verbosity of string coder tests

* test: remove redundant import

* chore: changeset

* chore: force rebuild

* test: remove commented out tests from string coder

---------

Co-authored-by: danielbate <--global>

* test: refactor `EnumCoder` tests (#938)

* test: refactor and increase verbosity of enum coder tests

* chore: changeset

* test: introduce test cases for native enums

* test: add feature test in abi coder for native enum

* test: remove abi coder test case for native enum as redundant

---------

Co-authored-by: danielbate <--global>

* test: abstract coder tests

* refactor: remove coders mass and snapshot tests

* chore: remove redundant import in ABI coder

* chore: linting

* chore: linting

* test: remove redundant enum coder tests

* test: refactor `OptionCoder` tests (#1006)

---------

Co-authored-by: danielbate <--global>
Co-authored-by: Sérgio Torres <[email protected]>
Co-authored-by: Anderson Arboleya <[email protected]>
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