Refactoring tests

[fgochez]

The tests have been restructured and enhanced for maintainability. In outline, the following changes have been made:

• Created a "testall" build target that generates a single "testall" executable which runs all tests.
• Removed "main" functions from tests and moved them to a stand-alone testxxx_main.cpp file. These will allow individual tests to be run in a standalone fashion.
• Refactored existing tests to make structure more consistent, added PartioTests namespace.
• Moved old "testkdtree.cpp" tests to "testparticlessimple.cpp", and added a new set of tests which test the KdTree class in a standalone fashion to "testkdtree.cpp".

In general existing test logic has not really been changed except for the KdTree tests, so modification of the tests is work in progress. However, I think it might be a good time to merge in the new structure and continue enhancing tests in the future.

Ongoing discussion:
http://groups.google.com/group/partio-discuss/browse_thread/thread/19984fe2b39749e4

[aselle]

Thanks for the contribution. Sorry I haven't gotten to this yet. But I hope
to real soon(tm).

On Thu, May 31, 2012 at 2:59 PM, Francisco Gochez <
reply@reply.github.com

wrote:

The tests have been restructured and enhanced for maintainability. In
outline, the following changes have been made:

• Created a "testall" build target that generates a single "testall"
  executable which runs all tests.
• Removed "main" functions from tests and moved them to a stand-alone
  testxxx_main.cpp file. These will allow individual tests to be run in a
  standalone fashion.
• Refactored existing tests to make structure more consistent, added
  PartioTests namespace.
• Moved old "testkdtree.cpp" tests to "testparticlessimple.cpp", and added
  a new set of tests which test the KdTree class in a standalone fashion to
  "testkdtree.cpp".

In general existing test logic has not really been changed except for the
KdTree tests, so modification of the tests is work in progress. However, I
think it might be a good time to merge in the new structure and continue
enhancing tests in the future.

Ongoing discussion:

http://groups.google.com/group/partio-discuss/browse_thread/thread/19984fe2b39749e4

You can merge this Pull Request by running:

git pull https://github.com/fgochez/partio master

Or you can view, comment on it, or merge it online at:

#26

-- Commit Summary --

• Refactoring test code. Test routines are being seperated out from "main"
• Added comment to point out failing tests
• Minor tweaks to code layout
• Changing namespace for tests from "Partio" to "PartioTests".
• Merge remote-tracking branch 'upstream/master'
• - moving header files out of "namespace Partio" (leaving them
• (WIP) - changing to a direct test of kdtree
• removing data generation set generation logic, replacing with
• Small addition to docs
• Correcting and adding more tests
• added tests with non-stl containers
• Re-adding the old "kdtree" tests as tests for the

-- File Changes --

M src/lib/core/KdTree.h (19)
M src/tests/CMakeLists.txt (16)
A src/tests/partiotesting.h (32)
A src/tests/testall.cpp (56)
M src/tests/testcache.cpp (18)
A src/tests/testcache.h (9)
A src/tests/testcache_main.cpp (45)
M src/tests/testiterator.cpp (30)
A src/tests/testiterator.h (8)
A src/tests/testiterator_main.cpp (40)
M src/tests/testkdtree.cpp (230)
A src/tests/testkdtree.h (49)
A src/tests/testkdtree_main.cpp (41)
A src/tests/testparticlessimple.cpp (130)
A src/tests/testparticlessimple.h (47)
A src/tests/testparticlessimple_main.cpp (41)
R src/tests/testsaveload.cpp (51)
A src/tests/testsaveload.h (49)
A src/tests/testsaveload_main.cpp (49)
M src/tests/teststr.cpp (60)
A src/tests/teststr.h (12)
A src/tests/teststr_main.cpp (44)

-- Patch Links --

https://github.com/wdas/partio/pull/26.patch
https://github.com/wdas/partio/pull/26.diff

---

Reply to this email directly or view it on GitHub:
#26