diff options
| author | Carl Hetherington <cth@carlh.net> | 2023-08-22 23:45:13 +0200 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2023-08-22 23:45:14 +0200 |
| commit | 5980cee25f537c54fc1155de5492bddd92bfa338 (patch) | |
| tree | 4f9e87de2571fc6fdfdfe7ca8c1a5dd2102ce7a1 /test | |
| parent | ac449700a47c164bf09ee2b0c18486ed255ca20f (diff) | |
Accept any case for subtitle alignment specifiers (#2602).v1.8.78
SMPTE seems pretty clear to me that these should be lower case, but
counterexamples have been seen in the wild and it seems to make
sense to allow them.
Perhaps the verifier should raise an error with non-lower-case
examples.
Diffstat (limited to 'test')
| -rw-r--r-- | test/h_align_test.cc | 8 | ||||
| -rw-r--r-- | test/v_align_test.cc | 8 |
2 files changed, 10 insertions, 6 deletions
diff --git a/test/h_align_test.cc b/test/h_align_test.cc index f8e9bd7f..60e87341 100644 --- a/test/h_align_test.cc +++ b/test/h_align_test.cc @@ -42,8 +42,10 @@ BOOST_AUTO_TEST_CASE(h_align_test) BOOST_CHECK(dcp::string_to_halign("left") == dcp::HAlign::LEFT); BOOST_CHECK(dcp::string_to_halign("center") == dcp::HAlign::CENTER); BOOST_CHECK(dcp::string_to_halign("right") == dcp::HAlign::RIGHT); - BOOST_CHECK_THROW(dcp::string_to_halign("Left"), dcp::ReadError); - BOOST_CHECK_THROW(dcp::string_to_halign("CentER"), dcp::ReadError); - BOOST_CHECK_THROW(dcp::string_to_halign("RIGHT"), dcp::ReadError); + BOOST_CHECK(dcp::string_to_halign("Left") == dcp::HAlign::LEFT); + BOOST_CHECK(dcp::string_to_halign("CentER") == dcp::HAlign::CENTER); + BOOST_CHECK(dcp::string_to_halign("RIGHT") == dcp::HAlign::RIGHT); + + BOOST_CHECK_THROW(dcp::string_to_halign("ostrabagalous"), dcp::ReadError); } diff --git a/test/v_align_test.cc b/test/v_align_test.cc index 69951390..6bd77e44 100644 --- a/test/v_align_test.cc +++ b/test/v_align_test.cc @@ -42,8 +42,10 @@ BOOST_AUTO_TEST_CASE(v_align_test) BOOST_CHECK(dcp::string_to_valign("top") == dcp::VAlign::TOP); BOOST_CHECK(dcp::string_to_valign("center") == dcp::VAlign::CENTER); BOOST_CHECK(dcp::string_to_valign("bottom") == dcp::VAlign::BOTTOM); - BOOST_CHECK_THROW(dcp::string_to_valign("TOP"), dcp::ReadError); - BOOST_CHECK_THROW(dcp::string_to_valign("CENter"), dcp::ReadError); - BOOST_CHECK_THROW(dcp::string_to_valign("Right"), dcp::ReadError); + BOOST_CHECK(dcp::string_to_valign("TOP") == dcp::VAlign::TOP); + BOOST_CHECK(dcp::string_to_valign("CENter") == dcp::VAlign::CENTER); + BOOST_CHECK(dcp::string_to_valign("Bottom") == dcp::VAlign::BOTTOM); + + BOOST_CHECK_THROW(dcp::string_to_valign("ostrabagalous"), dcp::ReadError); } |