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

Tests setup part two #3

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

Tests setup part two #3

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
3b5f4a5
Ignore audit advisory for integration tests
orbitalturtle May 27, 2023
a792209
Change CI tests to run sequentially
orbitalturtle Jun 2, 2023
8f3ee1c
tests: setup lnd for integration tests
orbitalturtle Apr 9, 2023
2c6fa55
tests: add support for ldk node in integration tests
orbitalturtle Jun 2, 2023
511a5b9
tests: move test lnd logs to a more permanent location
orbitalturtle Jun 3, 2023
f63c87d
tests: move test ldk logs to a more permanent location
orbitalturtle Jul 11, 2023
4af0d0a
actions: upload lnd and ldk logs using github actions
orbitalturtle Jun 21, 2023
e11c4e0
tests: Change lnd version to custom version that supports custom mess…
orbitalturtle Jul 12, 2023
bda8487
tests: restructure ldk node into a struct with info we need
orbitalturtle Jul 12, 2023
2dca9fc
tests: return needed electrds node and connect our bitcoind nodes
orbitalturtle Jul 13, 2023
cde0456
main: move main logic for running lndk into a library
orbitalturtle Jul 13, 2023
db0e8f1
FIX
orbitalturtle Jul 13, 2023
ca43927
ANOTHER FIX
orbitalturtle Jul 15, 2023
994e9ea
In progress...
orbitalturtle Jul 20, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .cargo/audit.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
[advisories]
ignore = ["RUSTSEC-2020-0071"]
14 changes: 13 additions & 1 deletion .github/workflows/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,19 @@ jobs:
name: cargo test
with:
command: test
args: --all-targets --benches
args: --all-targets --benches -- --test-threads=1
- name: Archive lnd logs
uses: actions/upload-artifact@v3
with:
name: lnd-logs
path: |
/tmp/lnd_logs
- name: Archive ldk logs
uses: actions/upload-artifact@v3
with:
name: ldk-logs
path: |
/tmp/ldk_logs
- uses: actions-rs/cargo@v1
name: cargo fmt
with:
Expand Down
5 changes: 5 additions & 0 deletions CONTRIBUTING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,3 +46,8 @@ cargo clippy
2. Error String: Lower case, no `.` terminating.
3. Display imps: Lower case, no `.` terminating.

### Integration tests

Note that the way the integration tests run now, it's best to run them sequentially rather than concurrently (which is Rust's default). You can run them sequentially with this command:

`cargo test -- --test-threads=1`
Loading