diff options
| author | Carl Hetherington <cth@carlh.net> | 2025-06-01 01:39:04 +0200 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2025-06-17 00:04:03 +0200 |
| commit | ec06811bae7ed4fc6bd80c3154fd473028ee8e13 (patch) | |
| tree | b398119a1a84d261ee16b57171652fd8714d657a /test/ratio_test.cc | |
| parent | 21e9bd77598f7bb4588ca9b16a58667e3340f6d9 (diff) | |
Pass Ratio around as an object rather than a pointer.
Diffstat (limited to 'test/ratio_test.cc')
| -rw-r--r-- | test/ratio_test.cc | 47 |
1 files changed, 19 insertions, 28 deletions
diff --git a/test/ratio_test.cc b/test/ratio_test.cc index a21859049..af3d9430a 100644 --- a/test/ratio_test.cc +++ b/test/ratio_test.cc @@ -32,44 +32,35 @@ using std::ostream; -BOOST_AUTO_TEST_CASE (ratio_test) +BOOST_AUTO_TEST_CASE(ratio_test) { - Ratio const * r = Ratio::from_id ("119"); - BOOST_CHECK (r); - BOOST_CHECK_EQUAL (fit_ratio_within (r->ratio(), dcp::Size (2048, 1080)), dcp::Size (1290, 1080)); + auto r = Ratio::from_id("119"); + BOOST_CHECK(fit_ratio_within(r.ratio(), dcp::Size(2048, 1080)) == dcp::Size(1290, 1080)); - r = Ratio::from_id ("133"); - BOOST_CHECK (r); - BOOST_CHECK_EQUAL (fit_ratio_within (r->ratio(), dcp::Size (2048, 1080)), dcp::Size (1440, 1080)); + r = Ratio::from_id("133"); + BOOST_CHECK(fit_ratio_within(r.ratio(), dcp::Size(2048, 1080)) == dcp::Size(1440, 1080)); - r = Ratio::from_id ("138"); - BOOST_CHECK (r); - BOOST_CHECK_EQUAL (fit_ratio_within (r->ratio(), dcp::Size (2048, 1080)), dcp::Size (1485, 1080)); + r = Ratio::from_id("138"); + BOOST_CHECK(fit_ratio_within(r.ratio(), dcp::Size(2048, 1080)) == dcp::Size(1485, 1080)); - r = Ratio::from_id ("166"); - BOOST_CHECK (r); - BOOST_CHECK_EQUAL (fit_ratio_within (r->ratio(), dcp::Size (2048, 1080)), dcp::Size (1800, 1080)); + r = Ratio::from_id("166"); + BOOST_CHECK(fit_ratio_within(r.ratio(), dcp::Size(2048, 1080)) == dcp::Size(1800, 1080)); - r = Ratio::from_id ("178"); - BOOST_CHECK (r); - BOOST_CHECK_EQUAL (fit_ratio_within (r->ratio(), dcp::Size (2048, 1080)), dcp::Size (1920, 1080)); + r = Ratio::from_id("178"); + BOOST_CHECK(fit_ratio_within(r.ratio(), dcp::Size(2048, 1080)) == dcp::Size(1920, 1080)); - r = Ratio::from_id ("185"); - BOOST_CHECK (r); - BOOST_CHECK_EQUAL (fit_ratio_within (r->ratio(), dcp::Size (2048, 1080)), dcp::Size (1998, 1080)); + r = Ratio::from_id("185"); + BOOST_CHECK(fit_ratio_within(r.ratio(), dcp::Size(2048, 1080)) == dcp::Size(1998, 1080)); - r = Ratio::from_id ("239"); - BOOST_CHECK (r); - BOOST_CHECK_EQUAL (fit_ratio_within (r->ratio(), dcp::Size (2048, 1080)), dcp::Size (2048, 858)); + r = Ratio::from_id("239"); + BOOST_CHECK(fit_ratio_within(r.ratio(), dcp::Size(2048, 1080)) == dcp::Size(2048, 858)); - r = Ratio::from_id ("190"); - BOOST_CHECK (r); - BOOST_CHECK_EQUAL (fit_ratio_within (r->ratio(), dcp::Size (2048, 1080)), dcp::Size (2048, 1080)); + r = Ratio::from_id("190"); + BOOST_CHECK(fit_ratio_within(r.ratio(), dcp::Size(2048, 1080)) == dcp::Size(2048, 1080)); } -BOOST_AUTO_TEST_CASE (ratios_use_same_pointers_test) +BOOST_AUTO_TEST_CASE(ratio_equivalence_test) { - auto const test = Ratio::from_id ("119"); - BOOST_CHECK_EQUAL (test, Ratio::from_id("119")); + BOOST_CHECK(Ratio::from_id("119") == Ratio::from_id("119")); } |