Test missing for std::string data type

ChimeraTK / ControlSystemAdapter

An adapter layer which allows to use control applications with different control system software environments.

GNU Lesser General Public License v3.0

3 stars 2 forks source link

Test missing for std::string data type #9

Closed mhier closed 5 years ago

mhier commented 7 years ago

There seem to be no test for the std::string data type. At least the just-fixed bug about the wrong default argument for the scalar initial value in DevicePVManager::createProcessArray is not covered.

killenb commented 5 years ago

Solved in 5c5527ced3240df604c522502a5b3307d6273479 All places where the supported data types are tested have been extended with std::string and (u)int64_t.

It turned out that std::string was not working for the TypeChangingDecorator as user type. This has been fixed as well.

The tests for the ReferenceTestApplication (fullStubTest) has also been extended with strings. In this course strings have been added to the ReferenceTestApplication, which also solves #15