Always C++ filesystem instead of boost::filesystem:

See merge request !18

boost::filesystem was used as a fallback for OSX < 10.15 that
lacked <filesystem>. This version is now non long-unsupported
by Apple, so let's drop the boost-filesystem dependency.

Use CMake-generated export headers.

See merge request !17

merge from ReForm

See merge request !16

Add missing include for ptrdiff_t

See merge request !15

Merge latest changes to Base from ReForm

See merge request !14

Martin Marinov authored 2 years ago and Martin Marinov committed 2 years ago

* Refactor shared data code from Checksum::Compare to Base::SafeSharedPtrT and reuse it in Checksum::Report::Write.
* Various changes to simplify and streamline report symbols naming and decouple symbols from the tests.

Add buffered text output file for faster text writes. Improve platform compatibility for printing.

* Add BASE_CONCAT and BASE_UNIQUE_NAME to BaseDefines.hh

* add Scope.hh utils containing helpers to perform actions at the end of a scope, such as calling a function or restoring a variables original value

* add a tolerant comparison class and make it the default for floating point values

* reorder Difference enum to make negligible changes least significant.

* report number of negligible tests in the summary at the end of the test run

* add DEB_time_session_avg which prints the average timing based on a user provided divisor

* add output operator for ptrdiff_t

* change base to build mac 10.15

* add log to check MAC_1015 variable in Filesystem.hh

* add Availability Macros header in Filesystem.hh

* add FILESYSTEM_USE_BOOST

Co-authored-by: administrator <administrator@autodesk.com>

* Fixes compile errors on VS2017, due to missing second locale argument in calls to std::tolower and std::isspace in Test files.
* Use standard naming convention C_LOCALE.

* Fix the check preventing Filesystem builds on APPLE w/o Boost
* Remove obsolete definitions from CMake
* Fix build error with OptionT<> when TEST_ON is undefined
* Add missing includes when TEST_ON is undefined
* Remove the error message if JOURNAL_USE_BOOST is undefined

* Add Base::OptionT<>
* Add StringOption and BoundOption
* Add BoundToggleGroup

* Make static data non-static in Test::Checksum::Compare. This allows
  us to pass instances of Test::Checksum::Compare between
  binaries. Static data in a class always refers to data in the
  current binary, and so it's not possible to pass it (as static data)
  between binaries.

* Move Test::Checksum::Compare implementation to TestChecksum.cc so
  that it can store a non-static reference to the checksum registry
  (for the reason described above).

* Remove use of Base::filesystem from TestChecksumCompare.hh, as
  ReForm cannot currently use std::filesystem.

* Remove the Test::Paths constructor that takes argc and argv values,
  as this is no longer used.

* Catch exceptions in TestReport more carefully.

* Expose the process name in TestArgs.

Co-authored-by: Martin Marinov <martin.marinov@autodesk.com>

* Move error-related code for Base/Test to TestError.(cc|hh)

* Create Test::Error, a derived class of Base::Error, to store error
  codes/messages for Base/Test

* Replace std exceptions with Test::Error wherever possible

* Create a new Compare class in TestChecksumCompare.(cc|hh) that
  contains the code from the function Test::compare_checksums()
  (previously in ReForm Preprocess) and Test::compare_reports()
  (previously in TestResultAnalysis.(cc|hh)).

  Checksum objects reside in the binary that implements the tested
  algorithms (A), which may not be the same binary that runs the
  comparisons (B). In order to deal with this, we create an instance
  of the Compare class for each copy of Base contained within the
  final binary. The Compare object from A can be provided to B to
  perform the comparisons.

* Create NullOutputStream and a global instance Base::null_os. This
  ignores any data streamed to it.

Co-authored-by: Martin Marinov <martin.marinov@autodesk.com>

* Remove make_summary_report() and associated code from TestReport

  The report executable no longer needs to be run twice to produce a
  comparison, and no longer produces the <test_dir>_base/ directory
  when processing <test_dir>/.

* Update documentation

* Move PathLink from TestReport.cc -> Paths/PathLink.hh

* Move TestFilesystem.hh -> Paths/Filesystem.hh

* Add macros (particularly BASE_TEST()) in TestCase.hh to create a
  wrapper for the Catch2 macro TEST_CASE() (or any other test macro
  that defines a void function) to allow checksum comparisons to be
  performed at the end of a test.

* Remove timing information and empty lines from checksum
  reports. Consequently, remove filter_report() from TestReport.cc and
  replace it with a simple copy.

... to fix an issue where multiple components are using the API (#56)

Also move the CHECK_CONDITION() macro and associated code to
TestChecksumCondition.(cc|hh).

If TEST_REPORT_PARSE_CTEST_ERRORS *is* defined, the report executable
will state the tests that are listed in the CTest log but for which no
checksums were found.

If TEST_REPORT_PARSE_CTEST_ERRORS *is not* defined, the report
executable will check that the number of tests listed in the CTest log
is the same as the number of tests for which checksums were found and
will print a warning if this is not the case.

XCode10 lacks support for several C++17 standard library
features. Hence, for macOS, std::size() has been replaced with calls
to sizeof(), and std::map::try_emplace() has been replaced with calls
to std::map::emplace().