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 TupleCoder tests #939

Conversation

danielbate
Copy link
Member

Related to #753

This PR will:

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

@danielbate danielbate requested a review from a team April 28, 2023 11:43
@danielbate danielbate self-assigned this Apr 28, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Apr 28, 2023

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 95.11% 4703/4945
🟢 Branches 83.97% 833/992
🟢 Functions 92.43% 903/977
🟢 Lines 95.13% 4496/4726

Test suite run success

827 tests passing in 127 suites.

Report generated by 🧪jest coverage report action from 9e3618f

Copy link
Member

@arboleya arboleya left a comment

Choose a reason for hiding this comment

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

💯 ⚡

----------------|---------|----------|---------|---------|-------------------------------------------------
File            | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
----------------|---------|----------|---------|---------|-------------------------------------------------
  tuple.ts      |     100 |      100 |     100 |     100 |

@danielbate danielbate merged commit 58f7898 into db/test/refactor-abi-coder-coders-tests May 2, 2023
@danielbate danielbate deleted the db/test/refactor-tuple-coder-tests branch May 2, 2023 14:32
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]>
danielbate added a commit that referenced this pull request May 18, 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>

* feat: add throw to string coder when encoding value with length mismatch

* test: add unit test for string cdoer encoder throw conditon

* chore: changeset

* chore: force rebuild

---------

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