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 CI workflow for pyln tests #49

Closed
wants to merge 67 commits into from
Closed

Add CI workflow for pyln tests #49

wants to merge 67 commits into from

Conversation

fmhoeger
Copy link
Collaborator

  • Add CI workflow for pyln tests
  • Update package versions used in nix-package build workflow

Includes all changes from #47 and #48

Needs some commit history cleanup before merging.

fmhoeger and others added 30 commits December 15, 2023 18:41
@fmhoeger
Add test for wallet remove
55ef84b
@fmhoeger
Delete smaug db when removing wallet (WIP)
22ec7a7
@fmhoeger
Remove the wallet db file only
0b34c82
@fmhoeger
Refactor constructing db_path to get_db_path
f41cd66
@fmhoeger
Refactor constructing db_path to get_db_path /2
9c2e8a0
@fmhoeger
Rename get_db_path -> get_db_file_path
cff8042
@fmhoeger
Add test for RPC list
004401d
@fmhoeger
Add additional asserts
b0b9795
@fmhoeger
Do not expose db_path, instead retrieve from inside test code
4a32a83
@fmhoeger
Merge branch '42-test-remove-rpc' into 46-test-list-rpc
586475d
@fmhoeger
Merge branch 'feat/add-tests' into 46-test-list-rpc
9adad32
@fmhoeger
Refactor tests
03a5f5a
@fmhoeger
Refactor; add tests for gap and birthday
0b4b8db
@fmhoeger
Cleanup/update .pre-commit-config.yaml
9697532
@fmhoeger
Add test for rpc add
37e4041
@fmhoeger
Cleanup
3c90d72
@fmhoeger
Add balance to smaug wallet ls RPC result; add tests
445d20c
@fmhoeger
Create pyln-tests.yml
ffbe5b0
@fmhoeger
Update pyln-tests.yml
3b9418e
@fmhoeger
Update pyln-tests.yml
31f97de
@fmhoeger
Update pyln-tests.yml
302b44c
@fmhoeger
Update pyln-tests.yml
fdb5c61
@fmhoeger
Update pyln-tests.yml
7c6b023
@fmhoeger
Update pyln-tests.yml
f9a78a1
@fmhoeger
Update pyln-tests.yml
4b4cd48
@fmhoeger
Update pyln-tests.yml
9267b2f
@fmhoeger
Update pyln-tests.yml
61b04fc
@fmhoeger
Update pyln-tests.yml
bdff438
@fmhoeger
Merge branch '46-test-list-rpc' into add-pyln-tests-ci-workflow
af5e6e9
@fmhoeger
Update pyln-tests.yml
321847d
@fmhoeger
Cleanup obsolete build steps
b81c1fb
@fmhoeger
Cleanup
b3e5796
@fmhoeger
Add startup-regtest.sh script; cleanup
16c2f03
@fmhoeger
Try fixing cln install script
a261d8e
@fmhoeger
Try fixing cln install script
5f98023
@fmhoeger
Try fixing cln install script
7beee9c
@fmhoeger
Try fixing cln install script
f9bbda0
@fmhoeger
Remove startup regtest script
44f81b8
@fmhoeger
Delete startup-regtest.sh script;
fa4a6be
@fmhoeger
Add cargo build
714ee76
@fmhoeger
Debug
bb2f9c5
@fmhoeger
Debug
54de3cc
@fmhoeger
Build CLN from source
ccec374
@fmhoeger
install CLN from niftynei/nifty/onchain_notif
2b0dd63
@fmhoeger
install CLN from niftynei/nifty/onchain_notif
6e5c72b
@fmhoeger
install CLN from niftynei/nifty/onchain_notif
b0c2fa8
@fmhoeger
install CLN from niftynei/nifty/onchain_notif
2fd770d
@fmhoeger
Fix tests step
e045392
@fmhoeger
Cleanup
e6a968e
@fmhoeger
Format yAML
6b01f1a
@fmhoeger
Use bitcoin 26.0
ddccf4b
@fmhoeger
Use bitcoin 26.0
a239542
@fmhoeger
Update nix build actions package versions
c0675fb
@fmhoeger
Merge branch '39-add-wallet-balance-to-wallet-output' into add-pyln-t…
87d7d10 
…ests-ci-workflow
@fmhoeger
Update regex for bircoin v26 (use h instead of single quote for harde…
175658e 
…ned paths)
@fmhoeger fmhoeger self-assigned this Dec 31, 2023
@fmhoeger fmhoeger changed the title Add pyln tests to CI workflow Add CI workflow for pyln tests Dec 31, 2023
@fmhoeger
Copy link
Collaborator Author

Closed this one as the actions are not triggered on PR from the forked repo

@fmhoeger fmhoeger closed this Dec 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chrisguida chrisguida Awaiting requested review from chrisguida

Assignees

@fmhoeger fmhoeger

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@fmhoeger