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

tests: basic cli/server integration tests #171

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

tests: basic cli/server integration tests #171

wants to merge 6 commits into from
+247 −83

Commits on Sep 6, 2024

  1. multi: remove RefCell from LndNodeSigner 

    Initially a RefCell was added to the LndNodeSigner because the original version
of tonic_lnd returns a mutable reference to the signer. However this reference
doesn't need to be mutable:
orbitalturtle/tonic_lnd#4

For this PR, this was needed so we could spin up the onion messenger and server
in a separate thread in the integration tests. But it's a change that could
help with future efficiency as well because the onion messenger can now be used
in a multi-threaded way.
    @orbitalturtle
    orbitalturtle committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    5c9f42e View commit details
    Browse the repository at this point in the history

  2. multi: move server host and port defaults to server.rs 

    It's just a more suitable location.
    @orbitalturtle
    orbitalturtle committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    c4386f3 View commit details
    Browse the repository at this point in the history

  3. server: split off server creation into run_server

    @orbitalturtle
    orbitalturtle committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    6b6a38e View commit details
    Browse the repository at this point in the history

  4. server: rename address to lnd_address for clarity

    @orbitalturtle
    orbitalturtle committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    be3dd86 View commit details
    Browse the repository at this point in the history

  5. tests: build LndkCli for server integration tests

    @orbitalturtle
    orbitalturtle committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    c53e369 View commit details
    Browse the repository at this point in the history

  6. tests: test cli - server interaction for making payments

    @orbitalturtle
    orbitalturtle committed Sep 6, 2024
    Configuration menu
    Copy the full SHA
    63b0bd7 View commit details
    Browse the repository at this point in the history