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

Updates and fixes to system tests #1449

Merged
merged 3 commits into from
Jun 14, 2023
Merged

Conversation

alpe
Copy link
Contributor

@alpe alpe commented Jun 14, 2023

Better CLI test framework

@alpe alpe requested a review from pinosu June 14, 2023 07:26
@codecov
Copy link

codecov bot commented Jun 14, 2023

Codecov Report

Merging #1449 (ca75b8e) into 1362_tests_update (ce088d9) will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@                  Coverage Diff                  @@
##           1362_tests_update    #1449      +/-   ##
=====================================================
- Coverage              60.03%   60.00%   -0.03%     
=====================================================
  Files                     61       61              
  Lines                   7571     7571              
=====================================================
- Hits                    4545     4543       -2     
- Misses                  2690     2691       +1     
- Partials                 336      337       +1     

see 1 file with indirect coverage changes

Copy link
Contributor

@pinosu pinosu left a comment

Choose a reason for hiding this comment

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

Great work! 💯

@alpe alpe merged commit 2f5888c into 1362_tests_update Jun 14, 2023
@alpe alpe deleted the 1362_tests_update2 branch June 14, 2023 12:14
alpe added a commit that referenced this pull request Jun 14, 2023
* Add tests

* Add test-system to CI

* Fix path

* Remove store artifact steps

* Add small fixes

* Add stake/unstake tests

* Add unsafe-reset-all extention + system test

* Replace ustake with stake

* Add more tests + fix bug in cli

* Fix comments and add multi contract system test

* Updates and fixes to system tests (#1449)

* Updates

* Minor cleanup

* Make tests pass

---------

Co-authored-by: Alexander Peters <alpe@users.noreply.github.com>
alpe added a commit that referenced this pull request Jun 15, 2023
* Start system tests

* Go mod tidy

* Add system tests (#1411)

* Add tests

* Add test-system to CI

* Fix path

* Remove store artifact steps

* Add small fixes

* Add stake/unstake tests

* Add unsafe-reset-all extention + system test

* Replace ustake with stake

* Add more tests + fix bug in cli

* Fix comments and add multi contract system test

* Updates and fixes to system tests (#1449)

* Updates

* Minor cleanup

* Make tests pass

---------

Co-authored-by: Alexander Peters <alpe@users.noreply.github.com>

* Fix Makefile to return exit code for system tests (#1450)

* Abort on error results

---------

Co-authored-by: pinosu <95283998+pinosu@users.noreply.github.com>
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.

2 participants