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.

Sign up for GitHub

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

Jump to bottom

Add test framework #15

Merged
merged 28 commits into from
Jun 6, 2024
Merged

Add test framework #15

merged 28 commits into from
Jun 6, 2024

Conversation

idky137
Copy link
Contributor

@idky137 idky137 commented May 3, 2024 •
edited
Loading

Adds test code to allow integration tests to be run without a manual launch of zcashd and lightwalletd (required for adding integration tests to CI). Restructures Nym Code and removes feature gate.

  • uses Zingolib's RegtestManager to launch Lightwalletd and Zcashd in regtest mode for integration tests.
  • uses Zingolib's ClientBuilder to create a test wallet.
  • adds proxy launcher for use in testing. Upgrades zingoproxyd using this.
  • adds test manager to launch RegtestManager and Zingo-Proxy test server.
  • simplifies feature use.
  • (moves Nym enhanced CompactTxStreamerServer service RPC code to zingo-rpc::rpc::nymservice)

Closes #11

Note: CI will be added/fixed in a separate PR.

idky137 added 23 commits April 23, 2024 17:58
@idky137
added zingo-testutils crate (empty)
8f32859
@idky137
added regtest conf
43b8a0d
@idky137
Merge branch 'dev' of github.com:zingolabs/zingo-proxy into add_test_…
bd6a1b2 
…framework
@idky137
added testutils, added walletrpc for testing reorg
992f52d
@idky137
fixed build errors
402e6f0
@idky137
moved proxy spawner to zingo-proxyd, rewrote zingoproxyd.rs to use ne…
574be64 
…w spawner
@idky137
zingoproxyd runs with new changes
980755c
@idky137
small fixes
9850ce8
@idky137
small fixes
820ef4d
@idky137
updated readme
9965a9e
@idky137
updated readme
a070b00
@idky137
test running with default port values
6928567
@idky137
tests running
0fe6d05
@idky137
test framework in place!
4c633d6
@idky137
renamed zingo-testutils for dependancy conflict, updated readme
7f9a310
@idky137
added second test, added grpc connector
9cc86f0
@idky137
fixed nym conf path
9d0f625
@idky137
fixed data leak from test config data
9f2d82c
@idky137
runing multiple tests with cargo tests fails due to ctrlc handler
54ae9d2
@idky137
fixed data leak
ad41896
@idky137
fixed nym_poc feature
ea58c5a
@idky137
Merge branch 'dev' of github.com:zingolabs/zingo-proxy into add_test_…
30e9881 
…framework
@idky137
updated readme
fb147fb
@idky137 idky137 marked this pull request as ready for review May 13, 2024 16:33
This was referenced May 13, 2024
Closed
Merged
@idky137 idky137 mentioned this pull request May 24, 2024
Merged
AloeareV
AloeareV approved these changes Jun 5, 2024
View reviewed changes
Copy link
Contributor

@AloeareV AloeareV left a comment

Choose a reason for hiding this comment

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

Other than the while vs select! issue we've discussed, this all looks awesome!

@idky137 idky137 merged commit b119bf9 into zingolabs:dev Jun 6, 2024
2 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AloeareV AloeareV AloeareV approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Test-Environment Builder
2 participants
@idky137 @AloeareV