Skip to content

test: Add s2v2 test (checking for NLS) #86

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

rzr
Copy link
Collaborator

@rzr rzr commented Apr 18, 2025

Change

test: Add s2v2 test (checking for NLS)

Notes this change should be adapted to provide a MQTT response,
once NLS is enabled on device, for zpc output (in debug mode) is checked.

Checklist

rzr and others added 3 commits April 18, 2025 15:24
Check usage_ for how to use it.

This tool works along z-wave applications adapted for linux,
This package is maintained at z-wave-alliance and only available
to Z-Wave-Alliance members.

It is also supported in headless mode and can be run in a ci job,
as long are assets are made available (using GH_ZWAVE_ACCESS_TOKEN secret)

Alternatively it can be adapted to support real hardware,
the play recipe should not change.

Origin: SiliconLabsSoftware#85
Relate-to: Z-Wave-Alliance/z-wave-stack-binaries#5
Relate-to: Z-Wave-Alliance/z-wave-stack#732
Relate-to: Z-Wave-Alliance/z-wave-stack-binaries#2
Relate-to: SiliconLabsSoftware/z-wave-engine-application-layer#23
Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
Notes this change should be adapted to provide a MQTT response,
once NLS is enabled on device, for zpc output (in debug mode) is checked.

Signed-off-by: Philippe Coval <philippe.coval@silabs.com>
@rzr rzr marked this pull request as draft April 18, 2025 13:29
@rzr
Copy link
Collaborator Author

rzr commented Apr 18, 2025

Please 1st merge #85 and then rebase it

command="EnableNls"
pub="ucl/by-unid/$nodeunid/State/Commands/$command"
message="{}"
pub_ "$pub" "$message" "" # TODO check sub , and run zpc in debug
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rzr rzr force-pushed the main branch 2 times, most recently from 5486b41 to 78dfbc4 Compare April 22, 2025 15:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant