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 #15

Merged
merged 6 commits into from
Oct 14, 2022
Merged

Test #15

merged 6 commits into from
Oct 14, 2022

Conversation

DanGould
Copy link
Contributor

@DanGould DanGould commented Oct 2, 2022

This integration test inits a lighining cluster of two peers in docker and then makes loin receiving server available over https via nginx reverse proxy.

The payjoin still needs to be made

@DanGould
Copy link
Contributor Author

DanGould commented Oct 4, 2022

If the test is written in rust, loin code should run directly instead of the output executable in the test .

@DanGould DanGould force-pushed the test branch 2 times, most recently from e3cf468 to 6c16048 Compare October 8, 2022 14:27
@DanGould
Copy link
Contributor Author

DanGould commented Oct 9, 2022

This is missing automation for localhost ssl cert & key. Other than that, it works

@DanGould DanGould marked this pull request as ready for review October 10, 2022 18:54
@DanGould
Copy link
Contributor Author

This should now generate a self-signed cert and payjoin on the local server. it depends on docker running on the host machine. Test it!

Still needs to be rebased with pretty messages and would benifit from the integration.rs test script being split. The ideas are all here.

@DanGould DanGould force-pushed the test branch 2 times, most recently from d525073 to 3504e82 Compare October 12, 2022 20:24
@DanGould
Copy link
Contributor Author

DanGould commented Oct 12, 2022

This sucks much less now. We use a Fixture to clean up after itself. A few things hang:

  • gitignore localhost certs
  • generate those certs @ the beginning
  • upgrade edition on the commit introducing TLS client. It's broken otherwise

@DanGould DanGould force-pushed the test branch 2 times, most recently from d202d0c to c0bc6d2 Compare October 13, 2022 01:33
@DanGould
Copy link
Contributor Author

This opens a channel and tests that it has a balance :)

@DanGould DanGould force-pushed the test branch 3 times, most recently from 99206b2 to 4869a3c Compare October 14, 2022 19:46
@DanGould DanGould force-pushed the test branch 2 times, most recently from 26a7836 to 758cb96 Compare October 14, 2022 20:29
@DanGould
Copy link
Contributor Author

There are lots of ways this test could be prettier, but it advances our codebase without regressions as I see it. LFG

@DanGould DanGould merged commit f266ccd into master Oct 14, 2022
@DanGould DanGould deleted the test branch October 14, 2022 20:35
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.

1 participant