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

[IS 12] Validate Get/Set/Add/Remove/Find methods #822

Merged
merged 48 commits into from
Aug 4, 2023
Merged

[IS 12] Validate Get/Set/Add/Remove/Find methods #822

merged 48 commits into from
Aug 4, 2023

Conversation

jonathan-r-thorpe
Copy link
Contributor

Checks for:

  • GetSequenceItem
  • SetSequenceItem
  • AddSequenceItem
  • RemoveSequenceItem
  • FindMembersByPath

garethsb and others added 30 commits May 24, 2023 16:34
@garethsb
Since jstasiak/zeroconf#666 (0.32.0) it needs a much much smaller mon…
b7ab7c6 
…keypatch to be able to advertise the overlong service type "_nmos-registration._tcp"
@garethsb
Merge pull request AMWA-TV#812 from AMWA-TV/zeroconf-marmoset
aa58b5e 
[IS-04-01] Much smaller zeroconf monkey patch
@jonathan-r-thorpe
Fixed error reporting
858ec15
@jonathan-r-thorpe
Added Set user label test
4732629
@jonathan-r-thorpe
Validate touchpoints
655b4a1
@jonathan-r-thorpe
Validate NcTouchpointNmos or NcTouchpointNmosChannelMapping depending…
fb85eb3 
… on contextNamespace
@jonathan-r-thorpe
Sequence command JSON helper functions
fa0a51b
@jonathan-r-thorpe
Check sequence manipulation functions
9935489
@jonathan-r-thorpe
Build device model object tree when validating device model
1901106 
Check FindMembersByRole
@garethsb @jonathan-r-thorpe
Merge pull request AMWA-TV#812 from AMWA-TV/zeroconf-marmoset
3761bab 
[IS-04-01] Much smaller zeroconf monkey patch

(cherry picked from commit aa58b5e)
@jonathan-r-thorpe
Recursively check each block's role paths
49f968a
@jonathan-r-thorpe
find members by role test
954d123
@jonathan-r-thorpe
Find members by class id test
e6b7f5c
@jonathan-r-thorpe
GetMemberDescriptors test
94aaedd 
Renumbered tests
@jonathan-r-thorpe
removed protocolVersion
d1890c6 
changed id and identity to classId
@jonathan-r-thorpe
GetSequenceLength test
b872ad2 
Disabled Add/Set/Remove sequence tests
@jonathan-r-thorpe
Moved IS-12 command support into IS12Utils
a64beef
@jonathan-r-thorpe
Refactored command execution
3f17d6b
@jonathan-r-thorpe
replaced protocol def dict with enumerations
d185e9c
@jonathan-r-thorpe
Renamed and reordered tests
8c7e295
@jonathan-r-thorpe
Set GetSequenceLength checked flag
debcbcb
@jonathan-r-thorpe
Subscription message
264453f 
(cherry picked from commit c963854efde2ba4b3c9fcf034ff1ead212fc7ca1)
@jonathan-r-thorpe
Subscriptions and notifications
46b2d17
@jonathan-r-thorpe
Subscribe to all objects and test changing all user labels
3956ee2
@jonathan-r-thorpe
Improve error reporting
921190b
@jonathan-r-thorpe
handle null descriptions
ef9006a
@jonathan-r-thorpe
Bug fix - now won't ignore description field in NcDescriptor
bc24b1e
@garethsb
Fix flake8/pycodestyle E721 do not compare types
884da90
@garethsb
Fix flake8/pycodestyle E721 do not compare types (AMWA-TV#825)
f7acf39
@jonathan-r-thorpe
Add ClassManager class
c51a42b
@jonathan-r-thorpe
Add Class Manager method tests
42bae13
@jonathan-r-thorpe
Factor out querying of class manager
814e6d8
@jonathan-r-thorpe
Tidy up querying of class manager and device model
7aa05b2
@jonathan-r-thorpe
Refactored validate device model
5531ca8
@jonathan-r-thorpe
Handle null sequences. Renamed some parameters
4a26c82
@jonathan-r-thorpe
Created NcBlock object for queried device model
0674217
@jonathan-r-thorpe
Fixed get_role_paths. Added get_oids
69a0962
@jonathan-r-thorpe
Added illegal and invalid oid tests
c9132a7
@jonathan-r-thorpe
Added illegal command handle test
7714d4b
@jonathan-r-thorpe
IndexOutOfBounds test. WebSocket kept open test
9b9cda6
@jonathan-r-thorpe
Renumbered tests. Added comments
a7e22fb
@jonathan-r-thorpe
Updated comment
df03b27
@jonathan-r-thorpe
Simplify querying of managers from device model
ac0e39d
@jonathan-r-thorpe
Removed redundant oid test
4bb4efb
@jonathan-r-thorpe
Merge branch 'master' into is-12-validate-methods
a6893ce
@jonathan-r-thorpe
correct linting error
6af4df6
@jonathan-r-thorpe
Move get_manager out of NcBlock
4dabde7
@jonathan-r-thorpe
Updated comments and links to specification
ebe162e
@jonathan-r-thorpe jonathan-r-thorpe merged commit e36e6cc into AMWA-TV:is-12 Aug 4, 2023
1 check passed
@jonathan-r-thorpe jonathan-r-thorpe deleted the is-12-validate-methods branch August 4, 2023 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jonathan-r-thorpe @garethsb