Skip to content

Replace Boost.Test with Boost.LightweightTest in test/extension/io - #464

Merged
mloskot merged 1 commit into
boostorg:developfrom
mloskot:ml/replace-boost-test-with-boost-lt-in-io
Mar 24, 2020
Merged

Replace Boost.Test with Boost.LightweightTest in test/extension/io#464
mloskot merged 1 commit into
boostorg:developfrom
mloskot:ml/replace-boost-test-with-boost-lt-in-io

Conversation

@mloskot

@mloskot mloskot commented Mar 23, 2020

Copy link
Copy Markdown
Member

Description

Move test sources to format-specific directories.
Add FIXME and TODO comments about issues discovered while building the format-specific tests.
Add missing #include-s with standard library headers.
Apply minor corrections and tidying up.

References

Tasklist

  • Move test sources to format-specific directories (e.g. test/extension/io/jpeg_.cpp to test/extension/io/jpeg/.cpp)
  • Refactor each .cpp file as separate test with main using Boost.LT
  • Update Jamfile files to catch up with the new folders layout
  • Update CMakeLists.txt. For now, only for the simple target. The full is never built on CI and we are going to re-work CMake anyway (see Swtich to common Boost configuration for CMake with boost_gil target #463)
  • Ensure all CI builds pass

Move test sources to format-specific directories.
Add FIXME and TODO comments about issues discovered while building the format-specific tests.
Add missing `#include`-s with standard library headers.
Apply minor corrections and tidying up.

Continues boostorg#459
Closes boostorg#461
@mloskot mloskot added cat/enhancement Improvements, but not fixes addressing identified bugs ext/io boost/gil/extension/io/ cat/refactoring Any nonfunctional changes test New tests development or missing tests issues, no new functionality labels Mar 23, 2020
@mloskot mloskot added this to the Boost 1.73 milestone Mar 23, 2020
@mloskot mloskot self-assigned this Mar 23, 2020
@mloskot
mloskot merged commit 70ff564 into boostorg:develop Mar 24, 2020
@mloskot
mloskot deleted the ml/replace-boost-test-with-boost-lt-in-io branch March 24, 2020 09:21
mloskot added a commit to mloskot/gil that referenced this pull request Mar 24, 2020
mloskot added a commit to mloskot/gil that referenced this pull request Mar 24, 2020
mloskot added a commit that referenced this pull request Mar 24, 2020
mloskot added a commit to mloskot/gil that referenced this pull request Mar 30, 2020
* origin/develop: (70 commits)
  Always build test/extension/io/targa tests for images hosted in source tree
  Simplify test/extension/io/CMakeLists.txt
  Remove detailed C++/compiler/test matrix from README.md
  Add utilities to make some basic types printable for BOOST_TEST_EQ [ci skip]
  Update release notes   [ci skip]
  Update release notes [ci skip]
  Add good-enough .clang-format to example/clang-format/
  Catch and report exceptions thrown from test/extension//simple test
  Fix initialization order of scanline_read_iterator members (-Wreorder)
  Remove compile-time configuration macros from Jamfile
  Catch and report exceptions thrown from test/extension/io/bmp tests
  Always build test/extension/io/bmp//make for images hosted in source tree
  Always build test/extension/io/bmp tests for images hosted in source tree
  Always build test/extension/io//simple if format libraries are available (boostorg#467)
  Clean up CI and CMake from Boost.Test and Boost.System remains (boostorg#465)
  Removed extension/dynamic_image/reduce.hpp as unused (boostorg#466)
  Replace Boost.Test with Boost.LightweightTest in test/extension/io (boostorg#464)
  Fix -Wunused-variable warning
  AzP: Switch to MacOS 10.14  [skip appveyor]
  Replace Boost.Test with Boost.LightweightTest in test/ (boostorg#459)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cat/enhancement Improvements, but not fixes addressing identified bugs cat/refactoring Any nonfunctional changes ext/io boost/gil/extension/io/ test New tests development or missing tests issues, no new functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace Boost.Test with Boost.LightweightTest in test/extension/io

1 participant