search

WojciechRynczuk / vcdMaker

A tool for converting text log files to the VCD format.
https://vcdmaker.org
MIT License
29 stars 2 forks source link

014 unit test #26

Closed endrjux closed 7 years ago

endrjux commented 7 years ago

Unit Tests for vcdMaker

See #14.

WojciechRynczuk commented 7 years ago

General remark: Maybe the unit testing files could be suffixed so as we wouldn't have two Signal.cpp files. What do you think about it?

WojciechRynczuk commented 7 years ago

General remark: There are doxygen warnings that shall be fixed. Most of them shall be removed if you'd rebase to the v3_release branch. But still a few of them will remain and need to be addressed.

WojciechRynczuk commented 7 years ago

I was also thinking about making the process of running UT under Windows easier. But it seems to me not worth the effort at the moment. They can be run on Linux smoothly. I like the output :)

WojciechRynczuk commented 7 years ago

How about vcdMerge or vcdMaker specific tests? Will we keep them here as well?

endrjux commented 7 years ago

Some answers:

  1. Directory structure is "prefix".
  2. Will fix.
  3. I think it should just work, but maybe something special needs to be done in CMake for VS.
  4. They will be places in \sources\<app>\test\unitTest.