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

tests: add basic test #11

Merged
merged 1 commit into from
May 2, 2024
Merged

tests: add basic test #11

merged 1 commit into from
May 2, 2024

Conversation

daywalker90
Copy link
Contributor

Hi, since the plugins repo will require plugins to have tests in the future, here is a very basic test.
setup.sh will build the binary during CI. tests/test_cln_ntfy.py contains the test and currently only checks if the plugins starts with the node and if the plugin options exist.

It would be nice to actually test a notification!

You can run tests like this:

Install dependencies:

pip3 install pyln-client pyln-testing

Build the plugin

cargo build

run tests :

pytest tests/test_cln_ntfy.py
# if you use a different profile than debug for compiling set the RUST_PROFILE like this:
RUST_PROFILE=release pytest tests/test_cln_ntfy.py

@daywalker90 daywalker90 reopened this Apr 23, 2024
@daywalker90
Copy link
Contributor Author

had to fix the setup.sh. It says i closed this PR but i did not do that oO

@yukibtc yukibtc merged commit 25c3c3e into yukibtc:master May 2, 2024
4 checks passed
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.

2 participants