Bump libdcp for enum class stuff.
[libsub.git] / test / ssa_reader_test.cc
index 333d5fd8388d6f8c0e9327cff76b06129342f730..d3a34bfa69ab9d43e6ef3c2bc25ebf2e891acd12 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2016 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2016-2021 Carl Hetherington <cth@carlh.net>
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
 #include "ssa_reader.h"
 #include "collect.h"
 #include "subtitle.h"
+#include "exceptions.h"
 #include <boost/test/unit_test.hpp>
 #include <boost/filesystem.hpp>
 #include <cstdio>
 #include <cmath>
+#include <iostream>
 
-using std::list;
 using std::fabs;
+using std::vector;
 
 BOOST_AUTO_TEST_CASE (ssa_reader_test)
 {
@@ -35,14 +37,14 @@ BOOST_AUTO_TEST_CASE (ssa_reader_test)
        FILE* f = fopen (p.string().c_str(), "r");
        sub::SSAReader reader (f);
        fclose (f);
-       list<sub::Subtitle> subs = sub::collect<std::list<sub::Subtitle> > (reader.subtitles ());
+       auto subs = sub::collect<vector<sub::Subtitle>> (reader.subtitles());
 
-       list<sub::Subtitle>::iterator i = subs.begin ();
+       auto i = subs.begin ();
 
        BOOST_REQUIRE (i != subs.end ());
        BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 2, 40, 650));
        BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 2, 41, 790));
-       list<sub::Line>::iterator j = i->lines.begin();
+       auto j = i->lines.begin();
        BOOST_REQUIRE (j != i->lines.end ());
        BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
        sub::Block b = j->blocks.front ();
@@ -75,9 +77,13 @@ BOOST_AUTO_TEST_CASE (ssa_reader_test)
 BOOST_AUTO_TEST_CASE (ssa_reader_line_test1)
 {
        sub::RawSubtitle base;
-       list<sub::RawSubtitle> r = sub::SSAReader::parse_line (base, "This is a line with some {\\i1}italics{\\i0} and then\\nthere is a new line.");
+       auto r = sub::SSAReader::parse_line (
+               base,
+               "This is a line with some {\\i1}italics{\\i0} and then\\nthere is a new line.",
+               1920, 1080
+               );
 
-       list<sub::RawSubtitle>::const_iterator i = r.begin ();
+       auto i = r.begin();
        BOOST_CHECK_EQUAL (i->text, "This is a line with some ");
        BOOST_CHECK_EQUAL (i->italic, false);
        ++i;
@@ -101,13 +107,32 @@ BOOST_AUTO_TEST_CASE (ssa_reader_line_test1)
 BOOST_AUTO_TEST_CASE (ssa_reader_line_test2)
 {
        sub::RawSubtitle base;
-       list<sub::RawSubtitle> r = sub::SSAReader::parse_line (base, "{\\i1}It's all just italics{\\i0}");
+       auto r = sub::SSAReader::parse_line (
+               base,
+               "{\\i1}It's all just italics{\\i0}",
+               1920, 1080
+               );
 
-       list<sub::RawSubtitle>::const_iterator i = r.begin ();
+       auto i = r.begin ();
        BOOST_CHECK_EQUAL (i->text, "It's all just italics");
        BOOST_CHECK_EQUAL (i->italic, true);
        ++i;
        BOOST_REQUIRE (i == r.end ());
+
+       r = sub::SSAReader::parse_line (
+               base,
+               "{\\i1}Italic{\\i0}\\Nand new line",
+               1920, 1080
+               );
+
+       i = r.begin ();
+       BOOST_CHECK_EQUAL (i->text, "Italic");
+       BOOST_CHECK_EQUAL (i->italic, true);
+       BOOST_CHECK (fabs ((72.0 * 1.2 / 792) - i->vertical_position.proportional.get()) < 1e-5);
+       ++i;
+       BOOST_CHECK_EQUAL (i->text, "and new line");
+       BOOST_CHECK_EQUAL (i->italic, false);
+       BOOST_CHECK (i->vertical_position.proportional.get() < 1e-5);
 }
 
 static void
@@ -124,8 +149,41 @@ test (boost::filesystem::path p)
 BOOST_AUTO_TEST_CASE (ssa_reader_test2)
 {
        test ("DKH_UT_EN20160601def.ssa");
+       test ("dcpsubtest-en.ssa");
+       test ("dcpsubtest-en.ssa");
+       test ("W_GERMAN_SUBS_grey.ass");
 }
 
+#define SUB_START(f, t) \
+       BOOST_REQUIRE (i != subs.end ()); \
+       BOOST_CHECK_EQUAL (i->from, f); \
+       BOOST_CHECK_EQUAL (i->to, t); \
+       j = i->lines.begin ();
+
+#define LINE(vp, vr, hp, hr)                 \
+       BOOST_REQUIRE (j != i->lines.end ()); \
+       BOOST_CHECK (j->vertical_position.proportional); \
+       BOOST_CHECK (fabs (j->vertical_position.proportional.get() - vp) < 1e-5); \
+       BOOST_CHECK (j->vertical_position.reference); \
+       BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), vr); \
+       BOOST_CHECK (fabs (j->horizontal_position.proportional - hp) < 1e-5); \
+       BOOST_CHECK_EQUAL (j->horizontal_position.reference, hr); \
+       k = j->blocks.begin (); \
+       ++j;
+
+#define BLOCK(t, f, s, b, i, u) \
+       BOOST_REQUIRE (k != j->blocks.end ()); \
+       BOOST_CHECK_EQUAL (k->text, t); \
+        BOOST_CHECK_EQUAL (k->font.get(), f); \
+       BOOST_CHECK_EQUAL (k->font_size.points().get(), s); \
+       BOOST_CHECK_EQUAL (k->bold, b); \
+       BOOST_CHECK_EQUAL (k->italic, i); \
+       BOOST_CHECK_EQUAL (k->underline, u); \
+       ++k;
+
+#define SUB_END() \
+       ++i;
+
 /** Test reading of a file within the libsub tree which exercises the parser */
 BOOST_AUTO_TEST_CASE (ssa_reader_test3)
 {
@@ -133,104 +191,337 @@ BOOST_AUTO_TEST_CASE (ssa_reader_test3)
        FILE* f = fopen (p.string().c_str(), "r");
        sub::SSAReader reader (f);
        fclose (f);
-       list<sub::Subtitle> subs = sub::collect<std::list<sub::Subtitle> > (reader.subtitles ());
+       auto subs = sub::collect<vector<sub::Subtitle>> (reader.subtitles());
 
-       list<sub::Subtitle>::iterator i = subs.begin ();
+       auto i = subs.begin();
+       vector<sub::Line>::iterator j;
+       vector<sub::Block>::iterator k;
 
        /* Hello world */
-       BOOST_REQUIRE (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 0, 1, 230));
-       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 0, 4, 550));
-       list<sub::Line>::iterator j = i->lines.begin();
-       BOOST_REQUIRE (j != i->lines.end ());
-       BOOST_CHECK (j->vertical_position.proportional);
-       BOOST_CHECK_EQUAL (j->vertical_position.proportional.get(), 0);
-       BOOST_CHECK (j->vertical_position.reference);
-       BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM_OF_SCREEN);
-       BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
-       sub::Block b = j->blocks.front ();
-       BOOST_CHECK_EQUAL (b.text, "Hello world");
-       BOOST_CHECK_EQUAL (b.font.get(), "Arial");
-       BOOST_CHECK_EQUAL (b.font_size.points().get(), 20);
-       BOOST_CHECK_EQUAL (b.bold, false);
-       BOOST_CHECK_EQUAL (b.italic, false);
-       BOOST_CHECK_EQUAL (b.underline, false);
-       ++i;
+       SUB_START (sub::Time::from_hms (0, 0, 1, 230), sub::Time::from_hms (0, 0, 4, 550));
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("Hello world", "Arial", 20, false, false, false);
+       SUB_END();
 
        /* This is vertically moved\nand has two lines. */
-       BOOST_REQUIRE (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 0, 5, 740));
-       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 0, 11, 0));
-       j = i->lines.begin();
-       BOOST_REQUIRE (j != i->lines.end ());
-       BOOST_CHECK (j->vertical_position.proportional);
+       SUB_START (sub::Time::from_hms (0, 0, 5, 740), sub::Time::from_hms (0, 0, 11, 0));
        /* The first line should be 900 pixels and one line (20
           points, 1.2 times spaced, as a proportion of the total
           screen height 729 points) up.
        */
-       BOOST_CHECK (fabs (j->vertical_position.proportional.get() - (900.0 / 1080) - (20.0 * 1.2 / 792)) < 1e-5);
-       BOOST_CHECK (j->vertical_position.reference);
-       BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM_OF_SCREEN);
-       BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
-       b = j->blocks.front ();
-       BOOST_CHECK_EQUAL (b.text, "This is vertically moved");
-       BOOST_CHECK_EQUAL (b.font.get(), "Arial");
-       BOOST_CHECK_EQUAL (b.font_size.points().get(), 20);
-       BOOST_CHECK_EQUAL (b.bold, false);
-       BOOST_CHECK_EQUAL (b.italic, false);
-       BOOST_CHECK_EQUAL (b.underline, false);
-       ++j;
-
-       BOOST_CHECK (fabs (j->vertical_position.proportional.get() - (900.0 / 1080)) < 1e-5);
-       BOOST_CHECK (j->vertical_position.reference);
-       BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM_OF_SCREEN);
-       BOOST_REQUIRE_EQUAL (j->blocks.size(), 1);
-       b = j->blocks.front ();
-       BOOST_CHECK_EQUAL (b.text, "and has two lines.");
-       BOOST_CHECK_EQUAL (b.font.get(), "Arial");
-       BOOST_CHECK_EQUAL (b.font_size.points().get(), 20);
-       BOOST_CHECK_EQUAL (b.bold, false);
-       BOOST_CHECK_EQUAL (b.italic, false);
-       BOOST_CHECK_EQUAL (b.underline, false);
-       ++j;
-       ++i;
+       LINE((900.0 / 1080) - (20.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK("This is vertically moved", "Arial", 20, false, false, false);
+       LINE((900.0 / 1080), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK("and has two lines.", "Arial", 20, false, false, false);
+       SUB_END();
 
        /* Some {\i1}italics{\i} are here. */
+       SUB_START (sub::Time::from_hms (0, 0, 7, 740), sub::Time::from_hms (0, 0, 9, 0));
+       LINE(0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK("Some ", "Arial", 20, false, false, false);
+       BLOCK("italics", "Arial", 20, false, true, false);
+       BLOCK(" are here.", "Arial", 20, false, false, false);
+       SUB_END();
+
+       /* Alignments */
+
+       SUB_START (sub::Time::from_hms (0, 0, 9, 230), sub::Time::from_hms (0, 0, 11, 560));
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::LEFT_OF_SCREEN);
+       BLOCK("bottom left", "Arial", 20, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 9, 240), sub::Time::from_hms (0, 0, 11, 560));
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK("bottom centre", "Arial", 20, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 9, 250), sub::Time::from_hms (0, 0, 11, 560));
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::RIGHT_OF_SCREEN);
+       BLOCK("bottom right", "Arial", 20, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 9, 260), sub::Time::from_hms (0, 0, 11, 560));
+       LINE (0, sub::VERTICAL_CENTRE_OF_SCREEN, 0, sub::LEFT_OF_SCREEN);
+       BLOCK("middle left", "Arial", 20, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 9, 270), sub::Time::from_hms (0, 0, 11, 560));
+       LINE (0, sub::VERTICAL_CENTRE_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK("middle centre", "Arial", 20, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 9, 280), sub::Time::from_hms (0, 0, 11, 560));
+       LINE (0, sub::VERTICAL_CENTRE_OF_SCREEN, 0, sub::RIGHT_OF_SCREEN);
+       BLOCK("middle right", "Arial", 20, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 9, 290), sub::Time::from_hms (0, 0, 11, 560));
+       LINE (0, sub::TOP_OF_SCREEN, 0, sub::LEFT_OF_SCREEN);
+       BLOCK("top left", "Arial", 20, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 9, 300), sub::Time::from_hms (0, 0, 11, 560));
+       LINE (0, sub::TOP_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK("top centre", "Arial", 20, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 9, 310), sub::Time::from_hms (0, 0, 11, 560));
+       LINE (0, sub::TOP_OF_SCREEN, 0, sub::RIGHT_OF_SCREEN);
+       BLOCK("top right", "Arial", 20, false, false, false);
+       SUB_END ();
+
+       BOOST_REQUIRE (i == subs.end ());
+}
+
+/** Test reading of a file within the libsub-test-private tree which exercises the parser */
+BOOST_AUTO_TEST_CASE (ssa_reader_test4)
+{
+       boost::filesystem::path p = private_test / "dcpsubtest2-en.ssa";
+       FILE* f = fopen (p.string().c_str(), "r");
+       sub::SSAReader reader (f);
+       fclose (f);
+       auto subs = sub::collect<vector<sub::Subtitle>> (reader.subtitles());
+
+       auto i = subs.begin();
+       vector<sub::Line>::iterator j;
+       vector<sub::Block>::iterator k;
+
        BOOST_REQUIRE (i != subs.end ());
-       BOOST_CHECK_EQUAL (i->from, sub::Time::from_hms (0, 0, 7, 740));
-       BOOST_CHECK_EQUAL (i->to, sub::Time::from_hms (0, 0, 9, 0));
-       j = i->lines.begin();
-       BOOST_REQUIRE (j != i->lines.end ());
-       BOOST_CHECK (j->vertical_position.proportional);
-       BOOST_CHECK_EQUAL (j->vertical_position.proportional.get(), 0);
-       BOOST_CHECK (j->vertical_position.reference);
-       BOOST_CHECK_EQUAL (j->vertical_position.reference.get(), sub::BOTTOM_OF_SCREEN);
-       BOOST_REQUIRE_EQUAL (j->blocks.size(), 3);
-       list<sub::Block>::const_iterator bi = j->blocks.begin ();
-       BOOST_CHECK_EQUAL (bi->text, "Some ");
-       BOOST_CHECK_EQUAL (bi->font.get(), "Arial");
-       BOOST_CHECK_EQUAL (bi->font_size.points().get(), 20);
-       BOOST_CHECK_EQUAL (bi->bold, false);
-       BOOST_CHECK_EQUAL (bi->italic, false);
-       BOOST_CHECK_EQUAL (bi->underline, false);
-       ++bi;
-       BOOST_CHECK_EQUAL (bi->text, "italics");
-       BOOST_CHECK_EQUAL (bi->font.get(), "Arial");
-       BOOST_CHECK_EQUAL (bi->font_size.points().get(), 20);
-       BOOST_CHECK_EQUAL (bi->bold, false);
-       BOOST_CHECK_EQUAL (bi->italic, true);
-       BOOST_CHECK_EQUAL (bi->underline, false);
-       ++bi;
-       BOOST_CHECK_EQUAL (bi->text, " are here.");
-       BOOST_CHECK_EQUAL (bi->font.get(), "Arial");
-       BOOST_CHECK_EQUAL (bi->font_size.points().get(), 20);
-       BOOST_CHECK_EQUAL (bi->bold, false);
-       BOOST_CHECK_EQUAL (bi->italic, false);
-       BOOST_CHECK_EQUAL (bi->underline, false);
-       ++bi;
-       ++j;
-       BOOST_REQUIRE (j == i->lines.end());
+
+       SUB_START (sub::Time::from_hms (0, 0, 1, 0), sub::Time::from_hms (0, 0, 3, 0));
+       /* The first line should be one line (50 points, 1.2 times
+          spaced, as a proportion of the total screen height 729
+          points) up.
+       */
+       LINE ((50.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("1st line: This is normal", "Verdana", 50, false, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("2d line: this is bold", "Verdana", 50, true, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 3, 100), sub::Time::from_hms (0, 0, 5, 100));
+       LINE ((50.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("1st line: this is bold", "Verdana", 50, true, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("2nd line: This is normal", "Verdana", 50, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 5, 200), sub::Time::from_hms (0, 0, 7, 200));
+       LINE ((50.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("1st line: this is bold", "Verdana", 50, true, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("2nd line: this is italics", "Verdana", 50, false, true, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 7, 300), sub::Time::from_hms (0, 0, 9, 300));
+       LINE ((50.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("1st line: this is italics", "Verdana", 50, false, true, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("2nd line: this is bold", "Verdana", 50, true, false, false);
+       SUB_END ();
+}
+
+/** Test reading of a .ass file */
+BOOST_AUTO_TEST_CASE (ssa_reader_test5)
+{
+       boost::filesystem::path p = private_test / "dcpsubtest-en.ass";
+       FILE* f = fopen (p.string().c_str(), "r");
+       sub::SSAReader reader (f);
+       fclose (f);
+       auto subs = sub::collect<vector<sub::Subtitle>> (reader.subtitles());
+
+       auto i = subs.begin ();
+       vector<sub::Line>::iterator j;
+       vector<sub::Block>::iterator k;
+
+       BOOST_REQUIRE (i != subs.end ());
+
+       SUB_START (sub::Time::from_hms (0, 0, 1, 0), sub::Time::from_hms (0, 0, 3, 0));
+       /* The first line should be one line (26 points, 1.2 times
+          spaced, as a proportion of the total screen height 729
+          points) up.
+       */
+       LINE ((26.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("1st subtitle, 1st line", "arial", 26, true, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("2nd subtitle, 2nd line", "arial", 26, true, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 3, 100), sub::Time::from_hms (0, 0, 5, 100));
+       LINE ((26.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("2nd subtitle, 1st line", "arial", 26, true, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("2nd subtitle, 2nd line", "arial", 26, true, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 5, 200), sub::Time::from_hms (0, 0, 7, 200));
+       LINE ((26.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("3rd subtitle, 1st line", "arial", 26, true, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("3rd subtitle, 2nd line", "arial", 26, true, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 7, 300), sub::Time::from_hms (0, 0, 9, 300));
+       LINE ((26.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("4th subtitle, 1st line", "arial", 26, true, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("4th subtitle, 2nd line", "arial", 26, true, false, false);
+       SUB_END ();
+}
+
+/** Test reading of another .ass file */
+BOOST_AUTO_TEST_CASE (ssa_reader_test6)
+{
+       boost::filesystem::path p = private_test / "DCP-o-matic_test_subs_1.ass";
+       FILE* f = fopen (p.string().c_str(), "r");
+       BOOST_REQUIRE (f);
+       sub::SSAReader reader (f);
+       fclose (f);
+       auto subs = sub::collect<vector<sub::Subtitle>> (reader.subtitles());
+
+       auto i = subs.begin ();
+       vector<sub::Line>::iterator j;
+       vector<sub::Block>::iterator k;
+
+       BOOST_REQUIRE (i != subs.end ());
+
+       SUB_START (sub::Time::from_hms (0, 0, 0, 70), sub::Time::from_hms (0, 0, 1, 110));
+       /* The first line should be one line (30 points, 1.2 times
+          spaced, as a proportion of the total screen height 729
+          points) up.
+       */
+       LINE ((30.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("This line is normal", "Arial", 30, false, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("This line is bold", "Arial", 30, true, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 1, 200), sub::Time::from_hms (0, 0, 2, 240));
+       LINE ((30.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("This line is bold", "Arial", 30, true, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("This line is normal", "Arial", 30, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 2, 300), sub::Time::from_hms (0, 0, 3, 380));
+       LINE ((30.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("This line is bold", "Arial", 30, true, false, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("This line is italic", "Arial", 30, false, true, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 3, 400), sub::Time::from_hms (0, 0, 4, 480));
+       LINE ((30.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("This line is italic", "Arial", 30, false, true, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("This line is bold", "Arial", 30, true, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 4, 510), sub::Time::from_hms (0, 0, 5, 600));
+       LINE ((30.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("Last three words are ", "Arial", 30, false, false, false);
+       BLOCK ("bold AND italic", "Arial", 30, true, true, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("Last three words are ", "Arial", 30, false, false, false);
+       BLOCK ("italic AND bold", "Arial", 30, true, true, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 5, 620), sub::Time::from_hms (0, 0, 6, 710));
+       LINE ((30.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("Last three words are ", "Arial", 30, false, false, false);
+       BLOCK ("bold AND italic", "Arial", 30, true, true, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("First three words", "Arial", 30, true, true, false);
+       BLOCK (" are italic AND bold", "Arial", 30, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 6, 730), sub::Time::from_hms (0, 0, 8, 30));
+       LINE ((30.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("Last three words are ", "Arial", 30, false, false, false);
+       BLOCK ("bold AND italic", "Arial", 30, true, true, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("This line is normal", "Arial", 30, false, false, false);
+       SUB_END ();
+
+       SUB_START (sub::Time::from_hms (0, 0, 8, 90), sub::Time::from_hms (0, 0, 9, 210));
+       LINE ((30.0 * 1.2 / 792), sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("Both lines are bold AND italic", "Arial", 30, true, true, false);
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("Both lines are bold AND italic", "Arial", 30, true, true, false);
+       SUB_END ();
+}
+
+/** Test \pos */
+BOOST_AUTO_TEST_CASE (ssa_reader_pos)
+{
+       boost::filesystem::path p = "test/data/test2.ssa";
+       FILE* f = fopen (p.string().c_str(), "r");
+       sub::SSAReader reader (f);
+       fclose (f);
+       auto subs = sub::collect<vector<sub::Subtitle>> (reader.subtitles());
+
+       auto i = subs.begin ();
+       vector<sub::Line>::iterator j;
+       vector<sub::Block>::iterator k;
+
+       /* Hello world */
+       SUB_START (sub::Time::from_hms (0, 0, 1, 230), sub::Time::from_hms (0, 0, 4, 550));
+       LINE (0, sub::BOTTOM_OF_SCREEN, 0, sub::HORIZONTAL_CENTRE_OF_SCREEN);
+       BLOCK ("Hello world this is ", "Arial", 20, false, false, false);
+       LINE (300.0 / 1080, sub::TOP_OF_SCREEN, 400.0 / 1920, sub::LEFT_OF_SCREEN);
+       BLOCK ("positioning.", "Arial", 20, false, false, false);
+       SUB_END();
+}
+
+/** Test \fs */
+BOOST_AUTO_TEST_CASE (ssa_reader_fs)
+{
+       sub::RawSubtitle base;
+       auto r = sub::SSAReader::parse_line (
+               base,
+               "This is a line with some {\\fs64}font sizing.",
+               1920, 1080
+               );
+
+       auto i = r.begin ();
+       BOOST_CHECK_EQUAL (i->text, "This is a line with some ");
        ++i;
+       BOOST_REQUIRE (i != r.end ());
 
-       BOOST_REQUIRE (i == subs.end ());
+       BOOST_CHECK_EQUAL (i->text, "font sizing.");
+       BOOST_CHECK (i->font_size.points());
+       BOOST_CHECK_EQUAL (i->font_size.points().get(), 64);
+       ++i;
+       BOOST_REQUIRE (i == r.end ());
+}
+
+/** Test a valid \c */
+BOOST_AUTO_TEST_CASE (ssa_reader_c)
+{
+       sub::RawSubtitle base;
+       auto r = sub::SSAReader::parse_line (
+               base,
+               "{\\c&H00FFFF&}Dieser Untertitel ist gelb",
+               1920, 1080
+               );
+
+       auto i = r.begin ();
+       BOOST_CHECK_EQUAL (i->text, "Dieser Untertitel ist gelb");
+       BOOST_CHECK (i->colour == sub::Colour::from_rgb_hex("ffff00"));
+       ++i;
+       BOOST_REQUIRE (i == r.end ());
+}
+
+/** Test invalid \c */
+BOOST_AUTO_TEST_CASE (ssa_reader_c_bad)
+{
+       sub::RawSubtitle base;
+       BOOST_CHECK_THROW(
+               sub::SSAReader::parse_line(
+                       base,
+                       "{\\c&H0}Dieser Untertitel ist gelb",
+                       1920, 1080
+                       ),
+               sub::SSAError
+               );
 }