Recursive globbing unit test source files. #160
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
willGraham01
approved these changes
Nov 2, 2022
Co-authored-by: Will Graham <[email protected]>
willGraham01
added a commit
that referenced
this pull request
Nov 16, 2022
* DispersiveMultLayer tests * CMakeLists will now auto-find all array_tests files * FrequencyVectors test, Vector class cleanup * Recursive globbing unit test source files. (#160) * Recursive globbing unit test source files. * Specify test_*.cpp Co-authored-by: Will Graham <[email protected]> Co-authored-by: Will Graham <[email protected]> * DTilde test * IncidentField tests * FieldSample tests * Missed data validation check * DetectorSensitivityArrays tests * try executing DSA fftw_plan * Reminder for me * Apply suggestions from code review Co-authored-by: Sam Cunliffe <[email protected]> * Some SPDLOG cleanups, SECTION checks, etc * Remove SPDLOG name-dump Co-authored-by: Sam Cunliffe <[email protected]>
willGraham01
added a commit
that referenced
this pull request
Nov 17, 2022
* DispersiveMultLayer tests * CMakeLists will now auto-find all array_tests files * FrequencyVectors test, Vector class cleanup * Recursive globbing unit test source files. (#160) * Recursive globbing unit test source files. * Specify test_*.cpp Co-authored-by: Will Graham <[email protected]> Co-authored-by: Will Graham <[email protected]> * DTilde test * IncidentField tests * FieldSample tests * Missed data validation check * DetectorSensitivityArrays tests * try executing DSA fftw_plan * Reminder for me * tests for Vector class * Tests for ComplexAmplitudeSample * Avoid seg faults in Vector tests * Unit tests for GratingStructure * Tests for Pupil * Unit tests for EHVec * Unit tests for FrequencyExtractVector * Compute number of elements correctly in FEV before displaying * CCollection, also highlight logical flaw that might need addressing * Clear SPDLOG clutter * tests for Vector class * Tests for ComplexAmplitudeSample * Avoid seg faults in Vector tests * Unit tests for GratingStructure * Tests for Pupil * Unit tests for EHVec * Unit tests for FrequencyExtractVector * Compute number of elements correctly in FEV before displaying * CCollection, also highlight logical flaw that might need addressing * Clear SPDLOG clutter * CMaterial tests, prevent matlab casting seg-faults * DMaterial test * Apply suggestions from code review Co-authored-by: Sam Cunliffe <[email protected]> * Some SPDLOG cleanups, SECTION checks, etc * Remove SPDLOG name-dump * Cleanup SPDLOG junk * Some test refactoring * This doesn't seg fault * XYZTensor redundancy * SPDLOG junk * Pupil test sectioning * Pupil test case * GratingStructure test case * FrequencyExtractVector * FrequencyComponentsVector * Apply suggestions from code review Will need to go through and check for consistency Co-authored-by: Sam Cunliffe <[email protected]> * Sam's review comments * Sam's comments #2 Co-authored-by: Sam Cunliffe <[email protected]>
willGraham01
added a commit
that referenced
this pull request
Nov 17, 2022
* DispersiveMultLayer tests * CMakeLists will now auto-find all array_tests files * FrequencyVectors test, Vector class cleanup * Recursive globbing unit test source files. (#160) * Recursive globbing unit test source files. * Specify test_*.cpp Co-authored-by: Will Graham <[email protected]> Co-authored-by: Will Graham <[email protected]> * DTilde test * IncidentField tests * FieldSample tests * Missed data validation check * DetectorSensitivityArrays tests * try executing DSA fftw_plan * Reminder for me * tests for Vector class * Tests for ComplexAmplitudeSample * Avoid seg faults in Vector tests * Unit tests for GratingStructure * Tests for Pupil * Unit tests for EHVec * Unit tests for FrequencyExtractVector * Compute number of elements correctly in FEV before displaying * CCollection, also highlight logical flaw that might need addressing * Clear SPDLOG clutter * tests for Vector class * Tests for ComplexAmplitudeSample * Avoid seg faults in Vector tests * Unit tests for GratingStructure * Tests for Pupil * Unit tests for EHVec * Unit tests for FrequencyExtractVector * Compute number of elements correctly in FEV before displaying * CCollection, also highlight logical flaw that might need addressing * Clear SPDLOG clutter * CMaterial tests, prevent matlab casting seg-faults * DMaterial test * Move field tests to their own subdirectory * Refactor existing tests into consistent structure * Refactor common inline functions into unit_test_utils.h * Trailing newline * Apply suggestions from code review Co-authored-by: Sam Cunliffe <[email protected]> * Some SPDLOG cleanups, SECTION checks, etc * Remove SPDLOG name-dump * Cleanup SPDLOG junk * Some test refactoring * This doesn't seg fault * XYZTensor redundancy * SPDLOG junk * Pupil test sectioning * Pupil test case * GratingStructure test case * FrequencyExtractVector * FrequencyComponentsVector * normalise_volume docstring and unit test * test_TDFieldExporter2D.cpp * Virtual destructor fix * Move field tests to their own subdirectory * Refactor existing tests into consistent structure * Refactor common inline functions into unit_test_utils.h * Trailing newline * normalise_volume docstring and unit test * test_TDFieldExporter2D.cpp * Virtual destructor fix * Apply suggestions from code review Will need to go through and check for consistency Co-authored-by: Sam Cunliffe <[email protected]> * Sam's review comments * Sam's review no.1 * Sam's test utilities refactor * array_tests use tdms_tests::TOLERANCE Co-authored-by: Sam Cunliffe <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.