#177 Add test for input adapters only

fktn-k / fkYAML

A C++ header-only YAML library

MIT License

69 stars 7 forks source link

#177 Add test for input adapters only #178

Closed fktn-k closed 11 months ago

fktn-k commented 11 months ago

Before this PR, there were no test cases only for input adapters. To ensure that input adapters work as they are expected with no dependencies on the other classes, new test cases for input adapters have been added with some test data files.

Along with the above implementation, some unused members in input adapter classes have been deleted.

coveralls commented 11 months ago

coverage: 100.0%. remained the same when pulling 0c78d4fbf3f2608a1bc58f8d7fa40e9461fbc158 on feature/177_add_test_for_input_adapters_only into 81f9b4eb2bf91081c0ea9c2bd7ad414d1f797eca on develop.

fktn-k commented 11 months ago

CTest failed in reading a test data file in the unit test cases for input adapters, just with MSVC (not with gcc/clang).
So I made a quick workaround to specify the test data directory path by creating a header file in test/unit_test/CMakeLists.txt file.