Add OK note when PKL and CPL annotation texts match.
[libdcp.git] / test / verify_test.cc
index a18058ef10968d0db40019e246fd620629b315fb..12cc953b7b04aace5ecaf098e15b9b77a3e4ea14 100644 (file)
@@ -63,6 +63,7 @@
 #include <boost/test/unit_test.hpp>
 #include <cstdio>
 #include <iostream>
+#include <tuple>
 
 
 using std::list;
@@ -83,19 +84,49 @@ static string filename_to_id(boost::filesystem::path path)
        return path.string().substr(4, path.string().length() - 8);
 }
 
-static boost::filesystem::path const dcp_test1_pkl = find_file("test/ref/DCP/dcp_test1", "pkl_").filename();
-static string const dcp_test1_pkl_id = filename_to_id(dcp_test1_pkl);
+static
+boost::filesystem::path
+dcp_test1_pkl()
+{
+       return find_file("test/ref/DCP/dcp_test1", "pkl_").filename();
+}
+
+static
+string
+dcp_test1_pkl_id()
+{
+       return filename_to_id(dcp_test1_pkl());
+}
 
-static boost::filesystem::path const dcp_test1_cpl = find_file("test/ref/DCP/dcp_test1", "cpl_").filename();
-static string const dcp_test1_cpl_id = filename_to_id(dcp_test1_cpl);
+static
+boost::filesystem::path
+dcp_test1_cpl()
+{
+       return find_file("test/ref/DCP/dcp_test1", "cpl_").filename();
+}
+
+static
+string
+dcp_test1_cpl_id()
+{
+       return filename_to_id(dcp_test1_cpl());
+}
 
 static string const dcp_test1_asset_map_id = "017b3de4-6dda-408d-b19b-6711354b0bc3";
 
-static boost::filesystem::path const encryption_test_cpl = find_file("test/ref/DCP/encryption_test", "cpl_").filename();
-static string const encryption_test_cpl_id = filename_to_id(encryption_test_cpl);
+static
+string
+encryption_test_cpl_id()
+{
+       return filename_to_id(find_file("test/ref/DCP/encryption_test", "cpl_").filename());
+}
 
-static boost::filesystem::path const encryption_test_pkl = find_file("test/ref/DCP/encryption_test", "pkl_").filename();
-static string const encryption_test_pkl_id = filename_to_id(encryption_test_pkl);
+static
+string
+encryption_test_pkl_id()
+{
+       return filename_to_id(find_file("test/ref/DCP/encryption_test", "pkl_").filename());
+}
 
 static void
 stage (string s, optional<path> p)
@@ -118,6 +149,43 @@ prepare_directory (path path)
 }
 
 
+static
+path
+find_prefix(path dir, string prefix)
+{
+       auto iter = std::find_if(directory_iterator(dir), directory_iterator(), [prefix](path const& p) {
+               return boost::starts_with(p.filename().string(), prefix);
+       });
+
+       BOOST_REQUIRE(iter != directory_iterator());
+       return iter->path();
+}
+
+
+static
+path
+find_cpl(path dir)
+{
+       return find_prefix(dir, "cpl_");
+}
+
+
+static
+path
+find_pkl(path dir)
+{
+       return find_prefix(dir, "pkl_");
+}
+
+
+static
+path
+find_asset_map(path dir)
+{
+       return find_prefix(dir, "ASSETMAP");
+}
+
+
 /** Copy dcp_test{reference_number} to build/test/verify_test{verify_test_suffix}
  *  to make a new sacrificial test DCP.
  */
@@ -165,50 +233,86 @@ dump_notes (vector<dcp::VerificationNote> const & notes)
 LIBDCP_ENABLE_WARNINGS
 
 
+static
+string
+to_string(dcp::VerificationNote const& note)
+{
+       string s = note_to_string(note) + dcp::String::compose(
+               "\n  [%1 %2 %3 %4 %5 %6 ",
+               static_cast<int>(note.type()),
+               static_cast<int>(note.code()),
+               note.note().get_value_or("<none>"),
+               note.file().get_value_or("<none>"),
+               note.line().get_value_or(0),
+               note.frame().get_value_or(0)
+               );
+
+       s += dcp::String::compose(
+               "%1 %2 %3 %4 %5]\n",
+               note.id().get_value_or("<none>"),
+               note.other_id().get_value_or("<none>"),
+               note.cpl_id().get_value_or("<none>"),
+               note.reference_hash().get_value_or("<none>"),
+               note.calculated_hash().get_value_or("<none>")
+               );
+
+       return s;
+}
+
+
 static
 void
-check_verify_result (vector<path> dir, vector<dcp::VerificationNote> test_notes)
+check_verify_result(vector<dcp::VerificationNote> notes, vector<dcp::VerificationNote> test_notes)
 {
-       auto notes = dcp::verify({dir}, &stage, &progress, {}, xsd_test);
-       std::sort (notes.begin(), notes.end());
-       std::sort (test_notes.begin(), test_notes.end());
+       std::sort(notes.begin(), notes.end());
+       std::sort(test_notes.begin(), test_notes.end());
 
-       string message = "\nVerification notes from test:\n";
-       for (auto i: notes) {
-               message += "  " + note_to_string(i) + "\n";
-               message += dcp::String::compose(
-                       "  [%1 %2 %3 %4 %5]\n",
-                       static_cast<int>(i.type()),
-                       static_cast<int>(i.code()),
-                       i.note().get_value_or("<none>"),
-                       i.file().get_value_or("<none>"),
-                       i.line().get_value_or(0)
-                       );
+       string message = "\n";
+
+       vector<dcp::VerificationNote> not_expected;
+       for (auto note: notes) {
+               auto iter = std::find_if(test_notes.begin(), test_notes.end(), [note](dcp::VerificationNote const& n) { return note.type() == n.type() && note.code() == n.code(); });
+               if (iter != test_notes.end() && *iter != note) {
+                       message += "Wrong details:\n --seen     " + to_string(note) + " --expected " + to_string(*iter) + "\n";
+               } else if (iter == test_notes.end()) {
+                       not_expected.push_back(note);
+               }
        }
-       message += "Expected:\n";
-       for (auto i: test_notes) {
-               message += "  " + note_to_string(i) + "\n";
-               message += dcp::String::compose(
-                       "  [%1 %2 %3 %4 %5]\n",
-                       static_cast<int>(i.type()),
-                       static_cast<int>(i.code()),
-                       i.note().get_value_or("<none>"),
-                       i.file().get_value_or("<none>"),
-                       i.line().get_value_or(0)
-                       );
+
+       vector<dcp::VerificationNote> not_seen;
+       for (auto note: test_notes) {
+               auto iter = std::find_if(notes.begin(), notes.end(), [note](dcp::VerificationNote const& n) { return note.type() == n.type() && note.code() == n.code(); });
+               if (iter == notes.end()) {
+                       not_seen.push_back(note);
+               }
        }
 
-       BOOST_REQUIRE_MESSAGE (notes == test_notes, message);
+       for (auto note: not_expected) {
+               message += "Not expected:\n" + to_string(note) + "\n";
+       }
+
+       for (auto note: not_seen) {
+               message += "Not seen:\n" + to_string(note) + "\n";
+       }
+
+       BOOST_REQUIRE_MESSAGE(notes == test_notes, message);
+}
+
+
+static
+void
+check_verify_result(vector<path> dir, vector<dcp::DecryptedKDM> kdm, vector<dcp::VerificationNote> test_notes)
+{
+       check_verify_result(dcp::verify({dir}, kdm, &stage, &progress, {}, xsd_test).notes, test_notes);
 }
 
 
 /* Copy dcp_test1 to build/test/verify_test{suffix} then edit a file found by the functor 'file',
- * replacing from with to.  Verify the resulting DCP and check that the results match the given
- * list of codes.
+ * replacing from with to.
  */
 static
 void
-check_verify_result_after_replace (string suffix, boost::function<path (string)> file, string from, string to, vector<dcp::VerificationNote::Code> codes)
+replace(string suffix, boost::function<path (string)> file, string from, string to)
 {
        auto dir = setup (1, suffix);
 
@@ -216,17 +320,6 @@ check_verify_result_after_replace (string suffix, boost::function<path (string)>
                Editor e (file(suffix));
                e.replace (from, to);
        }
-
-       auto notes = dcp::verify({dir}, &stage, &progress, {}, xsd_test);
-
-       BOOST_REQUIRE_EQUAL (notes.size(), codes.size());
-       auto i = notes.begin();
-       auto j = codes.begin();
-       while (i != notes.end()) {
-               BOOST_CHECK_EQUAL (i->code(), *j);
-               ++i;
-               ++j;
-       }
 }
 
 
@@ -239,14 +332,69 @@ add_font(shared_ptr<dcp::SubtitleAsset> asset)
 }
 
 
+class HashCalculator
+{
+public:
+       HashCalculator(boost::filesystem::path path)
+               : _path(path)
+               , _old_hash(dcp::make_digest(path, [](int64_t, int64_t) {}))
+       {}
+
+       std::string old_hash() const {
+               return _old_hash;
+       }
+
+       std::string new_hash() const {
+               return dcp::make_digest(_path, [](int64_t, int64_t) {});
+       }
+
+private:
+       boost::filesystem::path _path;
+       std::string _old_hash;
+};
+
+
+static
+dcp::VerificationNote
+ok(dcp::VerificationNote::Code code, shared_ptr<const dcp::CPL> cpl)
+{
+       return dcp::VerificationNote(dcp::VerificationNote::Type::OK, code).set_cpl_id(cpl->id());
+}
+
+
+static
+dcp::VerificationNote
+ok(dcp::VerificationNote::Code code, string note, shared_ptr<const dcp::CPL> cpl)
+{
+       return dcp::VerificationNote(dcp::VerificationNote::Type::OK, code, note).set_cpl_id(cpl->id());
+}
+
+
+static
+dcp::VerificationNote
+ok(dcp::VerificationNote::Code code, boost::filesystem::path path, shared_ptr<const dcp::CPL> cpl)
+{
+       return dcp::VerificationNote(dcp::VerificationNote::Type::OK, code, path).set_cpl_id(cpl->id());
+}
+
+
+void
+add(vector<dcp::VerificationNote>& notes, vector<dcp::VerificationNote> const& add)
+{
+       for (auto i: add) {
+               notes.push_back(i);
+       }
+}
+
+
 BOOST_AUTO_TEST_CASE (verify_no_error)
 {
        stages.clear ();
        auto dir = setup (1, "no_error");
-       auto notes = dcp::verify({dir}, &stage, &progress, {}, xsd_test);
+       auto notes = dcp::verify({dir}, {}, &stage, &progress, {}, xsd_test).notes;
 
-       path const cpl_file = dir / dcp_test1_cpl;
-       path const pkl_file = dir / dcp_test1_pkl;
+       path const cpl_file = dir / dcp_test1_cpl();
+       path const pkl_file = dir / dcp_test1_pkl();
        path const assetmap_file = dir / "ASSETMAP.xml";
 
        auto st = stages.begin();
@@ -280,14 +428,15 @@ BOOST_AUTO_TEST_CASE (verify_no_error)
        BOOST_CHECK_EQUAL (st->first, "Checking PKL");
        BOOST_REQUIRE (st->second);
        BOOST_CHECK_EQUAL (st->second.get(), canonical(pkl_file));
-       ++st;
-       BOOST_CHECK_EQUAL (st->first, "Checking ASSETMAP");
+       ++st; BOOST_CHECK_EQUAL (st->first, "Checking ASSETMAP");
        BOOST_REQUIRE (st->second);
        BOOST_CHECK_EQUAL (st->second.get(), canonical(assetmap_file));
        ++st;
        BOOST_REQUIRE (st == stages.end());
 
-       BOOST_CHECK_EQUAL (notes.size(), 0U);
+       for (auto note: notes) {
+               BOOST_CHECK(note.type() == dcp::VerificationNote::Type::OK);
+       }
 }
 
 
@@ -296,28 +445,47 @@ BOOST_AUTO_TEST_CASE (verify_incorrect_picture_sound_hash)
        using namespace boost::filesystem;
 
        auto dir = setup (1, "incorrect_picture_sound_hash");
+       auto cpl = make_shared<dcp::CPL>(find_cpl(dir));
 
        auto video_path = path(dir / "video.mxf");
+       HashCalculator video_calc(video_path);
        auto mod = fopen(video_path.string().c_str(), "r+b");
        BOOST_REQUIRE (mod);
-       fseek (mod, 4096, SEEK_SET);
+       BOOST_REQUIRE_EQUAL(fseek(mod, -16, SEEK_END), 0);
        int x = 42;
-       fwrite (&x, sizeof(x), 1, mod);
+       BOOST_REQUIRE(fwrite(&x, sizeof(x), 1, mod) == 1);
        fclose (mod);
 
        auto audio_path = path(dir / "audio.mxf");
+       HashCalculator audio_calc(audio_path);
        mod = fopen(audio_path.string().c_str(), "r+b");
        BOOST_REQUIRE (mod);
-       BOOST_REQUIRE_EQUAL (fseek(mod, -64, SEEK_END), 0);
+       BOOST_REQUIRE_EQUAL(fseek(mod, 0, SEEK_END), 0);
        BOOST_REQUIRE (fwrite (&x, sizeof(x), 1, mod) == 1);
        fclose (mod);
 
        dcp::ASDCPErrorSuspender sus;
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INCORRECT_PICTURE_HASH, canonical(video_path) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INCORRECT_SOUND_HASH, canonical(audio_path) },
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(dcp_test1_cpl_id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INCORRECT_PICTURE_HASH, canonical(video_path)
+                               ).set_cpl_id(dcp_test1_cpl_id()).set_reference_hash(video_calc.old_hash()).set_calculated_hash(video_calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INCORRECT_SOUND_HASH, canonical(audio_path)
+                               ).set_cpl_id(dcp_test1_cpl_id()).set_reference_hash(audio_calc.old_hash()).set_calculated_hash(audio_calc.new_hash()),
                });
 }
 
@@ -327,21 +495,41 @@ BOOST_AUTO_TEST_CASE (verify_mismatched_picture_sound_hashes)
        using namespace boost::filesystem;
 
        auto dir = setup (1, "mismatched_picture_sound_hashes");
+       auto cpl = make_shared<dcp::CPL>(find_cpl(dir));
+
+       HashCalculator calc(dir / dcp_test1_cpl());
 
        {
-               Editor e (dir / dcp_test1_pkl);
+               Editor e (dir / dcp_test1_pkl());
                e.replace ("<Hash>", "<Hash>x");
        }
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, dcp_test1_cpl_id, canonical(dir / dcp_test1_cpl) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_PICTURE_HASHES, canonical(dir / "video.mxf") },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_SOUND_HASHES, canonical(dir / "audio.mxf") },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "value 'xKcJb7S2K5cNm8RG4kfQD5FTeS0A=' is invalid Base64-encoded binary", canonical(dir / dcp_test1_pkl), 28 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "value 'xtfX1mVIKJCVr1m7Y32Nzxf0+Rpw=' is invalid Base64-encoded binary", canonical(dir / dcp_test1_pkl), 12 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "value 'xwUmt8G+cFFKMGt0ueS9+F1S4uhc=' is invalid Base64-encoded binary", canonical(dir / dcp_test1_pkl), 20 },
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(dir / dcp_test1_cpl())
+                               ).set_cpl_id(dcp_test1_cpl_id()).set_reference_hash("x" + calc.old_hash()).set_calculated_hash(calc.old_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_PICTURE_HASHES, canonical(dir / "video.mxf")
+                               ).set_cpl_id(dcp_test1_cpl_id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_SOUND_HASHES, canonical(dir / "audio.mxf")
+                               ).set_cpl_id(dcp_test1_cpl_id()),
+                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "value 'x3M7YTgvFKXXMEGLkIbV4miC90FE=' is invalid Base64-encoded binary", canonical(dir / dcp_test1_pkl()), 28 },
+                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "value 'xskI+5b/9LA/y6h0mcyxysJYanxI=' is invalid Base64-encoded binary", canonical(dir / dcp_test1_pkl()), 12 },
+                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "value 'xvsVjRV9vhTBPUWfE/TT1o2vdQsI=' is invalid Base64-encoded binary", canonical(dir / dcp_test1_pkl()), 20 },
                });
 }
 
@@ -350,33 +538,53 @@ BOOST_AUTO_TEST_CASE (verify_failed_read_content_kind)
 {
        auto dir = setup (1, "failed_read_content_kind");
 
+       HashCalculator calc(dir / dcp_test1_cpl());
+
        {
-               Editor e (dir / dcp_test1_cpl);
+               Editor e (dir / dcp_test1_cpl());
                e.replace ("<ContentKind>", "<ContentKind>x");
        }
 
+       auto cpl = std::make_shared<dcp::CPL>(find_cpl(dir));
+
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, dcp_test1_cpl_id, canonical(dir / dcp_test1_cpl) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_CONTENT_KIND, string("xtrailer") }
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(dir / dcp_test1_cpl())
+                               ).set_cpl_id(dcp_test1_cpl_id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_CONTENT_KIND, string("xtrailer")
+                               ).set_cpl_id(dcp_test1_cpl_id())
                });
 }
 
 
 static
 path
-cpl (string suffix)
+dcp_test1_cpl_path(string suffix)
 {
-       return dcp::String::compose("build/test/verify_test%1/%2", suffix, dcp_test1_cpl);
+       return dcp::String::compose("build/test/verify_test%1/%2", suffix, dcp_test1_cpl());
 }
 
 
 static
 path
-pkl (string suffix)
+dcp_test1_pkl_path(string suffix)
 {
-       return dcp::String::compose("build/test/verify_test%1/%2", suffix, dcp_test1_pkl);
+       return dcp::String::compose("build/test/verify_test%1/%2", suffix, dcp_test1_pkl());
 }
 
 
@@ -390,21 +598,59 @@ asset_map (string suffix)
 
 BOOST_AUTO_TEST_CASE (verify_invalid_picture_frame_rate)
 {
-       check_verify_result_after_replace (
-                       "invalid_picture_frame_rate", &cpl,
-                       "<FrameRate>24 1", "<FrameRate>99 1",
-                       { dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES,
-                         dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE }
-                       );
+       auto const suffix = "invalid_picture_frame_rate";
+
+       replace(suffix, &dcp_test1_cpl_path, "<FrameRate>24 1", "<FrameRate>99 1");
+
+       auto const dir = dcp::String::compose("build/test/verify_test%1", suffix);
+       auto const cpl_path = find_cpl(dir);
+       auto cpl = std::make_shared<dcp::CPL>(cpl_path);
+
+       std::vector<dcp::VerificationNote> expected =
+               {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(cpl_path)
+                               ).set_cpl_id(cpl->id()).set_calculated_hash("7n7GQ2TbxQbmHYuAR8ml7XDOep8=").set_reference_hash("skI+5b/9LA/y6h0mcyxysJYanxI="),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE, string{"99/1"}
+                               ).set_cpl_id(cpl->id())
+               };
+
+       check_verify_result(dcp::verify({dir}, {}, &stage, &progress, {}, xsd_test).notes, expected);
 }
 
 BOOST_AUTO_TEST_CASE (verify_missing_asset)
 {
        auto dir = setup (1, "missing_asset");
        remove (dir / "video.mxf");
+
+       auto cpl = std::make_shared<dcp::CPL>(find_cpl(dir));
+
        check_verify_result (
                { dir },
+               {},
                {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
                        { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_ASSET, canonical(dir) / "video.mxf" }
                });
 }
@@ -412,71 +658,224 @@ BOOST_AUTO_TEST_CASE (verify_missing_asset)
 
 BOOST_AUTO_TEST_CASE (verify_empty_asset_path)
 {
-       check_verify_result_after_replace (
-                       "empty_asset_path", &asset_map,
-                       "<Path>video.mxf</Path>", "<Path></Path>",
-                       { dcp::VerificationNote::Code::EMPTY_ASSET_PATH }
-                       );
+       auto const suffix = "empty_asset_path";
+
+       replace("empty_asset_path", &asset_map, "<Path>video.mxf</Path>", "<Path></Path>");
+
+       auto const dir = dcp::String::compose("build/test/verify_test%1", suffix);
+       auto const cpl_path = find_cpl(dir);
+       auto cpl = std::make_shared<dcp::CPL>(cpl_path);
+
+       std::vector<dcp::VerificationNote> expected = {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EMPTY_ASSET_PATH }
+               };
+
+       check_verify_result(dcp::verify({dir}, {}, &stage, &progress, {}, xsd_test).notes, expected);
 }
 
 
 BOOST_AUTO_TEST_CASE (verify_mismatched_standard)
 {
-       check_verify_result_after_replace (
-                       "mismatched_standard", &cpl,
-                       "http://www.smpte-ra.org/schemas/429-7/2006/CPL", "http://www.digicine.com/PROTO-ASDCP-CPL-20040511#",
-                       { dcp::VerificationNote::Code::MISMATCHED_STANDARD,
-                         dcp::VerificationNote::Code::INVALID_XML,
-                         dcp::VerificationNote::Code::INVALID_XML,
-                         dcp::VerificationNote::Code::INVALID_XML,
-                         dcp::VerificationNote::Code::INVALID_XML,
-                         dcp::VerificationNote::Code::INVALID_XML,
-                         dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES }
-                       );
+       auto const suffix = "mismatched_standard";
+
+       replace(suffix, &dcp_test1_cpl_path, "http://www.smpte-ra.org/schemas/429-7/2006/CPL", "http://www.digicine.com/PROTO-ASDCP-CPL-20040511#");
+
+       auto const dir = dcp::String::compose("build/test/verify_test%1", suffix);
+       auto const cpl_path = find_cpl(dir);
+       auto cpl = std::make_shared<dcp::CPL>(cpl_path);
+
+       std::vector<dcp::VerificationNote> expected = {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_STANDARD },
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "invalid character encountered", canonical(cpl_path), 42
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "no declaration found for element 'Id'", canonical(cpl_path), 53
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "no declaration found for element 'EditRate'", canonical(cpl_path), 54
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, "no declaration found for element 'IntrinsicDuration'", canonical(cpl_path), 55
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML,
+                               "element 'Id' is not allowed for content model '(Id,AnnotationText?,EditRate,IntrinsicDuration,"
+                               "EntryPoint?,Duration?,FullContentTitleText,ReleaseTerritory?,VersionNumber?,Chain?,Distributor?,"
+                               "Facility?,AlternateContentVersionList?,Luminance?,MainSoundConfiguration,MainSoundSampleRate,"
+                               "MainPictureStoredArea,MainPictureActiveArea,MainSubtitleLanguageList?,ExtensionMetadataList?,)'",
+                               canonical(cpl_path), 149
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(cpl_path)
+                               ).set_cpl_id(cpl->id()).set_reference_hash("skI+5b/9LA/y6h0mcyxysJYanxI=").set_calculated_hash("FZ9E7L/pOuJ6aZfbiaANTv8BFOo=")
+               };
+
+       check_verify_result(dcp::verify({dir}, {}, &stage, &progress, {}, xsd_test).notes, expected);
 }
 
 
 BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_id)
 {
+       auto const suffix = "invalid_xml_cpl_id";
+
        /* There's no MISMATCHED_CPL_HASHES error here because it can't find the correct hash by ID (since the ID is wrong) */
-       check_verify_result_after_replace (
-                       "invalid_xml_cpl_id", &cpl,
-                       "<Id>urn:uuid:6affb8ee-0020-4dff-a53c-17652f6358ab", "<Id>urn:uuid:6affb8ee-0020-4dff-a53c-17652f6358a",
-                       { dcp::VerificationNote::Code::INVALID_XML }
-                       );
+       replace("invalid_xml_cpl_id", &dcp_test1_cpl_path, "<Id>urn:uuid:6affb8ee-0020-4dff-a53c-17652f6358ab", "<Id>urn:uuid:6affb8ee-0020-4dff-a53c-17652f6358a");
+
+       auto const dir = dcp::String::compose("build/test/verify_test%1", suffix);
+       auto const cpl_path = find_cpl(dir);
+       auto cpl = std::make_shared<dcp::CPL>(cpl_path);
+
+       std::vector<dcp::VerificationNote> expected = {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML,
+                               "value 'urn:uuid:6affb8ee-0020-4dff-a53c-17652f6358a' does not match regular expression "
+                               "facet 'urn:uuid:[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}'", canonical(cpl_path), 3
+                               ).set_cpl_id(cpl->id())
+               };
+
+       check_verify_result(dcp::verify({dir}, {}, &stage, &progress, {}, xsd_test).notes, expected);
 }
 
 
 BOOST_AUTO_TEST_CASE (verify_invalid_xml_issue_date)
 {
-       check_verify_result_after_replace (
-                       "invalid_xml_issue_date", &cpl,
-                       "<IssueDate>", "<IssueDate>x",
-                       { dcp::VerificationNote::Code::INVALID_XML,
-                         dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES }
-                       );
+       auto const suffix = "invalid_xml_issue_date";
+
+       replace("invalid_xml_issue_date", &dcp_test1_cpl_path, "<IssueDate>", "<IssueDate>x");
+
+       auto const dir = dcp::String::compose("build/test/verify_test%1", suffix);
+       auto const cpl_path = find_cpl(dir);
+       auto cpl = std::make_shared<dcp::CPL>(cpl_path);
+
+       std::vector<dcp::VerificationNote> expected = {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(cpl_path)
+                               ).set_cpl_id(cpl->id()).set_reference_hash("skI+5b/9LA/y6h0mcyxysJYanxI=").set_calculated_hash("sz3BeIugJ567q3HMnA62JeRw4TE="),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML,
+                               "invalid character encountered",
+                               canonical(cpl_path), 5
+                               ).set_cpl_id(cpl->id()),
+               };
+
+       check_verify_result(dcp::verify({dir}, {}, &stage, &progress, {}, xsd_test).notes, expected);
 }
 
 
 BOOST_AUTO_TEST_CASE (verify_invalid_xml_pkl_id)
 {
-       check_verify_result_after_replace (
-               "invalid_xml_pkl_id", &pkl,
-               "<Id>urn:uuid:" + dcp_test1_pkl_id.substr(0, 3),
-               "<Id>urn:uuid:x" + dcp_test1_pkl_id.substr(1, 2),
-               { dcp::VerificationNote::Code::INVALID_XML }
-               );
+       auto const suffix = "invalid_xml_pkl_id";
+
+       replace("invalid_xml_pkl_id", &dcp_test1_pkl_path, "<Id>urn:uuid:" + dcp_test1_pkl_id().substr(0, 3), "<Id>urn:uuid:x" + dcp_test1_pkl_id().substr(1, 2));
+
+       auto const dir = dcp::String::compose("build/test/verify_test%1", suffix);
+       auto const pkl_path = find_pkl(dir);
+       auto const cpl_path = find_cpl(dir);
+       auto cpl = std::make_shared<dcp::CPL>(cpl_path);
+
+       std::vector<dcp::VerificationNote> expected = {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML,
+                               "value 'urn:uuid:x199d58b-5ef8-4d49-b270-07e590ccb280' does not match regular "
+                               "expression facet 'urn:uuid:[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}'",
+                               canonical(pkl_path), 3
+                               ),
+       };
+
+       check_verify_result(dcp::verify({dir}, {}, &stage, &progress, {}, xsd_test).notes, expected);
 }
 
 
 BOOST_AUTO_TEST_CASE (verify_invalid_xml_asset_map_id)
 {
-       check_verify_result_after_replace (
-               "invalid_xml_asset_map_id", &asset_map,
-               "<Id>urn:uuid:" + dcp_test1_asset_map_id.substr(0, 3),
-               "<Id>urn:uuid:x" + dcp_test1_asset_map_id.substr(1, 2),
-               { dcp::VerificationNote::Code::INVALID_XML }
-               );
+       auto const suffix = "invalid_xml_asset_map_id";
+
+       replace("invalid_xml_asset_map_id", &asset_map, "<Id>urn:uuid:" + dcp_test1_asset_map_id.substr(0, 3), "<Id>urn:uuid:x" + dcp_test1_asset_map_id.substr(1, 2));
+
+       auto const dir = dcp::String::compose("build/test/verify_test%1", suffix);
+       auto const cpl_path = find_cpl(dir);
+       auto const asset_map_path = find_asset_map(dir);
+       auto cpl = std::make_shared<dcp::CPL>(cpl_path);
+
+       std::vector<dcp::VerificationNote> expected = {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML,
+                               "value 'urn:uuid:x17b3de4-6dda-408d-b19b-6711354b0bc3' does not match regular "
+                               "expression facet 'urn:uuid:[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-[0-9a-fA-F]{12}'",
+                               canonical(asset_map_path), 3
+                               ),
+       };
+
+       check_verify_result(dcp::verify({dir}, {}, &stage, &progress, {}, xsd_test).notes, expected);
 }
 
 
@@ -484,11 +883,12 @@ BOOST_AUTO_TEST_CASE (verify_invalid_standard)
 {
        stages.clear ();
        auto dir = setup (3, "verify_invalid_standard");
-       auto notes = dcp::verify({dir}, &stage, &progress, {}, xsd_test);
+       auto notes = dcp::verify({dir}, {}, &stage, &progress, {}, xsd_test).notes;
 
        path const cpl_file = dir / "cpl_cbfd2bc0-21cf-4a8f-95d8-9cddcbe51296.xml";
        path const pkl_file = dir / "pkl_d87a950c-bd6f-41f6-90cc-56ccd673e131.xml";
        path const assetmap_file = dir / "ASSETMAP";
+       auto cpl = std::make_shared<dcp::CPL>(cpl_file);
 
        auto st = stages.begin();
        BOOST_CHECK_EQUAL (st->first, "Checking DCP");
@@ -528,13 +928,24 @@ BOOST_AUTO_TEST_CASE (verify_invalid_standard)
        ++st;
        BOOST_REQUIRE (st == stages.end());
 
-       BOOST_REQUIRE_EQUAL (notes.size(), 2U);
-       auto i = notes.begin ();
-       BOOST_CHECK_EQUAL (i->type(), dcp::VerificationNote::Type::BV21_ERROR);
-       BOOST_CHECK_EQUAL (i->code(), dcp::VerificationNote::Code::INVALID_STANDARD);
-       ++i;
-       BOOST_CHECK_EQUAL (i->type(), dcp::VerificationNote::Type::BV21_ERROR);
-       BOOST_CHECK_EQUAL (i->code(), dcp::VerificationNote::Code::INVALID_JPEG2000_GUARD_BITS_FOR_2K);
+       vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "j2c_c6035f97-b07d-4e1c-944d-603fc2ddc242.mxf"), cpl),
+               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "j2c_c6035f97-b07d-4e1c-944d-603fc2ddc242.mxf"), cpl)
+       };
+
+       for (int j = 0; j < 24; ++j) {
+               expected.push_back(
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_JPEG2000_GUARD_BITS_FOR_2K, string("2")
+                               ).set_cpl_id(cpl->id())
+               );
+       }
+
+       check_verify_result(notes, expected);
 }
 
 /* DCP with a short asset */
@@ -547,21 +958,41 @@ BOOST_AUTO_TEST_CASE (verify_invalid_duration)
        BOOST_REQUIRE(dcp.cpls().size() == 1);
        auto cpl = dcp.cpls()[0];
 
-       check_verify_result (
-               { dir },
-               {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_DURATION, string("d7576dcb-a361-4139-96b8-267f5f8d7f91") },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_INTRINSIC_DURATION, string("d7576dcb-a361-4139-96b8-267f5f8d7f91") },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_DURATION, string("a2a87f5d-b749-4a7e-8d0c-9d48a4abf626") },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_INTRINSIC_DURATION, string("a2a87f5d-b749-4a7e-8d0c-9d48a4abf626") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_JPEG2000_GUARD_BITS_FOR_2K, string("2") },
+       vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "j2c_d7576dcb-a361-4139-96b8-267f5f8d7f91.mxf"), cpl),
+               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "j2c_d7576dcb-a361-4139-96b8-267f5f8d7f91.mxf"), cpl),
+               { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_DURATION, string("d7576dcb-a361-4139-96b8-267f5f8d7f91")
+                       ).set_cpl_id(cpl->id()),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_INTRINSIC_DURATION, string("d7576dcb-a361-4139-96b8-267f5f8d7f91")
+                       ).set_cpl_id(cpl->id()),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_DURATION, string("a2a87f5d-b749-4a7e-8d0c-9d48a4abf626")
+                       ).set_cpl_id(cpl->id()),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_INTRINSIC_DURATION, string("a2a87f5d-b749-4a7e-8d0c-9d48a4abf626")
+                       ).set_cpl_id(cpl->id()),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::WARNING,
+                       dcp::VerificationNote::Code::EMPTY_CONTENT_VERSION_LABEL_TEXT,
+                       cpl->file().get()
+                       ).set_cpl_id(cpl->id())
+       };
+
+       for (int i = 0; i < 23; ++i) {
+               expected.push_back(
                        dcp::VerificationNote(
-                               dcp::VerificationNote::Type::WARNING,
-                               dcp::VerificationNote::Code::EMPTY_CONTENT_VERSION_LABEL_TEXT,
-                               cpl->file().get()
-                       ).set_id("d74fda30-d5f4-4c5f-870f-ebc089d97eb7")
-               });
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_JPEG2000_GUARD_BITS_FOR_2K, string("2")
+                               ).set_cpl_id(cpl->id())
+                       );
+       }
+
+       check_verify_result({ dir }, {}, expected);
 }
 
 
@@ -600,13 +1031,37 @@ BOOST_AUTO_TEST_CASE (verify_invalid_picture_frame_size_in_bytes)
        prepare_directory (dir);
        auto cpl = dcp_from_frame (oversized_frame, dir);
 
-       check_verify_result (
-               { dir },
-               {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_JPEG2000_CODESTREAM, string("missing marker start byte") },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_SIZE_IN_BYTES, canonical(dir / "pic.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
-               });
+       vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "pic.mxf"), cpl),
+       };
+
+       for (auto i = 0; i < 24; ++i) {
+               expected.push_back(
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_JPEG2000_CODESTREAM, string("missing marker start byte")
+                               ).set_frame(i).set_frame_rate(24).set_cpl_id(cpl->id())
+                       );
+       }
+
+       for (auto i = 0; i < 24; ++i) {
+               expected.push_back(
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_SIZE_IN_BYTES, canonical(dir / "pic.mxf")
+                               ).set_frame(i).set_frame_rate(24).set_cpl_id(cpl->id())
+                       );
+       }
+
+       expected.push_back(
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                       ).set_cpl_id(cpl->id())
+               );
+
+       check_verify_result({ dir }, {}, expected);
 }
 
 
@@ -628,13 +1083,37 @@ BOOST_AUTO_TEST_CASE (verify_nearly_invalid_picture_frame_size_in_bytes)
        prepare_directory (dir);
        auto cpl = dcp_from_frame (oversized_frame, dir);
 
-       check_verify_result (
-               { dir },
-               {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_JPEG2000_CODESTREAM, string("missing marker start byte") },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::NEARLY_INVALID_PICTURE_FRAME_SIZE_IN_BYTES, canonical(dir / "pic.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
-               });
+       vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "pic.mxf"), cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+       };
+
+       for (auto i = 0; i < 24; ++i) {
+               expected.push_back(
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_JPEG2000_CODESTREAM, string("missing marker start byte")
+                               ).set_frame(i).set_frame_rate(24).set_cpl_id(cpl->id())
+                       );
+       }
+
+       for (auto i = 0; i < 24; ++i) {
+               expected.push_back(
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::NEARLY_INVALID_PICTURE_FRAME_SIZE_IN_BYTES, canonical(dir / "pic.mxf")
+                               ).set_frame(i).set_frame_rate(24).set_cpl_id(cpl->id())
+               );
+       }
+
+       expected.push_back(
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                       ).set_cpl_id(cpl->id())
+               );
+
+       check_verify_result ({ dir }, {}, expected);
 }
 
 
@@ -649,7 +1128,18 @@ BOOST_AUTO_TEST_CASE (verify_valid_picture_frame_size_in_bytes)
        prepare_directory (dir);
        auto cpl = dcp_from_frame (frame, dir);
 
-       check_verify_result ({ dir }, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+       check_verify_result(
+               { dir },
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "pic.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "pic.mxf"), cpl),
+                       dcp::VerificationNote(dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -660,12 +1150,19 @@ BOOST_AUTO_TEST_CASE (verify_valid_interop_subtitles)
        copy_file ("test/data/subs1.xml", dir / "subs.xml");
        auto asset = make_shared<dcp::InteropSubtitleAsset>(dir / "subs.xml");
        auto reel_asset = make_shared<dcp::ReelInteropSubtitleAsset>(asset, dcp::Fraction(24, 1), 16 * 24, 0);
-       write_dcp_with_single_asset (dir, reel_asset, dcp::Standard::INTEROP);
+       auto cpl = write_dcp_with_single_asset(dir, reel_asset, dcp::Standard::INTEROP);
 
        check_verify_result (
-               {dir}, {
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
                        { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"theFontId"} }
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"theFontId"}
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -677,12 +1174,19 @@ BOOST_AUTO_TEST_CASE(verify_catch_missing_font_file_with_interop_ccap)
        copy_file("test/data/subs1.xml", dir / "ccap.xml");
        auto asset = make_shared<dcp::InteropSubtitleAsset>(dir / "ccap.xml");
        auto reel_asset = make_shared<dcp::ReelInteropClosedCaptionAsset>(asset, dcp::Fraction(24, 1), 16 * 24, 0);
-       write_dcp_with_single_asset(dir, reel_asset, dcp::Standard::INTEROP);
+       auto cpl = write_dcp_with_single_asset(dir, reel_asset, dcp::Standard::INTEROP);
 
        check_verify_result (
-               {dir}, {
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
                        { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"theFontId"} }
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"theFontId"}
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -696,7 +1200,7 @@ BOOST_AUTO_TEST_CASE (verify_invalid_interop_subtitles)
        copy_file ("test/data/subs1.xml", dir / "subs.xml");
        auto asset = make_shared<dcp::InteropSubtitleAsset>(dir / "subs.xml");
        auto reel_asset = make_shared<dcp::ReelInteropSubtitleAsset>(asset, dcp::Fraction(24, 1), 16 * 24, 0);
-       write_dcp_with_single_asset (dir, reel_asset, dcp::Standard::INTEROP);
+       auto cpl = write_dcp_with_single_asset(dir, reel_asset, dcp::Standard::INTEROP);
 
        {
                Editor e (dir / "subs.xml");
@@ -705,17 +1209,25 @@ BOOST_AUTO_TEST_CASE (verify_invalid_interop_subtitles)
 
        check_verify_result (
                { dir },
+               {},
                {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
                        { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'Foo'"), path(), 5 },
-                       {
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'Foo'"), path(), 5
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
                                dcp::VerificationNote::Type::ERROR,
                                dcp::VerificationNote::Code::INVALID_XML,
                                string("element 'Foo' is not allowed for content model '(SubtitleID,MovieTitle,ReelNumber,Language,LoadFont*,Font*,Subtitle*)'"),
                                path(),
                                29
-                       },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"theFontId"} }
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"theFontId"}
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -727,14 +1239,22 @@ BOOST_AUTO_TEST_CASE(verify_interop_subtitle_asset_with_no_subtitles)
        copy_file("test/data/subs4.xml", dir / "subs.xml");
        auto asset = make_shared<dcp::InteropSubtitleAsset>(dir / "subs.xml");
        auto reel_asset = make_shared<dcp::ReelInteropSubtitleAsset>(asset, dcp::Fraction(24, 1), 16 * 24, 0);
-       write_dcp_with_single_asset(dir, reel_asset, dcp::Standard::INTEROP);
+       auto cpl = write_dcp_with_single_asset(dir, reel_asset, dcp::Standard::INTEROP);
 
        check_verify_result (
                { dir },
+               {},
                {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
                        { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE, asset->id(), boost::filesystem::canonical(asset->file().get()) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"theFontId"} }
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE, asset->id(), boost::filesystem::canonical(asset->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"theFontId"}
+                               ).set_cpl_id(cpl->id())
                });
 
 }
@@ -747,13 +1267,19 @@ BOOST_AUTO_TEST_CASE(verify_interop_subtitle_asset_with_single_space_subtitle)
        copy_file("test/data/subs5.xml", dir / "subs.xml");
        auto asset = make_shared<dcp::InteropSubtitleAsset>(dir / "subs.xml");
        auto reel_asset = make_shared<dcp::ReelInteropSubtitleAsset>(asset, dcp::Fraction(24, 1), 16 * 24, 0);
-       write_dcp_with_single_asset(dir, reel_asset, dcp::Standard::INTEROP);
+       auto cpl = write_dcp_with_single_asset(dir, reel_asset, dcp::Standard::INTEROP);
 
        check_verify_result (
                { dir },
+               {},
                {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
                        { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"Arial"} }
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"Arial"}
+                               ).set_cpl_id(cpl->id())
                });
 
 }
@@ -770,10 +1296,21 @@ BOOST_AUTO_TEST_CASE (verify_valid_smpte_subtitles)
 
        check_verify_result(
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE, string{"2021-04-14T13:19:14.000+02:00"} },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT, asset->id() }
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE, string{"2021-04-14T13:19:14.000+02:00"}
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT, asset->id()
+                               ).set_cpl_id(cpl->id()),
                });
 }
 
@@ -792,19 +1329,34 @@ BOOST_AUTO_TEST_CASE (verify_invalid_smpte_subtitles)
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'Foo'"), path(), 2 },
-                       {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'Foo'"), path(), 2
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
                                dcp::VerificationNote::Type::ERROR,
                                dcp::VerificationNote::Code::INVALID_XML,
                                string("element 'Foo' is not allowed for content model '(Id,ContentTitleText,AnnotationText?,IssueDate,ReelNumber?,Language?,EditRate,TimeCodeRate,StartTime?,DisplayType?,LoadFont*,SubtitleList)'"),
                                path(),
                                2
-                       },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(dir / "subs.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE, string{"2020-05-09T00:29:21.000+02:00"} },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT, asset->id() }
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(dir / "subs.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE, string{"2020-05-09T00:29:21.000+02:00"}
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT, asset->id()
+                               ).set_cpl_id(cpl->id()),
                });
 }
 
@@ -820,13 +1372,30 @@ BOOST_AUTO_TEST_CASE (verify_empty_text_node_in_subtitles)
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EMPTY_TEXT },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE, canonical(dir / "subs.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE, string{"2021-08-09T18:34:46.000+02:00"} },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT, asset->id() }
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EMPTY_TEXT
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE, canonical(dir / "subs.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE, string{"2021-08-09T18:34:46.000+02:00"}
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT, asset->id()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -843,9 +1412,15 @@ BOOST_AUTO_TEST_CASE (verify_empty_text_node_in_subtitles_with_child_nodes)
 
        check_verify_result (
                { dir },
+               {},
                {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
                        { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"font0"} }
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"font0"}
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -862,11 +1437,21 @@ BOOST_AUTO_TEST_CASE (verify_empty_text_node_in_subtitles_with_empty_child_nodes
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE, asset->id(), boost::filesystem::canonical(asset->file().get()) },
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE, asset->id(), boost::filesystem::canonical(asset->file().get())
+                               ).set_cpl_id(cpl->id()),
                        { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EMPTY_TEXT },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"font0"} },
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EMPTY_TEXT
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_FONT, string{"font0"}
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -892,9 +1477,16 @@ BOOST_AUTO_TEST_CASE (verify_external_asset)
 
        check_verify_result (
                { vf_dir },
+               {},
                {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
                        { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EXTERNAL_ASSET, picture->asset()->id() },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -929,38 +1521,6 @@ BOOST_AUTO_TEST_CASE (verify_valid_cpl_metadata)
 }
 
 
-path
-find_prefix(path dir, string prefix)
-{
-       auto iter = std::find_if(directory_iterator(dir), directory_iterator(), [prefix](path const& p) {
-               return boost::starts_with(p.filename().string(), prefix);
-       });
-
-       BOOST_REQUIRE(iter != directory_iterator());
-       return iter->path();
-}
-
-
-path find_cpl (path dir)
-{
-       return find_prefix(dir, "cpl_");
-}
-
-
-path
-find_pkl(path dir)
-{
-       return find_prefix(dir, "pkl_");
-}
-
-
-path
-find_asset_map(path dir)
-{
-       return find_prefix(dir, "ASSETMAP");
-}
-
-
 /* DCP with invalid CompositionMetadataAsset */
 BOOST_AUTO_TEST_CASE (verify_invalid_cpl_metadata_bad_tag)
 {
@@ -986,6 +1546,8 @@ BOOST_AUTO_TEST_CASE (verify_invalid_cpl_metadata_bad_tag)
        dcp.set_annotation_text("hello");
        dcp.write_xml();
 
+       HashCalculator calc(find_cpl(dir));
+
        {
                Editor e (find_cpl(dir));
                e.replace ("MainSound", "MainSoundX");
@@ -993,10 +1555,26 @@ BOOST_AUTO_TEST_CASE (verify_invalid_cpl_metadata_bad_tag)
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:MainSoundXConfiguration'"), canonical(cpl->file().get()), 50 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:MainSoundXSampleRate'"), canonical(cpl->file().get()), 51 },
-                       {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "pic.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1440x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "pic.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:MainSoundXConfiguration'"), canonical(cpl->file().get()), 50
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:MainSoundXSampleRate'"), canonical(cpl->file().get()), 51
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
                                dcp::VerificationNote::Type::ERROR,
                                dcp::VerificationNote::Code::INVALID_XML,
                                string("element 'meta:MainSoundXConfiguration' is not allowed for content model "
@@ -1006,9 +1584,10 @@ BOOST_AUTO_TEST_CASE (verify_invalid_cpl_metadata_bad_tag)
                                       "MainSoundSampleRate,MainPictureStoredArea,MainPictureActiveArea,MainSubtitleLanguageList?,"
                                       "ExtensionMetadataList?,)'"),
                                canonical(cpl->file().get()),
-                               71
-                       },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), canonical(cpl->file().get()) },
+                               71).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash())
                });
 }
 
@@ -1040,6 +1619,7 @@ BOOST_AUTO_TEST_CASE (verify_invalid_cpl_metadata_missing_tag)
 
        check_verify_result (
                { dir },
+               {},
                {{ dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::FAILED_READ, string("missing XML tag Width in MainPictureStoredArea") }}
                );
 }
@@ -1059,10 +1639,21 @@ BOOST_AUTO_TEST_CASE (verify_invalid_language1)
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("badlang") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("wrong-andbad") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("badlang")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("wrong-andbad")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1082,10 +1673,21 @@ BOOST_AUTO_TEST_CASE (verify_invalid_language2)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("badlang") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("wrong-andbad") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("badlang")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("wrong-andbad")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1124,17 +1726,38 @@ BOOST_AUTO_TEST_CASE (verify_invalid_language3)
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("this-is-wrong") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("andso-is-this") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("fred-jim") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("frobozz") },
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "videofoo.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1440x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "videofoo.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("this-is-wrong")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("andso-is-this")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("fred-jim")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_LANGUAGE, string("frobozz")
+                               ).set_cpl_id(cpl->id()),
                });
 }
 
 
 static
-vector<dcp::VerificationNote>
+std::tuple<vector<dcp::VerificationNote>, shared_ptr<dcp::CPL>, boost::filesystem::path>
 check_picture_size (int width, int height, int frame_rate, bool three_d)
 {
        using namespace boost::filesystem;
@@ -1184,7 +1807,8 @@ check_picture_size (int width, int height, int frame_rate, bool three_d)
        d->set_annotation_text("A Test DCP");
        d->write_xml();
 
-       return dcp::verify({dcp_path}, &stage, &progress, {}, xsd_test);
+       /* It seems that for the Ubuntu 16.04 compiler we can't use an initializer list here */
+       return std::tuple<vector<dcp::VerificationNote>, shared_ptr<dcp::CPL>, boost::filesystem::path>{ dcp::verify({dcp_path}, {}, &stage, &progress, {}, xsd_test).notes, cpl, dcp_path };
 }
 
 
@@ -1192,8 +1816,26 @@ static
 void
 check_picture_size_ok (int width, int height, int frame_rate, bool three_d)
 {
-       auto notes = check_picture_size(width, height, frame_rate, three_d);
-       BOOST_CHECK_EQUAL (notes.size(), 0U);
+       vector<dcp::VerificationNote> notes;
+       shared_ptr<dcp::CPL> cpl;
+       boost::filesystem::path dir;
+       std::tie(notes, cpl, dir) = check_picture_size(width, height, frame_rate, three_d);
+
+       std::vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::OK,
+                       dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                       dcp::String::compose("%1x%2", width, height),
+                       cpl->file().get()
+                       ).set_cpl_id(cpl->id()),
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl)
+       };
+       check_verify_result(notes, expected);
 }
 
 
@@ -1201,10 +1843,29 @@ static
 void
 check_picture_size_bad_frame_size (int width, int height, int frame_rate, bool three_d)
 {
-       auto notes = check_picture_size(width, height, frame_rate, three_d);
-       BOOST_REQUIRE_EQUAL (notes.size(), 1U);
-       BOOST_CHECK_EQUAL (notes.front().type(), dcp::VerificationNote::Type::BV21_ERROR);
-       BOOST_CHECK_EQUAL (notes.front().code(), dcp::VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS);
+       vector<dcp::VerificationNote> notes;
+       shared_ptr<dcp::CPL> cpl;
+       boost::filesystem::path dir;
+       std::tie(notes, cpl, dir) = check_picture_size(width, height, frame_rate, three_d);
+
+       std::vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::OK,
+                       dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                       dcp::String::compose("%1x%2", width, height),
+                       cpl->file().get()
+                       ).set_cpl_id(cpl->id()),
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS, dcp::String::compose("%1x%2", width, height), canonical(dir / "video.mxf")
+                       ).set_cpl_id(cpl->id())
+       };
+       check_verify_result(notes, expected);
 }
 
 
@@ -1212,10 +1873,33 @@ static
 void
 check_picture_size_bad_2k_frame_rate (int width, int height, int frame_rate, bool three_d)
 {
-       auto notes = check_picture_size(width, height, frame_rate, three_d);
-       BOOST_REQUIRE_EQUAL (notes.size(), 2U);
-       BOOST_CHECK_EQUAL (notes.back().type(), dcp::VerificationNote::Type::BV21_ERROR);
-       BOOST_CHECK_EQUAL (notes.back().code(), dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_2K);
+       vector<dcp::VerificationNote> notes;
+       shared_ptr<dcp::CPL> cpl;
+       boost::filesystem::path dir;
+       std::tie(notes, cpl, dir) = check_picture_size(width, height, frame_rate, three_d);
+
+       std::vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::OK,
+                       dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                       dcp::String::compose("%1x%2", width, height),
+                       cpl->file().get()
+                       ).set_cpl_id(cpl->id()),
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE, dcp::String::compose("%1/1", frame_rate * (three_d ? 2 : 1))
+                       ).set_cpl_id(cpl->id()),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_2K, dcp::String::compose("%1/1", frame_rate), canonical(dir / "video.mxf")
+                       ).set_cpl_id(cpl->id())
+       };
+
+       check_verify_result(notes, expected);
 }
 
 
@@ -1223,10 +1907,30 @@ static
 void
 check_picture_size_bad_4k_frame_rate (int width, int height, int frame_rate, bool three_d)
 {
-       auto notes = check_picture_size(width, height, frame_rate, three_d);
-       BOOST_REQUIRE_EQUAL (notes.size(), 1U);
-       BOOST_CHECK_EQUAL (notes.front().type(), dcp::VerificationNote::Type::BV21_ERROR);
-       BOOST_CHECK_EQUAL (notes.front().code(), dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_4K);
+       vector<dcp::VerificationNote> notes;
+       shared_ptr<dcp::CPL> cpl;
+       boost::filesystem::path dir;
+       std::tie(notes, cpl, dir) = check_picture_size(width, height, frame_rate, three_d);
+
+       std::vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::OK,
+                       dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                       dcp::String::compose("%1x%2", width, height),
+                       cpl->file().get()
+                       ).set_cpl_id(cpl->id()),
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_4K, dcp::String::compose("%1/1", frame_rate), canonical(dir / "video.mxf")
+                       ).set_cpl_id(cpl->id())
+       };
+
+       check_verify_result(notes, expected);
 }
 
 
@@ -1272,10 +1976,26 @@ BOOST_AUTO_TEST_CASE (verify_picture_size)
        check_picture_size_bad_4k_frame_rate (3996, 2160, 48, false);
 
        /* No 4K 3D */
-       auto notes = check_picture_size(3996, 2160, 24, true);
-       BOOST_REQUIRE_EQUAL (notes.size(), 1U);
-       BOOST_CHECK_EQUAL (notes.front().type(), dcp::VerificationNote::Type::BV21_ERROR);
-       BOOST_CHECK_EQUAL (notes.front().code(), dcp::VerificationNote::Code::INVALID_PICTURE_ASSET_RESOLUTION_FOR_3D);
+       vector<dcp::VerificationNote> notes;
+       shared_ptr<dcp::CPL> cpl;
+       boost::filesystem::path dir;
+       std::tie(notes, cpl, dir) = check_picture_size(3996, 2160, 24, true);
+
+       std::vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::OK,
+                       dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                       string{"3996x2160"},
+                       cpl->file().get()
+                       ).set_cpl_id(cpl->id()),
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+               { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_ASSET_RESOLUTION_FOR_3D },
+       };
 }
 
 
@@ -1305,7 +2025,8 @@ add_test_subtitle (shared_ptr<dcp::SubtitleAsset> asset, int start_frame, int en
                        dcp::Colour(),
                        dcp::Time(),
                        dcp::Time(),
-                       0
+                       0,
+                       std::vector<dcp::Ruby>()
                )
        );
 }
@@ -1328,16 +2049,27 @@ BOOST_AUTO_TEST_CASE (verify_invalid_closed_caption_xml_size_in_bytes)
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(dir / "subs.mxf") },
-                       {
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(dir / "subs.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
                                dcp::VerificationNote::Type::BV21_ERROR,
                                dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_XML_SIZE_IN_BYTES,
                                string("419371"),
                                canonical(dir / "subs.mxf")
-                       },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1372,12 +2104,30 @@ verify_timed_text_asset_too_large (string name)
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_TIMED_TEXT_SIZE_IN_BYTES, string("121695488"), canonical(dir / "subs.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_TIMED_TEXT_FONT_SIZE_IN_BYTES, string("121634816"), canonical(dir / "subs.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(dir / "subs.mxf") },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_TIMED_TEXT_SIZE_IN_BYTES, string("121698284"), canonical(dir / "subs.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR,
+                               dcp::VerificationNote::Code::INVALID_TIMED_TEXT_FONT_SIZE_IN_BYTES,
+                               dcp::raw_convert<string>(121634816),
+                               canonical(dir / "subs.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(dir / "subs.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1424,14 +2174,32 @@ BOOST_AUTO_TEST_CASE (verify_missing_subtitle_language)
        subs->write (dir / "subs.mxf");
 
        auto reel_subs = make_shared<dcp::ReelSMPTESubtitleAsset>(subs, dcp::Fraction(24, 1), 106, 0);
-       dcp->cpls()[0]->reels()[0]->add(reel_subs);
+       auto cpl = dcp->cpls()[0];
+       cpl->reels()[0]->add(reel_subs);
        dcp->write_xml();
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE, canonical(dir / "subs.mxf") },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME }
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE, canonical(dir / "subs.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1467,10 +2235,31 @@ BOOST_AUTO_TEST_CASE (verify_mismatched_subtitle_languages)
 
        check_verify_result (
                { path },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(path / "subs1.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(path / "subs2.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_SUBTITLE_LANGUAGES }
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(path / "video0.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(path / "video1.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(path / "video0.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(path / "video1.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(path / "subs1.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(path / "subs2.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_SUBTITLE_LANGUAGES
+                               ).set_cpl_id(cpl->id()),
                });
 }
 
@@ -1506,9 +2295,28 @@ BOOST_AUTO_TEST_CASE (verify_multiple_closed_caption_languages_allowed)
 
        check_verify_result (
                { path },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(path / "subs1.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(path / "subs2.mxf") }
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(path / "video0.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(path / "video1.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(path / "video0.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(path / "video1.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(path / "subs1.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(path / "subs2.mxf")
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1548,14 +2356,32 @@ BOOST_AUTO_TEST_CASE (verify_missing_subtitle_start_time)
        subs->write (dir / "subs.mxf");
 
        auto reel_subs = make_shared<dcp::ReelSMPTESubtitleAsset>(subs, dcp::Fraction(24, 1), 106, 0);
-       dcp->cpls()[0]->reels()[0]->add(reel_subs);
+       auto cpl = dcp->cpls()[0];
+       cpl->reels()[0]->add(reel_subs);
        dcp->write_xml();
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(dir / "subs.mxf") },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME }
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_START_TIME, canonical(dir / "subs.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1596,14 +2422,32 @@ BOOST_AUTO_TEST_CASE (verify_invalid_subtitle_start_time)
        subs->write (dir / "subs.mxf");
 
        auto reel_subs = make_shared<dcp::ReelSMPTESubtitleAsset>(subs, dcp::Fraction(24, 1), 106, 0);
-       dcp->cpls().front()->reels().front()->add(reel_subs);
+       auto cpl = dcp->cpls()[0];
+       cpl->reels().front()->add(reel_subs);
        dcp->write_xml();
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_SUBTITLE_START_TIME, canonical(dir / "subs.mxf") },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_SUBTITLE_START_TIME, canonical(dir / "subs.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1629,7 +2473,7 @@ public:
 
 template <class T>
 shared_ptr<dcp::CPL>
-dcp_with_text (path dir, vector<TestText> subs)
+dcp_with_text(path dir, vector<TestText> subs, optional<dcp::Key> key = boost::none, optional<string> key_id = boost::none)
 {
        prepare_directory (dir);
        auto asset = make_shared<dcp::SMPTESubtitleAsset>();
@@ -1638,6 +2482,10 @@ dcp_with_text (path dir, vector<TestText> subs)
                add_test_subtitle (asset, i.in, i.out, i.v_position, i.v_align, i.text);
        }
        asset->set_language (dcp::LanguageTag("de-DE"));
+       if (key && key_id) {
+               asset->set_key(*key);
+               asset->set_key_id(*key_id);
+       }
        add_font(asset);
        asset->write (dir / "subs.mxf");
 
@@ -1690,9 +2538,18 @@ BOOST_AUTO_TEST_CASE (verify_invalid_subtitle_first_text_time)
        auto cpl = dcp_with_text<dcp::ReelSMPTESubtitleAsset> (dir, {{ 4 * 24 - 1, 5 * 24 }});
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 
 }
@@ -1703,7 +2560,18 @@ BOOST_AUTO_TEST_CASE (verify_valid_subtitle_first_text_time)
        auto const dir = path("build/test/verify_valid_subtitle_first_text_time");
        /* Just late enough */
        auto cpl = dcp_with_text<dcp::ReelSMPTESubtitleAsset> (dir, {{ 4 * 24, 5 * 24 }});
-       check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -1748,7 +2616,18 @@ BOOST_AUTO_TEST_CASE (verify_valid_subtitle_first_text_time_on_second_reel)
        dcp->set_annotation_text("hello");
        dcp->write_xml();
 
-       check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -1763,9 +2642,18 @@ BOOST_AUTO_TEST_CASE (verify_invalid_subtitle_spacing)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_SPACING },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_SPACING
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1779,7 +2667,19 @@ BOOST_AUTO_TEST_CASE (verify_valid_subtitle_spacing)
                        { 4 * 24,      5 * 24 },
                        { 5 * 24 + 16, 8 * 24 },
                });
-       check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -1789,9 +2689,18 @@ BOOST_AUTO_TEST_CASE (verify_invalid_subtitle_duration)
        auto cpl = dcp_with_text<dcp::ReelSMPTESubtitleAsset> (dir, {{ 4 * 24, 4 * 24 + 1 }});
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_DURATION },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_DURATION
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1800,7 +2709,19 @@ BOOST_AUTO_TEST_CASE (verify_valid_subtitle_duration)
 {
        auto const dir = path("build/test/verify_valid_subtitle_duration");
        auto cpl = dcp_with_text<dcp::ReelSMPTESubtitleAsset> (dir, {{ 4 * 24, 4 * 24 + 17 }});
-       check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -1819,11 +2740,24 @@ BOOST_AUTO_TEST_CASE (verify_subtitle_overlapping_reel_boundary)
        auto cpl = write_dcp_with_single_asset (dir, reel_asset);
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_DURATION , "72 96", boost::filesystem::canonical(asset->file().get()) },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::SUBTITLE_OVERLAPS_REEL_BOUNDARY },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_DURATION , "72 96", boost::filesystem::canonical(asset->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::SUBTITLE_OVERLAPS_REEL_BOUNDARY
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 
 }
@@ -1842,9 +2776,18 @@ BOOST_AUTO_TEST_CASE (verify_invalid_subtitle_line_count1)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_COUNT },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_COUNT
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1859,7 +2802,19 @@ BOOST_AUTO_TEST_CASE (verify_valid_subtitle_line_count1)
                        { 96, 200, 0.1, dcp::VAlign::CENTER, "have" },
                        { 96, 200, 0.2, dcp::VAlign::CENTER, "four" },
                });
-       check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -1876,9 +2831,18 @@ BOOST_AUTO_TEST_CASE (verify_invalid_subtitle_line_count2)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_COUNT },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_COUNT
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1894,7 +2858,19 @@ BOOST_AUTO_TEST_CASE (verify_valid_subtitle_line_count2)
                        { 150, 180, 0.2, dcp::VAlign::CENTER, "four" },
                        { 190, 250, 0.3, dcp::VAlign::CENTER, "lines" }
                });
-       check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -1908,9 +2884,18 @@ BOOST_AUTO_TEST_CASE (verify_invalid_subtitle_line_length1)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::NEARLY_INVALID_SUBTITLE_LINE_LENGTH },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::NEARLY_INVALID_SUBTITLE_LINE_LENGTH
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1925,9 +2910,18 @@ BOOST_AUTO_TEST_CASE (verify_invalid_subtitle_line_length2)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_LENGTH },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_LINE_LENGTH
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1945,9 +2939,18 @@ BOOST_AUTO_TEST_CASE (verify_valid_closed_caption_line_count1)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_COUNT},
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_COUNT
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1962,7 +2965,19 @@ BOOST_AUTO_TEST_CASE (verify_valid_closed_caption_line_count2)
                        { 96, 200, 0.1, dcp::VAlign::CENTER, "have" },
                        { 96, 200, 0.2, dcp::VAlign::CENTER, "four" },
                });
-       check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -1979,9 +2994,18 @@ BOOST_AUTO_TEST_CASE (verify_invalid_closed_caption_line_count3)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_COUNT},
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_COUNT
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -1997,7 +3021,19 @@ BOOST_AUTO_TEST_CASE (verify_valid_closed_caption_line_count4)
                        { 150, 180, 0.2, dcp::VAlign::CENTER, "four" },
                        { 190, 250, 0.3, dcp::VAlign::CENTER, "lines" }
                });
-       check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -2009,10 +3045,18 @@ BOOST_AUTO_TEST_CASE (verify_valid_closed_caption_line_length)
                {
                        { 96, 300, 0.0, dcp::VAlign::CENTER, "01234567890123456789012345678901" }
                });
+
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2027,9 +3071,18 @@ BOOST_AUTO_TEST_CASE (verify_invalid_closed_caption_line_length)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_LENGTH },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_LENGTH
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2046,8 +3099,15 @@ BOOST_AUTO_TEST_CASE (verify_mismatched_closed_caption_valign1)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2064,9 +3124,18 @@ BOOST_AUTO_TEST_CASE (verify_mismatched_closed_caption_valign2)
                });
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CLOSED_CAPTION_VALIGN },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CLOSED_CAPTION_VALIGN
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2081,10 +3150,18 @@ BOOST_AUTO_TEST_CASE (verify_incorrect_closed_caption_ordering1)
                        { 96, 300, 0.1, dcp::VAlign::TOP, "is" },
                        { 96, 300, 0.2, dcp::VAlign::TOP, "fine" },
                });
-       check_verify_result (
+
+       check_verify_result(
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2099,10 +3176,18 @@ BOOST_AUTO_TEST_CASE (verify_incorrect_closed_caption_ordering2)
                        { 96, 300, 0.1, dcp::VAlign::BOTTOM, "is" },
                        { 96, 300, 0.0, dcp::VAlign::BOTTOM, "also fine" },
                });
-       check_verify_result (
+
+       check_verify_result(
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2113,9 +3198,18 @@ BOOST_AUTO_TEST_CASE (verify_incorrect_closed_caption_ordering3)
        auto cpl = dcp_with_text_from_file<dcp::ReelSMPTEClosedCaptionAsset> (dir, "test/data/verify_incorrect_closed_caption_ordering3.xml");
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INCORRECT_CLOSED_CAPTION_ORDERING },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INCORRECT_CLOSED_CAPTION_ORDERING
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2124,10 +3218,18 @@ BOOST_AUTO_TEST_CASE (verify_incorrect_closed_caption_ordering4)
 {
        auto const dir = path ("build/test/verify_incorrect_closed_caption_ordering4");
        auto cpl = dcp_with_text_from_file<dcp::ReelSMPTEClosedCaptionAsset> (dir, "test/data/verify_incorrect_closed_caption_ordering4.xml");
-       check_verify_result (
+
+       check_verify_result(
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2155,9 +3257,20 @@ BOOST_AUTO_TEST_CASE (verify_invalid_sound_frame_rate)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_SOUND_FRAME_RATE, string("96000"), canonical(dir / "audiofoo.mxf") },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "videofoo.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "videofoo.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_SOUND_FRAME_RATE, string("96000"), canonical(dir / "audiofoo.mxf")
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2172,6 +3285,8 @@ BOOST_AUTO_TEST_CASE (verify_missing_cpl_annotation_text)
 
        auto const cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                BOOST_REQUIRE (cpl->file());
                Editor e(cpl->file().get());
@@ -2180,9 +3295,24 @@ BOOST_AUTO_TEST_CASE (verify_missing_cpl_annotation_text)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_ANNOTATION_TEXT, cpl->id(), canonical(cpl->file().get()) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), canonical(cpl->file().get()) }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_ANNOTATION_TEXT, canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash())
                });
 }
 
@@ -2196,6 +3326,8 @@ BOOST_AUTO_TEST_CASE (verify_mismatched_cpl_annotation_text)
        BOOST_REQUIRE_EQUAL (dcp->cpls().size(), 1U);
        auto const cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                BOOST_REQUIRE (cpl->file());
                Editor e(cpl->file().get());
@@ -2204,9 +3336,24 @@ BOOST_AUTO_TEST_CASE (verify_mismatched_cpl_annotation_text)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISMATCHED_CPL_ANNOTATION_TEXT, cpl->id(), canonical(cpl->file().get()) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), canonical(cpl->file().get()) }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISMATCHED_CPL_ANNOTATION_TEXT, canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(cpl->file().get())
+                               ).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()).set_cpl_id(cpl->id())
                });
 }
 
@@ -2235,9 +3382,20 @@ BOOST_AUTO_TEST_CASE (verify_mismatched_asset_duration)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_ASSET_DURATION },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), canonical(cpl->file().get()) }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_ASSET_DURATION
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2306,9 +3464,22 @@ BOOST_AUTO_TEST_CASE (verify_missing_main_subtitle_from_some_reels)
                auto cpl = verify_subtitles_must_be_in_all_reels_check (dir, true, false);
                check_verify_result (
                        { dir },
+                       {},
                        {
-                               { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_MAIN_SUBTITLE_FROM_SOME_REELS },
-                               { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
+                               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video2.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video2.mxf"), cpl),
+                               dcp::VerificationNote(
+                                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_MAIN_SUBTITLE_FROM_SOME_REELS
+                                       ).set_cpl_id(cpl->id()),
+                               dcp::VerificationNote(
+                                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                                       ).set_cpl_id(cpl->id())
                        });
 
        }
@@ -2316,13 +3487,43 @@ BOOST_AUTO_TEST_CASE (verify_missing_main_subtitle_from_some_reels)
        {
                path dir ("build/test/verify_subtitles_must_be_in_all_reels2");
                auto cpl = verify_subtitles_must_be_in_all_reels_check (dir, true, true);
-               check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+               check_verify_result(
+                       {dir},
+                       {},
+                       {
+                               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video2.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video2.mxf"), cpl),
+                               dcp::VerificationNote(
+                                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                                       ).set_cpl_id(cpl->id())
+                       });
        }
 
        {
                path dir ("build/test/verify_subtitles_must_be_in_all_reels1");
                auto cpl = verify_subtitles_must_be_in_all_reels_check (dir, false, false);
-               check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+               check_verify_result(
+                       {dir},
+                       {},
+                       {
+                               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video2.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video2.mxf"), cpl),
+                               dcp::VerificationNote(
+                                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                                       ).set_cpl_id(cpl->id())
+                       });
        }
 }
 
@@ -2389,22 +3590,65 @@ BOOST_AUTO_TEST_CASE (verify_mismatched_closed_caption_asset_counts)
                auto cpl = verify_closed_captions_must_be_in_all_reels_check (dir, 3, 4);
                check_verify_result (
                        {dir},
+                       {},
                        {
-                               { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_CLOSED_CAPTION_ASSET_COUNTS },
-                               { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video2.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video2.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                               dcp::VerificationNote(
+                                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_CLOSED_CAPTION_ASSET_COUNTS
+                                       ).set_cpl_id(cpl->id()),
+                               dcp::VerificationNote(
+                                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                                       ).set_cpl_id(cpl->id())
                        });
        }
 
        {
                path dir ("build/test/verify_closed_captions_must_be_in_all_reels2");
                auto cpl = verify_closed_captions_must_be_in_all_reels_check (dir, 4, 4);
-               check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+               check_verify_result(
+                       {dir},
+                       {},
+                       {
+                               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video2.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video2.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                               dcp::VerificationNote(
+                                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                                       ).set_cpl_id(cpl->id())
+                       });
        }
 
        {
                path dir ("build/test/verify_closed_captions_must_be_in_all_reels3");
                auto cpl = verify_closed_captions_must_be_in_all_reels_check (dir, 0, 0);
-               check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }});
+               check_verify_result(
+                       {dir},
+                       {},
+                       {
+                               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video2.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video1.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video2.mxf"), cpl),
+                               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                               dcp::VerificationNote(
+                                       dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                                       ).set_cpl_id(cpl->id())
+                       });
        }
 }
 
@@ -2445,9 +3689,20 @@ verify_text_entry_point_check (path dir, dcp::VerificationNote::Code code, boost
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, code, subs->id() },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, code, subs->id()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2502,6 +3757,8 @@ BOOST_AUTO_TEST_CASE (verify_missing_hash)
        BOOST_REQUIRE (cpl->reels()[0]->main_picture());
        auto asset_id = cpl->reels()[0]->main_picture()->id();
 
+       HashCalculator calc(cpl->file().get());
+
        {
                BOOST_REQUIRE (cpl->file());
                Editor e(cpl->file().get());
@@ -2510,9 +3767,25 @@ BOOST_AUTO_TEST_CASE (verify_missing_hash)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_HASH, asset_id }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_HASH, asset_id
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2526,15 +3799,34 @@ verify_markers_test (
        )
 {
        auto dcp = make_simple (dir);
-       dcp->cpls()[0]->set_content_kind (dcp::ContentKind::FEATURE);
+       auto cpl = dcp->cpls()[0];
+       cpl->set_content_kind(dcp::ContentKind::FEATURE);
        auto markers_asset = make_shared<dcp::ReelMarkersAsset>(dcp::Fraction(24, 1), 24);
        for (auto const& i: markers) {
                markers_asset->set (i.first, i.second);
        }
-       dcp->cpls()[0]->reels()[0]->add(markers_asset);
+       cpl->reels()[0]->add(markers_asset);
        dcp->write_xml();
 
-       check_verify_result ({dir}, test_notes);
+       for (auto& note: test_notes) {
+               note.set_cpl_id(cpl->id());
+       }
+
+       test_notes.push_back(ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl));
+       test_notes.push_back(ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl));
+       test_notes.push_back(ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl));
+       test_notes.push_back(ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl));
+       test_notes.push_back(
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::OK,
+                       dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                       string{"1998x1080"},
+                       cpl->file().get()
+                       ).set_cpl_id(cpl->id())
+               );
+       test_notes.push_back(ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl));
+       test_notes.push_back(ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl));
+       check_verify_result({dir}, {}, test_notes);
 }
 
 
@@ -2630,7 +3922,26 @@ BOOST_AUTO_TEST_CASE (verify_missing_cpl_metadata_version_number)
        cpl->unset_version_number();
        dcp->write_xml();
 
-       check_verify_result ({dir}, {{ dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA_VERSION_NUMBER, cpl->id(), cpl->file().get() }});
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA_VERSION_NUMBER, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
+               });
 }
 
 
@@ -2643,6 +3954,8 @@ BOOST_AUTO_TEST_CASE (verify_missing_extension_metadata1)
        BOOST_REQUIRE_EQUAL (dcp->cpls().size(), 1U);
        auto cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                Editor e (cpl->file().get());
                e.delete_lines ("<meta:ExtensionMetadataList>", "</meta:ExtensionMetadataList>");
@@ -2650,9 +3963,25 @@ BOOST_AUTO_TEST_CASE (verify_missing_extension_metadata1)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_EXTENSION_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_EXTENSION_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2665,6 +3994,8 @@ BOOST_AUTO_TEST_CASE (verify_missing_extension_metadata2)
 
        auto cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                Editor e (cpl->file().get());
                e.delete_lines ("<meta:ExtensionMetadata scope=\"http://isdcf.com/ns/cplmd/app\">", "</meta:ExtensionMetadata>");
@@ -2672,9 +4003,25 @@ BOOST_AUTO_TEST_CASE (verify_missing_extension_metadata2)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_EXTENSION_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_EXTENSION_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2687,6 +4034,8 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata3)
 
        auto const cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                Editor e (cpl->file().get());
                e.replace ("<meta:Name>A", "<meta:NameX>A");
@@ -2695,10 +4044,27 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata3)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:NameX'"), cpl->file().get(), 70 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("element 'meta:NameX' is not allowed for content model '(Name,PropertyList?,)'"), cpl->file().get(), 77 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:NameX'"), cpl->file().get(), 70
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("element 'meta:NameX' is not allowed for content model '(Name,PropertyList?,)'"), cpl->file().get(), 77).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
                });
 }
 
@@ -2711,6 +4077,8 @@ BOOST_AUTO_TEST_CASE (verify_invalid_extension_metadata1)
 
        auto cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                Editor e (cpl->file().get());
                e.replace ("Application", "Fred");
@@ -2718,9 +4086,25 @@ BOOST_AUTO_TEST_CASE (verify_invalid_extension_metadata1)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_EXTENSION_METADATA, string("<Name> should be 'Application'"), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_EXTENSION_METADATA, string("<Name> should be 'Application'"), cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2733,6 +4117,8 @@ BOOST_AUTO_TEST_CASE (verify_invalid_extension_metadata2)
 
        auto cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                Editor e (cpl->file().get());
                e.replace ("DCP Constraints Profile", "Fred");
@@ -2740,9 +4126,25 @@ BOOST_AUTO_TEST_CASE (verify_invalid_extension_metadata2)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_EXTENSION_METADATA, string("<Name> property should be 'DCP Constraints Profile'"), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_EXTENSION_METADATA, string("<Name> property should be 'DCP Constraints Profile'"), cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2755,6 +4157,8 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata6)
 
        auto const cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                Editor e (cpl->file().get());
                e.replace ("<meta:Value>", "<meta:ValueX>");
@@ -2763,10 +4167,28 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata6)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:ValueX'"), cpl->file().get(), 74 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("element 'meta:ValueX' is not allowed for content model '(Name,Value)'"), cpl->file().get(), 75 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:ValueX'"), cpl->file().get(), 74
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("element 'meta:ValueX' is not allowed for content model '(Name,Value)'"), cpl->file().get(), 75
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash())
                });
 }
 
@@ -2779,6 +4201,8 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata7)
 
        auto const cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                Editor e (cpl->file().get());
                e.replace ("SMPTE-RDD-52:2020-Bv2.1", "Fred");
@@ -2786,9 +4210,25 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata7)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_EXTENSION_METADATA, string("<Value> property should be 'SMPTE-RDD-52:2020-Bv2.1'"), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_EXTENSION_METADATA, string("<Value> property should be 'SMPTE-RDD-52:2020-Bv2.1'"), cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2801,6 +4241,8 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata8)
 
        auto const cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                Editor e (cpl->file().get());
                e.replace ("<meta:Property>", "<meta:PropertyX>");
@@ -2809,10 +4251,27 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata8)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:PropertyX'"), cpl->file().get(), 72 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("element 'meta:PropertyX' is not allowed for content model '(Property+)'"), cpl->file().get(), 76 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:PropertyX'"), cpl->file().get(), 72
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("element 'meta:PropertyX' is not allowed for content model '(Property+)'"), cpl->file().get(), 76).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
                });
 }
 
@@ -2825,6 +4284,8 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata9)
 
        auto const cpl = dcp->cpls()[0];
 
+       HashCalculator calc(cpl->file().get());
+
        {
                Editor e (cpl->file().get());
                e.replace ("<meta:PropertyList>", "<meta:PropertyListX>");
@@ -2833,10 +4294,28 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata9)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:PropertyListX'"), cpl->file().get(), 71 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("element 'meta:PropertyListX' is not allowed for content model '(Name,PropertyList?,)'"), cpl->file().get(), 77 },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->id(), cpl->file().get() },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("no declaration found for element 'meta:PropertyListX'"), cpl->file().get(), 71
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_XML, string("element 'meta:PropertyListX' is not allowed for content model '(Name,PropertyList?,)'"), cpl->file().get(), 77
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl->file().get()
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
                });
 }
 
@@ -2844,31 +4323,56 @@ BOOST_AUTO_TEST_CASE (verify_invalid_xml_cpl_extension_metadata9)
 
 BOOST_AUTO_TEST_CASE (verify_unsigned_cpl_with_encrypted_content)
 {
-       path dir = "build/test/verify_unsigned_cpl_with_encrypted_content";
+       path const dir = "build/test/verify_unsigned_cpl_with_encrypted_content";
        prepare_directory (dir);
        for (auto i: directory_iterator("test/ref/DCP/encryption_test")) {
                copy_file (i.path(), dir / i.path().filename());
        }
 
-       path const pkl = dir / ( "pkl_" + encryption_test_pkl_id + ".xml" );
-       path const cpl = dir / ( "cpl_" + encryption_test_cpl_id + ".xml");
+       path const pkl = dir / ( "pkl_" + encryption_test_pkl_id() + ".xml");
+       path const cpl_path = dir / ( "cpl_" + encryption_test_cpl_id() + ".xml");
+
+       HashCalculator calc(cpl_path);
 
        {
-               Editor e (cpl);
+               Editor e(cpl_path);
                e.delete_lines ("<dsig:Signature", "</dsig:Signature>");
        }
 
+       auto cpl = std::make_shared<dcp::CPL>(cpl_path);
+
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, encryption_test_cpl_id, canonical(cpl) },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_PKL_ANNOTATION_TEXT_WITH_CPL, encryption_test_pkl_id, canonical(pkl), },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_FFOC },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_LFOC },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, encryption_test_cpl_id, canonical(cpl) },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::UNSIGNED_CPL_WITH_ENCRYPTED_CONTENT, encryption_test_cpl_id, canonical(cpl) }
+                       ok(dcp::VerificationNote::Code::ALL_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(cpl_path)
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_PKL_ANNOTATION_TEXT_WITH_CPL, encryption_test_pkl_id(), canonical(pkl)
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_FFOC
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_LFOC
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, canonical(cpl_path)
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::UNSIGNED_CPL_WITH_ENCRYPTED_CONTENT, canonical(cpl_path)
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2881,23 +4385,45 @@ BOOST_AUTO_TEST_CASE (verify_unsigned_pkl_with_encrypted_content)
                copy_file (i.path(), dir / i.path().filename());
        }
 
-       path const cpl = dir / ("cpl_" + encryption_test_cpl_id + ".xml");
-       path const pkl = dir / ("pkl_" + encryption_test_pkl_id + ".xml");
+       path const cpl_path = dir / ("cpl_" + encryption_test_cpl_id() + ".xml");
+       path const pkl = dir / ("pkl_" + encryption_test_pkl_id() + ".xml");
        {
                Editor e (pkl);
                e.delete_lines ("<dsig:Signature", "</dsig:Signature>");
        }
 
+       auto cpl = std::make_shared<dcp::CPL>(cpl_path);
+
        check_verify_result (
                {dir},
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_PKL_ANNOTATION_TEXT_WITH_CPL, encryption_test_pkl_id, canonical(pkl) },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_FFOC },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_LFOC },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, encryption_test_cpl_id, canonical(cpl) },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::UNSIGNED_PKL_WITH_ENCRYPTED_CONTENT, encryption_test_pkl_id, canonical(pkl) },
+                       ok(dcp::VerificationNote::Code::ALL_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_PKL_ANNOTATION_TEXT_WITH_CPL, encryption_test_pkl_id(), canonical(pkl)
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_FFOC
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_LFOC
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, canonical(cpl_path)
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::UNSIGNED_PKL_WITH_ENCRYPTED_CONTENT, encryption_test_pkl_id(), canonical(pkl)
+                               )
                });
 }
 
@@ -2911,11 +4437,29 @@ BOOST_AUTO_TEST_CASE (verify_unsigned_pkl_with_unencrypted_content)
        }
 
        {
-               Editor e (dir / dcp_test1_pkl);
+               Editor e (dir / dcp_test1_pkl());
                e.delete_lines ("<dsig:Signature", "</dsig:Signature>");
        }
 
-       check_verify_result ({dir}, {});
+       auto cpl = make_shared<dcp::CPL>(find_cpl(dir));
+
+       check_verify_result(
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+               });
 }
 
 
@@ -2980,8 +4524,22 @@ BOOST_AUTO_TEST_CASE (verify_partially_encrypted)
 
        check_verify_result (
                {dir},
+               {},
                {
-                       {dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::PARTIALLY_ENCRYPTED},
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1440x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::PARTIALLY_ENCRYPTED
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -2992,8 +4550,8 @@ BOOST_AUTO_TEST_CASE (verify_jpeg2000_codestream_2k)
        dcp::MonoPictureAsset picture (find_file(private_test / "data" / "JourneyToJah_TLR-1_F_EN-DE-FR_CH_51_2K_LOK_20140225_DGL_SMPTE_OV", "j2c.mxf"));
        auto reader = picture.start_read ();
        auto frame = reader->get_frame (0);
-       verify_j2k(frame, 0, 24, notes);
-       BOOST_REQUIRE_EQUAL (notes.size(), 0U);
+       verify_j2k(frame, 0, 0, 24, notes);
+       BOOST_CHECK(notes.empty());
 }
 
 
@@ -3003,8 +4561,8 @@ BOOST_AUTO_TEST_CASE (verify_jpeg2000_codestream_4k)
        dcp::MonoPictureAsset picture (find_file(private_test / "data" / "sul", "TLR"));
        auto reader = picture.start_read ();
        auto frame = reader->get_frame (0);
-       verify_j2k(frame, 0, 24, notes);
-       BOOST_REQUIRE_EQUAL (notes.size(), 0U);
+       verify_j2k(frame, 0, 0, 24, notes);
+       BOOST_CHECK(notes.empty());
 }
 
 
@@ -3018,8 +4576,8 @@ BOOST_AUTO_TEST_CASE (verify_jpeg2000_codestream_libdcp)
        dcp::MonoPictureAsset picture (find_file(dir, "video"));
        auto reader = picture.start_read ();
        auto frame = reader->get_frame (0);
-       verify_j2k(frame, 0, 24, notes);
-       BOOST_REQUIRE_EQUAL (notes.size(), 0U);
+       verify_j2k(frame, 0, 0, 24, notes);
+       BOOST_CHECK(notes.empty());
 }
 
 
@@ -3079,11 +4637,24 @@ BOOST_AUTO_TEST_CASE (verify_mismatched_subtitle_resource_id)
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_DURATION , "240 0", boost::filesystem::canonical(subs_mxf) },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_RESOURCE_ID },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_DURATION , "240 0", boost::filesystem::canonical(subs_mxf)
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_RESOURCE_ID
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -3144,12 +4715,27 @@ BOOST_AUTO_TEST_CASE (verify_incorrect_timed_text_id)
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_DURATION , "240 0", boost::filesystem::canonical(subs_mxf) },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INCORRECT_TIMED_TEXT_ASSET_ID },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get() },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE, string{"2018-10-02T12:25:14+02:00"} }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISMATCHED_TIMED_TEXT_DURATION , "240 0", boost::filesystem::canonical(subs_mxf)
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INCORRECT_TIMED_TEXT_ASSET_ID
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_ISSUE_DATE, string{"2018-10-02T12:25:14+02:00"}
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -3157,17 +4743,28 @@ BOOST_AUTO_TEST_CASE (verify_incorrect_timed_text_id)
 /** Check a DCP with a 3D asset marked as 2D */
 BOOST_AUTO_TEST_CASE (verify_threed_marked_as_twod)
 {
+       auto const path = private_test / "data" / "xm";
+
+       auto cpl = std::make_shared<dcp::CPL>(find_prefix(path, "CPL_"));
+       BOOST_REQUIRE(cpl);
+
        check_verify_result (
-               { private_test / "data" / "xm" },
+               { path },
+               {},
                {
-                       {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(path / "0d6f57e6-adac-4e1d-bfbe-d162bf13e2cd_j2c.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(path / "0d6f57e6-adac-4e1d-bfbe-d162bf13e2cd_j2c.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
                                dcp::VerificationNote::Type::WARNING,
-                               dcp::VerificationNote::Code::THREED_ASSET_MARKED_AS_TWOD, boost::filesystem::canonical(find_file(private_test / "data" / "xm", "j2c"))
-                       },
-                       {
+                               dcp::VerificationNote::Code::THREED_ASSET_MARKED_AS_TWOD, boost::filesystem::canonical(find_file(path, "j2c"))
+                               ),
+                       dcp::VerificationNote(
                                dcp::VerificationNote::Type::BV21_ERROR,
                                dcp::VerificationNote::Code::INVALID_STANDARD
-                       },
+                               )
                });
 
 }
@@ -3180,6 +4777,8 @@ BOOST_AUTO_TEST_CASE (verify_unexpected_things_in_main_markers)
        auto dcp = make_simple (dir, 1, 24);
        dcp->write_xml();
 
+       HashCalculator calc(find_cpl(dir));
+
        {
                Editor e (find_cpl(dir));
                e.insert(
@@ -3188,14 +4787,32 @@ BOOST_AUTO_TEST_CASE (verify_unexpected_things_in_main_markers)
                        );
        }
 
-       dcp::CPL cpl (find_cpl(dir));
+       auto cpl = make_shared<dcp::CPL>(find_cpl(dir));
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl.id(), canonical(find_cpl(dir)) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::UNEXPECTED_ENTRY_POINT },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::UNEXPECTED_DURATION },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::UNEXPECTED_ENTRY_POINT
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::UNEXPECTED_DURATION
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -3207,18 +4824,36 @@ BOOST_AUTO_TEST_CASE(verify_invalid_content_kind)
        auto dcp = make_simple (dir, 1, 24);
        dcp->write_xml();
 
+       HashCalculator calc(find_cpl(dir));
+
        {
                Editor e(find_cpl(dir));
                e.replace("trailer", "trip");
        }
 
-       dcp::CPL cpl (find_cpl(dir));
+       auto cpl = std::make_shared<dcp::CPL>(find_cpl(dir));
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl.id(), canonical(find_cpl(dir)) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_CONTENT_KIND, string("trip") }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_CONTENT_KIND, string("trip")
+                               ).set_cpl_id(cpl->id()),
                });
 
 }
@@ -3231,19 +4866,34 @@ BOOST_AUTO_TEST_CASE(verify_valid_content_kind)
        auto dcp = make_simple (dir, 1, 24);
        dcp->write_xml();
 
+       HashCalculator calc(find_cpl(dir));
+
        {
                Editor e(find_cpl(dir));
                e.replace("<ContentKind>trailer</ContentKind>", "<ContentKind scope=\"http://bobs.contents/\">trip</ContentKind>");
        }
 
-       dcp::CPL cpl (find_cpl(dir));
+       auto cpl = std::make_shared<dcp::CPL>(find_cpl(dir));
 
        check_verify_result (
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl.id(), canonical(find_cpl(dir)) },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
                });
-
 }
 
 
@@ -3256,6 +4906,8 @@ BOOST_AUTO_TEST_CASE(verify_invalid_main_picture_active_area_1)
 
        auto constexpr area = "<meta:MainPictureActiveArea>";
 
+       HashCalculator calc(find_cpl(dir));
+
        {
                Editor e(find_cpl(dir));
                e.delete_lines_after(area, 2);
@@ -3264,14 +4916,26 @@ BOOST_AUTO_TEST_CASE(verify_invalid_main_picture_active_area_1)
        }
 
        dcp::PKL pkl(find_pkl(dir));
-       dcp::CPL cpl(find_cpl(dir));
+       auto cpl = std::make_shared<dcp::CPL>(find_cpl(dir));
 
        check_verify_result(
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl.id(), canonical(find_cpl(dir)) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "width 1997 is not a multiple of 2", canonical(find_cpl(dir)) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "height 4080 is bigger than the asset height 1080", canonical(find_cpl(dir)) },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "width 1997 is not a multiple of 2", canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "height 4080 is bigger than the asset height 1080", canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id()),
                });
 }
 
@@ -3285,6 +4949,8 @@ BOOST_AUTO_TEST_CASE(verify_invalid_main_picture_active_area_2)
 
        auto constexpr area = "<meta:MainPictureActiveArea>";
 
+       HashCalculator calc(find_cpl(dir));
+
        {
                Editor e(find_cpl(dir));
                e.delete_lines_after(area, 2);
@@ -3293,15 +4959,29 @@ BOOST_AUTO_TEST_CASE(verify_invalid_main_picture_active_area_2)
        }
 
        dcp::PKL pkl(find_pkl(dir));
-       dcp::CPL cpl(find_cpl(dir));
+       auto cpl = std::make_shared<dcp::CPL>(find_cpl(dir));
 
        check_verify_result(
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, cpl.id(), canonical(find_cpl(dir)) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "height 5125 is not a multiple of 2", canonical(find_cpl(dir)) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "width 9900 is bigger than the asset width 1998", canonical(find_cpl(dir)) },
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "height 5125 is bigger than the asset height 1080", canonical(find_cpl(dir)) },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_CPL_HASHES, canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id()).set_reference_hash(calc.old_hash()).set_calculated_hash(calc.new_hash()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "height 5125 is not a multiple of 2", canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "width 9900 is bigger than the asset width 1998", canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_PICTURE_ACTIVE_AREA, "height 5125 is bigger than the asset height 1080", canonical(find_cpl(dir))
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -3321,10 +5001,22 @@ BOOST_AUTO_TEST_CASE(verify_duplicate_pkl_asset_ids)
        }
 
        dcp::PKL pkl(find_pkl(dir));
+       auto cpl = std::make_shared<dcp::CPL>(find_cpl(dir));
 
        check_verify_result(
                { dir },
+               {},
                {
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
                        { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::DUPLICATE_ASSET_ID_IN_PKL, pkl.id(), canonical(find_pkl(dir)) },
                });
 }
@@ -3346,12 +5038,28 @@ BOOST_AUTO_TEST_CASE(verify_duplicate_assetmap_asset_ids)
 
        dcp::PKL pkl(find_pkl(dir));
        dcp::AssetMap asset_map(find_asset_map(dir));
+       auto cpl = make_shared<dcp::CPL>(find_cpl(dir));
 
        check_verify_result(
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::DUPLICATE_ASSET_ID_IN_ASSETMAP, asset_map.id(), canonical(find_asset_map(dir)) },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EXTERNAL_ASSET, string("5407b210-4441-4e97-8b16-8bdc7c12da54") },
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               canonical(cpl->file().get())
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::DUPLICATE_ASSET_ID_IN_ASSETMAP, asset_map.id(), canonical(find_asset_map(dir))
+                               ),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EXTERNAL_ASSET, string("5407b210-4441-4e97-8b16-8bdc7c12da54")
+                               )
                });
 }
 
@@ -3422,8 +5130,25 @@ BOOST_AUTO_TEST_CASE(verify_mismatched_sound_channel_counts)
 
        check_verify_result(
                { path },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_SOUND_CHANNEL_COUNTS, canonical(find_file(path, "audio2")) },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(path / "video1.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(path / "video1.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(path / "video2.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(path / "video2.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_SOUND_CHANNEL_COUNTS, canonical(find_file(path, "audio2"))
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -3472,8 +5197,23 @@ BOOST_AUTO_TEST_CASE(verify_invalid_main_sound_configuration)
 
        check_verify_result(
                { path },
+               {},
                {
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_SOUND_CONFIGURATION, std::string{"MainSoundConfiguration has 6 channels but sound assets have 2"}, canonical(find_cpl(path)) },
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(path / "video1.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(path / "video1.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_MAIN_SOUND_CONFIGURATION, std::string{"MainSoundConfiguration has 6 channels but sound assets have 2"}, canonical(find_cpl(path))
+                               ).set_cpl_id(cpl->id())
                });
 }
 
@@ -3528,29 +5268,90 @@ BOOST_AUTO_TEST_CASE(verify_invalid_tile_part_size)
        dcp->set_annotation_text("A Test DCP");
        dcp->write_xml();
 
-       check_verify_result(
-               { path },
-               {
-                       dcp::VerificationNote(dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_PART_SIZE).set_frame(0).set_component(0).set_size(1321721),
-                       { dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_SIZE_IN_BYTES, canonical(path / "video.mxf") },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_FFOC },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_LFOC },
-               });
+       vector<dcp::VerificationNote> expected = {
+               ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::OK,
+                       dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                       string{"1998x1080"},
+                       cpl->file().get()
+                       ).set_cpl_id(cpl->id()),
+               ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(path / "video.mxf"), cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+               ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+               ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_FFOC
+                       ).set_cpl_id(cpl->id()),
+               dcp::VerificationNote(
+                       dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::MISSING_LFOC
+                       ).set_cpl_id(cpl->id())
+       };
+
+       for (auto frame = 0; frame < 24; frame++) {
+               expected.push_back(
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_PICTURE_FRAME_SIZE_IN_BYTES, canonical(path / "video.mxf")
+                       ).set_frame(frame).set_frame_rate(24).set_cpl_id(cpl->id())
+               );
+       }
+
+       int component_sizes[] = {
+               1321816,
+               1294414,
+               1289881,
+       };
+
+       for (auto frame = 0; frame < 24; frame++) {
+               for (auto component = 0; component < 3; component++) {
+                       expected.push_back(
+                               dcp::VerificationNote(
+                                       dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::INVALID_JPEG2000_TILE_PART_SIZE
+                                       ).set_frame(frame).set_component(component).set_size(component_sizes[component]).set_cpl_id(cpl->id())
+                               );
+               }
+       }
+
+       check_verify_result({ path }, {}, expected);
 }
 
 
 BOOST_AUTO_TEST_CASE(verify_too_many_subtitle_namespaces)
 {
        boost::filesystem::path const dir = "test/ref/DCP/subtitle_namespace_test";
+       dcp::DCP dcp(dir);
+       dcp.read();
+       BOOST_REQUIRE(!dcp.cpls().empty());
+       auto cpl = dcp.cpls()[0];
+
        check_verify_result(
                { dir },
+               {},
                {
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE, canonical(find_file(dir, "sub_")) },
-                       { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, "fc815694-7977-4a27-a8b3-32b9d4075e4c", canonical(find_file(dir, "cpl_")) },
-                       { dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT, std::string{"315de731-1173-484c-9a35-bdacf5a9d99d"} }
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"Dcp_FTR-1_F_XX-XX_MOS_2K_20230407_SMPTE_OV"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "j2c_42b34dcd-caa5-4c7b-aa0f-66a590947ba1.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "j2c_42b34dcd-caa5-4c7b-aa0f-66a590947ba1.mxf"), cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFEC_IN_FEATURE
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_FFMC_IN_FEATURE
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INVALID_SUBTITLE_FIRST_TEXT_TIME
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_SUBTITLE_LANGUAGE, canonical(find_file(dir, "sub_"))
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, canonical(find_file(dir, "cpl_"))
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::INCORRECT_SUBTITLE_NAMESPACE_COUNT, std::string{"315de731-1173-484c-9a35-bdacf5a9d99d"}
+                               ).set_cpl_id(cpl->id()),
                });
 }
 
@@ -3566,12 +5367,17 @@ BOOST_AUTO_TEST_CASE(verify_missing_load_font_for_font)
        }
        auto asset = make_shared<dcp::InteropSubtitleAsset>(dir / "subs.xml");
        auto reel_asset = make_shared<dcp::ReelInteropSubtitleAsset>(asset, dcp::Fraction(24, 1), 16 * 24, 0);
-       write_dcp_with_single_asset (dir, reel_asset, dcp::Standard::INTEROP);
+       auto cpl = write_dcp_with_single_asset(dir, reel_asset, dcp::Standard::INTEROP);
 
        check_verify_result (
-               {dir}, {
+               {dir},
+               {},
+               {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
                        { dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::INVALID_STANDARD },
-                       dcp::VerificationNote(dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_LOAD_FONT_FOR_FONT).set_id("theFontId")
+                       dcp::VerificationNote(dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_LOAD_FONT_FOR_FONT).set_id("theFontId").set_cpl_id(cpl->id())
                });
 
 }
@@ -3612,13 +5418,27 @@ BOOST_AUTO_TEST_CASE(verify_missing_load_font)
        subs->write(dir / "subs.mxf");
 
        auto reel_subs = make_shared<dcp::ReelSMPTESubtitleAsset>(subs, dcp::Fraction(24, 1), 202, 0);
-       dcp->cpls()[0]->reels()[0]->add(reel_subs);
+       auto cpl = dcp->cpls()[0];
+       cpl->reels()[0]->add(reel_subs);
        dcp->write_xml();
 
        check_verify_result (
                { dir },
+               {},
                {
-                       dcp::VerificationNote(dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_LOAD_FONT).set_id(reel_subs->id())
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISSING_LOAD_FONT).set_id(reel_subs->id()).set_cpl_id(cpl->id())
                });
 }
 
@@ -3630,6 +5450,7 @@ BOOST_AUTO_TEST_CASE(verify_spots_wrong_asset)
 
        auto dcp1 = make_simple(dir / "1");
        dcp1->write_xml();
+       auto cpl = dcp1->cpls()[0];
 
        auto const asset_1 = dcp::MonoPictureAsset(dir / "1" / "video.mxf").id();
 
@@ -3642,7 +5463,18 @@ BOOST_AUTO_TEST_CASE(verify_spots_wrong_asset)
 
        check_verify_result(
                {dir / "1"},
+               {},
                {
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
                        dcp::VerificationNote(dcp::VerificationNote::Type::ERROR, dcp::VerificationNote::Code::MISMATCHED_ASSET_MAP_ID).set_id(asset_1).set_other_id(asset_2)
                });
 }
@@ -3661,7 +5493,57 @@ BOOST_AUTO_TEST_CASE(verify_cpl_content_version_label_text_empty)
 
        check_verify_result(
                {dir},
+               {},
+               {
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::OK,
+                               dcp::VerificationNote::Code::VALID_MAIN_PICTURE_ACTIVE_AREA,
+                               string{"1998x1080"},
+                               cpl->file().get()
+                               ).set_cpl_id(cpl->id()),
+                       ok(dcp::VerificationNote::Code::NONE_ENCRYPTED, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"A Test DCP"}, cpl),
+                       ok(dcp::VerificationNote::Code::CORRECT_PICTURE_HASH, canonical(dir / "video.mxf"), cpl),
+                       ok(dcp::VerificationNote::Code::VALID_PICTURE_FRAME_SIZES_IN_BYTES, canonical(dir / "video.mxf"), cpl),
+                       dcp::VerificationNote(dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EMPTY_CONTENT_VERSION_LABEL_TEXT, cpl->file().get()).set_cpl_id(cpl->id())
+               });
+}
+
+
+/** Check that we don't get any strange errors when verifying encrypted DCPs (DoM #2659) */
+BOOST_AUTO_TEST_CASE(verify_encrypted_smpte_dcp)
+{
+       auto const dir = path("build/test/verify_encrypted_smpte_dcp");
+       dcp::Key key;
+       auto key_id = dcp::make_uuid();
+       auto cpl = dcp_with_text<dcp::ReelSMPTESubtitleAsset>(dir, {{ 4 * 24, 5 * 24 }}, key, key_id);
+
+       dcp::DecryptedKDM kdm(dcp::LocalTime(), dcp::LocalTime(), "", "", "");
+       kdm.add_key(dcp::DecryptedKDMKey(string{"MDIK"}, key_id, key, cpl->id(), dcp::Standard::SMPTE));
+
+       path const pkl_file = find_file(dir, "pkl_");
+       path const cpl_file = find_file(dir, "cpl_");
+
+       check_verify_result(
+               { dir },
+               { kdm },
                {
-                       dcp::VerificationNote(dcp::VerificationNote::Type::WARNING, dcp::VerificationNote::Code::EMPTY_CONTENT_VERSION_LABEL_TEXT, cpl->file().get()).set_id(cpl->id())
+                       ok(dcp::VerificationNote::Code::MATCHING_PKL_ANNOTATION_TEXT_WITH_CPL, cpl),
+                       ok(dcp::VerificationNote::Code::MATCHING_CPL_HASHES, cpl),
+                       ok(dcp::VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, string{"hello"}, cpl),
+                       ok(dcp::VerificationNote::Code::ALL_ENCRYPTED, cpl),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::MISSING_CPL_METADATA, canonical(cpl_file)
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::UNSIGNED_CPL_WITH_ENCRYPTED_CONTENT, canonical(cpl_file)
+                               ).set_cpl_id(cpl->id()),
+                       dcp::VerificationNote(
+                               dcp::VerificationNote::Type::BV21_ERROR, dcp::VerificationNote::Code::UNSIGNED_PKL_WITH_ENCRYPTED_CONTENT, filename_to_id(pkl_file.filename()), canonical(pkl_file)
+                               )
                });
 }
+
+