-
Notifications
You must be signed in to change notification settings - Fork 11
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/integration tests on actions #64
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…nto feat/integration-tests-on-actions
…nto feat/integration-tests-on-actions
ssantos21
requested changes
Jun 25, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good to me, but there are some issues to be addressed.
ssantos21
added a commit
that referenced
this pull request
Jul 1, 2024
* feat: add integration tests to actions * fix: nodejs dependency installation * fix: path in dependency installation for client * fix: linking of dependency mercurynodejslib * fix: port issue with mercury server * fix: host of mercury server * debug: services on actions * debug: mercury server availability * debug: mercury server through curl * debug: mercury server though curl * debug: running containers * debug: mercury server through exec and curl * debug: mercury server through curl * debug: test with mercurylayer.com * fix: mercurylayer api url * fix: remove mercury logs * feat: add test for iterative self transfer * debug: mercury server through curl * debug: mercury server container * debug: mercury server * debug: list all containers * debug: mercury server through container id * debug: fix mercury server container name * debug: docker logs * debug: increase wait time for service to be ready * debug: sql error * debug: mercury server post request * debug: electrum client * debug: test failure * debug: post request /deposit/init/pod * debug: internal server error for mercury server * fix: expose port for ssl of electrum server * debug: electrum server * debug: bitcoin core * fix: increase wait time to service ready for bitcoin core * fix: command args for bitcoin core * debug: enclave * debug: enclave /get_public_key * fix: add port for enclave to docker-compose * debug: get public key from enclave * debug: use lockbox url of testnet * fix: enclave dockerfile cmd * fix: cmd for enclave server * debug: deposit msg * debug: depositMsg for /deposit/init/pod * debug: statechain_id in pod_deposit * debug: enclave logs * debig: network connectivity between containers * debug: lockbox url * fix: mercury server in docker compose * fix: dockerfile location in docker compose * fix: dockerfile location in mercury server docker compose * fix: lockbox env name * fix: env name for lockbox * feat: add deposit using bitcoin-cli * debug: bitcoin-cli for deposit * fix: bitcoin-cli cmd execution * fix: create wallet before send to address * fix: address in deposit * fix: address in generatetoaddress * fix: add rpcfee field to sendtoaddress cmd * fix: increase rpcfee * fix: add fallbackfee to bitcoin core * fix: generate block after deposit * fix: increase generate block to confirmation target * fix: change withdraw address to regtest * fix: adjust sleep time for services to be ready * debug: wallet database deletion * fix: change wallet name to fix sql error * fix: db lock error * fix: path in removeDatabase fn * fix: databaseFile variable * fix: blockheight from electrum server * fix: getElectrumClient import * debug: db file location * debug: find wallet.db file * fix: removeDb fn * fix: wallet names in tests * fix: lockheight of mercury server * fix: initial blocks generation * fix: path import * fix: path in removeDatabase fn * fix: path import in removeDatabase fn * fix: unit test for iterative self transfer * fix: locktime for mercury server * fix: lockheight_init for mercury server * debug: lockheight_init for mercury server * fix: lh_decrement and lockheight_init * chore: add test for transfer sender after transfer receiver * chore: refactor depositCoin * chore: add test for deposit and transfer * fix: token generation * fix: assert error * fix: typos for wallet name * fix: variable type * chore: add test for interruption of transfer-send in sign first * fix: increase block generation for deposit coin step * fix: generateBlock * fix: wait for coin confirmed * fix: increase block after coin deposit * chore: add test for transferSend signSecond interruption * fix: block unblock port of mercury server for test * debug: network name for mercury server * fix: disconnect network for test * fix: enclose post request of sign into try catch * fix: enclose transfer/sender request in try catch * fix: get_new_x1 fn * fix: error handling for mercury server post request * fix: variable error * fix: handle the case if response undefined * fix: error handling for signFirst and signSecond * fix: assert errors in test * fix: add test for electrum unavailability * chore: add test for transfer receive for electrum unavailability * chore: add test for transfer receive for mercury server unavailability * fix: mercury and electrum unavailability tests * debug: coin deposition * debug: coin deposit verification * debug: listStatecoins * fix: get_new_x1 * fix: add generateBlock if coin status in mempool * fix: generateBlock for coins in mempool * debug: fee rate * debug: coin confirmation * debug: coin status * debug: checkDeposit * fix: typos * debug: network disconnect * debug: fee_rate * debug: blockheight * fix: getElectrumClient error * fix: ElectrumCli import * debug: running all tests * fix: remove generateBlock if coin waiting to be confirmed * fix: replace electrumx with electrs * fix: electrs volume mount * fix: get_new_x1 * fix: add entrypoint file for electrs * fix: add volume to docker compose * fix: electrs configs * fix: add generateBlock if coin stuck in mempool * fix: signSecond test for interruption * fix: error message for electrs unavailability * fix: assertions for server unavailability * fix: exit test on completion * fix: tear down of docker compose * fix: remove npm link * fix: replace lockbox_url to enclaves in env variables * fix: assert statement for mercury server unavailability * fix: remove build.yml from actions * chore: remove unused imports and logs * fix: add regtest and signet to config * fix: getElectrumClient import --------- Co-authored-by: DhananjayPurohit <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.