ChimeraTK / ChimeraTK/ControlSystemAdapter

Test all data types with persistent data storage

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

Nobody has claimed this yet.

redmine
Dominant language
C++
Stars
3
Forks
3
PR merge metrics
No merged PRs in 30d

Description

The tests for persistent data storage only use uint16_t, float and int32_t. They should be done for all data types, especially also for std::string.

Recommendation: Start adding a variable with std::string to all test before adding the remaining numerical types.

This is a follow up of #9. It should be done after #11 (or maybe as a sub-task if it?)

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 locating the persistent data storage tests and reviewing follow-up issues #9 and #11, especially the dependency on #11. Add std::string coverage to each relevant test before covering the remaining numerical types; done means all data types are exercised.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
databases, testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.