Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add unit tests #27

Open
achimnol opened this issue Jan 19, 2016 · 0 comments
Open

Add unit tests #27

achimnol opened this issue Jan 19, 2016 · 0 comments

Comments

@achimnol
Copy link
Member

It might seem too late, but still unit testing rocks.
Let's add infrastructure to test common headers and functions throughout the framework.

achimnol added a commit that referenced this issue Jan 20, 2016
 * Insert commented "#require <path/to/module.o>" lines to declare
   dependent object files for each test case.

 * Those object files will be automatically rebuilt just like
   normal "all" compilation.
achimnol added a commit that referenced this issue Jan 20, 2016
achimnol added a commit that referenced this issue Jan 20, 2016
 * `snakemake testall` combines all test cases in a single binary,
   tests/test_all.
   You may use gtest's filter options to run only specific test cases.

 * `snakemake test` builds individual executables for each test suite.
   This is for isolated, faster-iteration test cases.

 * `snakemake cleantest` removes test-related binaries.
   (It does NOT remove other framework/element binaries, though.
   You should use `snakemake clean` to remove them.)
achimnol added a commit that referenced this issue Jan 22, 2016
 * Removes non-necessary "default_value" template argument for
   FixedRing.

 * Forbids implicit mis-use of default constructor of FixedRing.

 * Adds "NEW" macro that allocates a memory block using NUMA-aware
   malloc and then calls the constructor on the pointer to it.
achimnol added a commit that referenced this issue Jan 23, 2016
 * Since googletest v1.3, it provides a "fusing" script that combines
   all gtest source files into a single .cc file.

 * To update to latest version of gtest, first run git submodule update
   and then run cd 3rdparty && 3rdparty/update-googletest.sh.
achimnol added a commit that referenced this issue Feb 8, 2016
 * Tried to support OpenCL, but confirmed that we need OpenCL 2.0+.
   (which is not supported by current generation of Xeon Phi...)

   - Related codes will be rewritten someday using SVM (shared virtual
     memory) APIs in OpenCL 2.0+.

 * Reduced memory footprint of batch_ids array passed to the device.

 * Rollbacked ev_prepare -> ev_check watcher type change (c732a25),
   as it has broken CPU-only cross-node IP forwarding scenarios. :(

 * TODO: fix IPsec GPU-only mode...
achimnol added a commit that referenced this issue Feb 23, 2016
 * I've found alignment issues in complex/nested structs in CUDA;
   we need some thorough tests for them!
achimnol added a commit that referenced this issue Feb 23, 2016
 * It is working as expected.
achimnol added a commit that referenced this issue Feb 23, 2016
 * It is working as exepcted. ...then what's the problem with CUDA? :(
achimnol added a commit that referenced this issue Feb 24, 2016
 * It means that the CPU version and GPU version yields the same
   results upon the same inputs.
achimnol added a commit that referenced this issue Feb 27, 2016
achimnol added a commit that referenced this issue Feb 28, 2016
 * Now, it's time to write IPsec tests to get rid of the memory bug!
achimnol added a commit that referenced this issue Mar 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant