ChimeraTK / ChimeraTK/ControlSystemAdapter-DoocsAdapter

Complete tests of ZeroMQ sending

Open
#17 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C++
Stars
0
Forks
1
Avg merge
23h 29m
Merged PRs (30d)
1

Description

Follow up issue of #12 and #14. Depends on #14.

Write additional test cases for the different scenarios of ZeroMQ sending. Currently missing are different variants of configuration (macro pulse number source can be specified for each property or for the location) including bad ones (e.g. non-existing macro pulse number source). Also check if the data consistency is properly tested (see #14).

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading issues #12 and #14, then locate the existing tests for ZeroMQ sending and their configuration setup. Add cases for property-level and location-level macro pulse number sources, including nonexistent sources, and verify data consistency as described in #14; done means all these scenarios are covered by passing tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
networking, testing
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.