Skip to content

Conversation

@jonathan-r-thorpe
Copy link
Contributor

BCP-008-01 Receiver Status Monitoring test suite

jonathan-r-thorpe and others added 30 commits October 6, 2023 11:37
AMWA-TV#839)

* Since jstasiak/zeroconf#666 (0.32.0) it needs a much much smaller monkeypatch to be able to advertise the overlong service type "_nmos-registration._tcp"

* Fix flake8/pycodestyle E721 do not compare types

* Bump Node 12.x to Node 16.x

* python-zeroconf 0.75.0 strict=False for _nmos-registration._tcp

* Python 3.6 (and 3.7!) is End-Of-Life, and so is Node.js 14.x
Libraries like python-zeroconf are starting to depend on more recent Python features

* Pass IP address of API under test to testssl.sh

So that same one is used by Python tests and testssl.sh tests

Co-authored-by: Simon Lo <[email protected]>

* Add MOCK_SERVICES_WARM_UP_DELAY

* Delay running tests until after MOCK_SERVICES_WARM_UP_DELAY

Co-authored-by: Simon Lo <[email protected]>

* Tweak message

Co-authored-by: jonathan-r-thorpe <[email protected]>

* Enhance test_35 (subscription and notification mechanism test) to allow interference notifications while running test

* Update IS1201Test.py

shorten line for flake8

---------

Co-authored-by: Gareth Sylvester-Bradley <[email protected]>
Co-authored-by: Gareth Sylvester-Bradley <[email protected]>
Co-authored-by: jonathan-r-thorpe <[email protected]>
(cherry picked from commit 70f633c56cf55481cf440f4d76ced36b3273cd6e)
@jonathan-r-thorpe jonathan-r-thorpe marked this pull request as ready for review July 30, 2025 16:14
Copy link
Contributor

@cristian-recoseanu cristian-recoseanu left a comment

Choose a reason for hiding this comment

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

All looks great. Tested both BCP-008-01 and BCP-008-02 using https://github.com/AMWA-TV/nmos-device-control-mock

@jonathan-r-thorpe jonathan-r-thorpe merged commit b286324 into AMWA-TV:master Jul 31, 2025
1 check passed
@jonathan-r-thorpe jonathan-r-thorpe deleted the bcp-008-01 branch July 31, 2025 11:00
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.

4 participants