Add test cases for Reader #1108

dota17 · 2019-11-28T12:58:59Z

Add test cases for Reader in src\lib_json\json_reader.cpp.

coveralls · 2019-11-28T13:01:58Z

Coverage increased (+1.4%) to 93.521% when pulling a56f35a on dota17:addTestCase into 9e0d70a on open-source-parsers:master.

src/test_lib_json/main.cpp

BillyDonahue · 2019-12-02T04:22:50Z

Suggested changes submitted as PR to your fork. dota17#75

refactor structured error testing

update testcase for reader

ecc5ebb

BillyDonahue reviewed Nov 28, 2019

View reviewed changes

src/test_lib_json/main.cpp Show resolved Hide resolved

src/test_lib_json/main.cpp Outdated Show resolved Hide resolved

remove extra offsets test

771ea78

dota17 force-pushed the addTestCase branch from 6ed9b02 to 771ea78 Compare November 29, 2019 03:36

dota17 and others added 2 commits November 30, 2019 17:38

add a helper function

e4f0554

refactor structured error testing

776a852

Merge pull request #75 from BillyDonahue/addTestCaseTweak

e50300d

refactor structured error testing

dota17 force-pushed the addTestCase branch from a58cd3a to 478d077 Compare December 2, 2019 07:27

run clang-format

a56f35a

dota17 force-pushed the addTestCase branch from 478d077 to a56f35a Compare December 2, 2019 07:50

dota17 merged commit a3c8642 into open-source-parsers:master Dec 4, 2019

dota17 deleted the addTestCase branch December 26, 2019 08:12

Provide feedback