From ab01846a8df829d5a7d4e68bc2333c07639e6ab9 Mon Sep 17 00:00:00 2001
From: Behdad Esfahbod <behdad@behdad.org>
Date: Mon, 26 Dec 2016 17:35:21 -0500
Subject: [PATCH] [mtiLib] Add some more tests

---
 Lib/fontTools/mtiLib/mti_test.py              |  6 ++
 .../testdata/featurename-backward.ttx.GSUB    | 59 +++++++++++++
 .../mtiLib/testdata/featurename-backward.txt  | 14 +++
 .../testdata/featurename-forward.ttx.GSUB     | 59 +++++++++++++
 .../mtiLib/testdata/featurename-forward.txt   | 14 +++
 .../testdata/lookupnames-backward.ttx.GSUB    | 87 +++++++++++++++++++
 .../mtiLib/testdata/lookupnames-backward.txt  | 36 ++++++++
 .../testdata/lookupnames-forward.ttx.GSUB     | 87 +++++++++++++++++++
 .../mtiLib/testdata/lookupnames-forward.txt   | 36 ++++++++
 .../mtiLib/testdata/mixed-toplevels.ttx.GSUB  | 87 +++++++++++++++++++
 .../mtiLib/testdata/mixed-toplevels.txt       | 36 ++++++++
 11 files changed, 521 insertions(+)
 create mode 100644 Lib/fontTools/mtiLib/testdata/featurename-backward.ttx.GSUB
 create mode 100644 Lib/fontTools/mtiLib/testdata/featurename-backward.txt
 create mode 100644 Lib/fontTools/mtiLib/testdata/featurename-forward.ttx.GSUB
 create mode 100644 Lib/fontTools/mtiLib/testdata/featurename-forward.txt
 create mode 100644 Lib/fontTools/mtiLib/testdata/lookupnames-backward.ttx.GSUB
 create mode 100644 Lib/fontTools/mtiLib/testdata/lookupnames-backward.txt
 create mode 100644 Lib/fontTools/mtiLib/testdata/lookupnames-forward.ttx.GSUB
 create mode 100644 Lib/fontTools/mtiLib/testdata/lookupnames-forward.txt
 create mode 100644 Lib/fontTools/mtiLib/testdata/mixed-toplevels.ttx.GSUB
 create mode 100644 Lib/fontTools/mtiLib/testdata/mixed-toplevels.txt

diff --git a/Lib/fontTools/mtiLib/mti_test.py b/Lib/fontTools/mtiLib/mti_test.py
index 2a8083702..7b38eb0e0 100644
--- a/Lib/fontTools/mtiLib/mti_test.py
+++ b/Lib/fontTools/mtiLib/mti_test.py
@@ -20,6 +20,12 @@ class BuilderTest(unittest.TestCase):
             #'mti/cmap',
         ),
         'GSUB': (
+            'featurename-backward',
+            'featurename-forward',
+            'lookupnames-backward',
+            'lookupnames-forward',
+            'mixed-toplevels',
+
             'mti/scripttable',
             'mti/chainedclass',
             'mti/chainedcoverage',
diff --git a/Lib/fontTools/mtiLib/testdata/featurename-backward.ttx.GSUB b/Lib/fontTools/mtiLib/testdata/featurename-backward.ttx.GSUB
new file mode 100644
index 000000000..c80a96c10
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/featurename-backward.ttx.GSUB
@@ -0,0 +1,59 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<GSUB>
+  <Version value="0x00010000"/>
+  <ScriptList>
+    <!-- ScriptCount=2 -->
+    <ScriptRecord index="0">
+      <ScriptTag value="tel2"/>
+      <Script>
+        <DefaultLangSys>
+          <ReqFeatureIndex value="65535"/>
+          <!-- FeatureCount=2 -->
+          <FeatureIndex index="0" value="0"/>
+          <FeatureIndex index="1" value="1"/>
+        </DefaultLangSys>
+        <!-- LangSysCount=0 -->
+      </Script>
+    </ScriptRecord>
+    <ScriptRecord index="1">
+      <ScriptTag value="telu"/>
+      <Script>
+        <DefaultLangSys>
+          <ReqFeatureIndex value="65535"/>
+          <!-- FeatureCount=2 -->
+          <FeatureIndex index="0" value="0"/>
+          <FeatureIndex index="1" value="1"/>
+        </DefaultLangSys>
+        <!-- LangSysCount=0 -->
+      </Script>
+    </ScriptRecord>
+  </ScriptList>
+  <FeatureList>
+    <!-- FeatureCount=2 -->
+    <FeatureRecord index="0">
+      <FeatureTag value="akhn"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="0"/>
+      </Feature>
+    </FeatureRecord>
+    <FeatureRecord index="1">
+      <FeatureTag value="akh2"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="0"/>
+      </Feature>
+    </FeatureRecord>
+  </FeatureList>
+  <LookupList>
+    <!-- LookupCount=1 -->
+    <Lookup index="0">
+      <!-- LookupType=1 -->
+      <LookupFlag value="0"/>
+      <!-- SubTableCount=1 -->
+      <SingleSubst index="0" Format="1">
+        <Substitution in="a" out="b"/>
+      </SingleSubst>
+    </Lookup>
+  </LookupList>
+</GSUB>
diff --git a/Lib/fontTools/mtiLib/testdata/featurename-backward.txt b/Lib/fontTools/mtiLib/testdata/featurename-backward.txt
new file mode 100644
index 000000000..7e3d5d614
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/featurename-backward.txt
@@ -0,0 +1,14 @@
+
+feature table begin
+f0	akhn	l1
+1	akh2	l1
+feature table end
+
+script table begin
+telu	default		0, 1
+tel2	default		f0, 1
+script table end
+
+lookup	l1	single
+a	b
+lookup end
diff --git a/Lib/fontTools/mtiLib/testdata/featurename-forward.ttx.GSUB b/Lib/fontTools/mtiLib/testdata/featurename-forward.ttx.GSUB
new file mode 100644
index 000000000..c80a96c10
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/featurename-forward.ttx.GSUB
@@ -0,0 +1,59 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<GSUB>
+  <Version value="0x00010000"/>
+  <ScriptList>
+    <!-- ScriptCount=2 -->
+    <ScriptRecord index="0">
+      <ScriptTag value="tel2"/>
+      <Script>
+        <DefaultLangSys>
+          <ReqFeatureIndex value="65535"/>
+          <!-- FeatureCount=2 -->
+          <FeatureIndex index="0" value="0"/>
+          <FeatureIndex index="1" value="1"/>
+        </DefaultLangSys>
+        <!-- LangSysCount=0 -->
+      </Script>
+    </ScriptRecord>
+    <ScriptRecord index="1">
+      <ScriptTag value="telu"/>
+      <Script>
+        <DefaultLangSys>
+          <ReqFeatureIndex value="65535"/>
+          <!-- FeatureCount=2 -->
+          <FeatureIndex index="0" value="0"/>
+          <FeatureIndex index="1" value="1"/>
+        </DefaultLangSys>
+        <!-- LangSysCount=0 -->
+      </Script>
+    </ScriptRecord>
+  </ScriptList>
+  <FeatureList>
+    <!-- FeatureCount=2 -->
+    <FeatureRecord index="0">
+      <FeatureTag value="akhn"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="0"/>
+      </Feature>
+    </FeatureRecord>
+    <FeatureRecord index="1">
+      <FeatureTag value="akh2"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="0"/>
+      </Feature>
+    </FeatureRecord>
+  </FeatureList>
+  <LookupList>
+    <!-- LookupCount=1 -->
+    <Lookup index="0">
+      <!-- LookupType=1 -->
+      <LookupFlag value="0"/>
+      <!-- SubTableCount=1 -->
+      <SingleSubst index="0" Format="1">
+        <Substitution in="a" out="b"/>
+      </SingleSubst>
+    </Lookup>
+  </LookupList>
+</GSUB>
diff --git a/Lib/fontTools/mtiLib/testdata/featurename-forward.txt b/Lib/fontTools/mtiLib/testdata/featurename-forward.txt
new file mode 100644
index 000000000..971ca763b
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/featurename-forward.txt
@@ -0,0 +1,14 @@
+
+script table begin
+telu	default		0, 1
+tel2	default		f0, 1
+script table end
+
+feature table begin
+f0	akhn	l1
+1	akh2	l1
+feature table end
+
+lookup	l1	single
+a	b
+lookup end
diff --git a/Lib/fontTools/mtiLib/testdata/lookupnames-backward.ttx.GSUB b/Lib/fontTools/mtiLib/testdata/lookupnames-backward.ttx.GSUB
new file mode 100644
index 000000000..3f90ff27c
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/lookupnames-backward.ttx.GSUB
@@ -0,0 +1,87 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<GSUB>
+  <Version value="0x00010000"/>
+  <ScriptList>
+    <!-- ScriptCount=1 -->
+    <ScriptRecord index="0">
+      <ScriptTag value="telu"/>
+      <Script>
+        <DefaultLangSys>
+          <ReqFeatureIndex value="65535"/>
+          <!-- FeatureCount=2 -->
+          <FeatureIndex index="0" value="0"/>
+          <FeatureIndex index="1" value="1"/>
+        </DefaultLangSys>
+        <!-- LangSysCount=0 -->
+      </Script>
+    </ScriptRecord>
+  </ScriptList>
+  <FeatureList>
+    <!-- FeatureCount=2 -->
+    <FeatureRecord index="0">
+      <FeatureTag value="akhn"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="0"/>
+      </Feature>
+    </FeatureRecord>
+    <FeatureRecord index="1">
+      <FeatureTag value="akh2"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="1"/>
+      </Feature>
+    </FeatureRecord>
+  </FeatureList>
+  <LookupList>
+    <!-- LookupCount=2 -->
+    <Lookup index="0">
+      <!-- LookupType=1 -->
+      <LookupFlag value="0"/>
+      <!-- SubTableCount=1 -->
+      <SingleSubst index="0" Format="1">
+        <Substitution in="uuvowelsignkannada" out="uuvowelsignaltkannada"/>
+        <Substitution in="uvowelsignkannada" out="uvowelsignaltkannada"/>
+      </SingleSubst>
+    </Lookup>
+    <Lookup index="1">
+      <!-- LookupType=6 -->
+      <LookupFlag value="0"/>
+      <!-- SubTableCount=1 -->
+      <ChainContextSubst index="0" Format="2">
+        <Coverage Format="1">
+          <Glyph value="uvowelsignkannada"/>
+          <Glyph value="uuvowelsignkannada"/>
+        </Coverage>
+        <BacktrackClassDef Format="2">
+          <ClassDef glyph="pakannada" class="1"/>
+          <ClassDef glyph="pevowelkannada" class="1"/>
+          <ClassDef glyph="phakannada" class="1"/>
+          <ClassDef glyph="phevowelkannada" class="1"/>
+          <ClassDef glyph="vakannada" class="1"/>
+          <ClassDef glyph="vevowelkannada" class="1"/>
+        </BacktrackClassDef>
+        <InputClassDef Format="1">
+          <ClassDef glyph="uuvowelsignkannada" class="1"/>
+          <ClassDef glyph="uvowelsignkannada" class="1"/>
+        </InputClassDef>
+        <!-- ChainSubClassSetCount=2 -->
+        <ChainSubClassSet index="0" empty="1"/>
+        <ChainSubClassSet index="1">
+          <!-- ChainSubClassRuleCount=1 -->
+          <ChainSubClassRule index="0">
+            <!-- BacktrackGlyphCount=1 -->
+            <Backtrack index="0" value="1"/>
+            <!-- InputGlyphCount=1 -->
+            <!-- LookAheadGlyphCount=0 -->
+            <!-- SubstCount=1 -->
+            <SubstLookupRecord index="0">
+              <SequenceIndex value="0"/>
+              <LookupListIndex value="0"/>
+            </SubstLookupRecord>
+          </ChainSubClassRule>
+        </ChainSubClassSet>
+      </ChainContextSubst>
+    </Lookup>
+  </LookupList>
+</GSUB>
diff --git a/Lib/fontTools/mtiLib/testdata/lookupnames-backward.txt b/Lib/fontTools/mtiLib/testdata/lookupnames-backward.txt
new file mode 100644
index 000000000..067d2c1a7
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/lookupnames-backward.txt
@@ -0,0 +1,36 @@
+
+lookup	l1	single
+
+uvowelsignkannada	uvowelsignaltkannada
+uuvowelsignkannada	uuvowelsignaltkannada
+
+lookup end
+
+lookup	l0	chained
+
+backtrackclass definition begin
+pakannada	1
+phakannada	1
+vakannada	1
+pevowelkannada	1
+phevowelkannada	1
+vevowelkannada	1
+class definition end
+
+class definition begin
+uvowelsignkannada	1
+uuvowelsignkannada	1
+class definition end
+
+class-chain	1	1		1,l1
+
+lookup end
+
+script table begin
+telu	default		0, 1
+script table end
+
+feature table begin
+0	akhn	l1
+1	akh2	l0
+feature table end
diff --git a/Lib/fontTools/mtiLib/testdata/lookupnames-forward.ttx.GSUB b/Lib/fontTools/mtiLib/testdata/lookupnames-forward.ttx.GSUB
new file mode 100644
index 000000000..d40bf1974
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/lookupnames-forward.ttx.GSUB
@@ -0,0 +1,87 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<GSUB>
+  <Version value="0x00010000"/>
+  <ScriptList>
+    <!-- ScriptCount=1 -->
+    <ScriptRecord index="0">
+      <ScriptTag value="telu"/>
+      <Script>
+        <DefaultLangSys>
+          <ReqFeatureIndex value="65535"/>
+          <!-- FeatureCount=2 -->
+          <FeatureIndex index="0" value="0"/>
+          <FeatureIndex index="1" value="1"/>
+        </DefaultLangSys>
+        <!-- LangSysCount=0 -->
+      </Script>
+    </ScriptRecord>
+  </ScriptList>
+  <FeatureList>
+    <!-- FeatureCount=2 -->
+    <FeatureRecord index="0">
+      <FeatureTag value="akhn"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="1"/>
+      </Feature>
+    </FeatureRecord>
+    <FeatureRecord index="1">
+      <FeatureTag value="akh2"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="0"/>
+      </Feature>
+    </FeatureRecord>
+  </FeatureList>
+  <LookupList>
+    <!-- LookupCount=2 -->
+    <Lookup index="0">
+      <!-- LookupType=6 -->
+      <LookupFlag value="0"/>
+      <!-- SubTableCount=1 -->
+      <ChainContextSubst index="0" Format="2">
+        <Coverage Format="1">
+          <Glyph value="uvowelsignkannada"/>
+          <Glyph value="uuvowelsignkannada"/>
+        </Coverage>
+        <BacktrackClassDef Format="2">
+          <ClassDef glyph="pakannada" class="1"/>
+          <ClassDef glyph="pevowelkannada" class="1"/>
+          <ClassDef glyph="phakannada" class="1"/>
+          <ClassDef glyph="phevowelkannada" class="1"/>
+          <ClassDef glyph="vakannada" class="1"/>
+          <ClassDef glyph="vevowelkannada" class="1"/>
+        </BacktrackClassDef>
+        <InputClassDef Format="1">
+          <ClassDef glyph="uuvowelsignkannada" class="1"/>
+          <ClassDef glyph="uvowelsignkannada" class="1"/>
+        </InputClassDef>
+        <!-- ChainSubClassSetCount=2 -->
+        <ChainSubClassSet index="0" empty="1"/>
+        <ChainSubClassSet index="1">
+          <!-- ChainSubClassRuleCount=1 -->
+          <ChainSubClassRule index="0">
+            <!-- BacktrackGlyphCount=1 -->
+            <Backtrack index="0" value="1"/>
+            <!-- InputGlyphCount=1 -->
+            <!-- LookAheadGlyphCount=0 -->
+            <!-- SubstCount=1 -->
+            <SubstLookupRecord index="0">
+              <SequenceIndex value="0"/>
+              <LookupListIndex value="1"/>
+            </SubstLookupRecord>
+          </ChainSubClassRule>
+        </ChainSubClassSet>
+      </ChainContextSubst>
+    </Lookup>
+    <Lookup index="1">
+      <!-- LookupType=1 -->
+      <LookupFlag value="0"/>
+      <!-- SubTableCount=1 -->
+      <SingleSubst index="0" Format="1">
+        <Substitution in="uuvowelsignkannada" out="uuvowelsignaltkannada"/>
+        <Substitution in="uvowelsignkannada" out="uvowelsignaltkannada"/>
+      </SingleSubst>
+    </Lookup>
+  </LookupList>
+</GSUB>
diff --git a/Lib/fontTools/mtiLib/testdata/lookupnames-forward.txt b/Lib/fontTools/mtiLib/testdata/lookupnames-forward.txt
new file mode 100644
index 000000000..16c981929
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/lookupnames-forward.txt
@@ -0,0 +1,36 @@
+
+lookup	l0	chained
+
+backtrackclass definition begin
+pakannada	1
+phakannada	1
+vakannada	1
+pevowelkannada	1
+phevowelkannada	1
+vevowelkannada	1
+class definition end
+
+class definition begin
+uvowelsignkannada	1
+uuvowelsignkannada	1
+class definition end
+
+class-chain	1	1		1,l1
+
+lookup end
+
+script table begin
+telu	default		0, 1
+script table end
+
+lookup	l1	single
+
+uvowelsignkannada	uvowelsignaltkannada
+uuvowelsignkannada	uuvowelsignaltkannada
+
+lookup end
+
+feature table begin
+0	akhn	l1
+1	akh2	l0
+feature table end
diff --git a/Lib/fontTools/mtiLib/testdata/mixed-toplevels.ttx.GSUB b/Lib/fontTools/mtiLib/testdata/mixed-toplevels.ttx.GSUB
new file mode 100644
index 000000000..d40bf1974
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/mixed-toplevels.ttx.GSUB
@@ -0,0 +1,87 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<GSUB>
+  <Version value="0x00010000"/>
+  <ScriptList>
+    <!-- ScriptCount=1 -->
+    <ScriptRecord index="0">
+      <ScriptTag value="telu"/>
+      <Script>
+        <DefaultLangSys>
+          <ReqFeatureIndex value="65535"/>
+          <!-- FeatureCount=2 -->
+          <FeatureIndex index="0" value="0"/>
+          <FeatureIndex index="1" value="1"/>
+        </DefaultLangSys>
+        <!-- LangSysCount=0 -->
+      </Script>
+    </ScriptRecord>
+  </ScriptList>
+  <FeatureList>
+    <!-- FeatureCount=2 -->
+    <FeatureRecord index="0">
+      <FeatureTag value="akhn"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="1"/>
+      </Feature>
+    </FeatureRecord>
+    <FeatureRecord index="1">
+      <FeatureTag value="akh2"/>
+      <Feature>
+        <!-- LookupCount=1 -->
+        <LookupListIndex index="0" value="0"/>
+      </Feature>
+    </FeatureRecord>
+  </FeatureList>
+  <LookupList>
+    <!-- LookupCount=2 -->
+    <Lookup index="0">
+      <!-- LookupType=6 -->
+      <LookupFlag value="0"/>
+      <!-- SubTableCount=1 -->
+      <ChainContextSubst index="0" Format="2">
+        <Coverage Format="1">
+          <Glyph value="uvowelsignkannada"/>
+          <Glyph value="uuvowelsignkannada"/>
+        </Coverage>
+        <BacktrackClassDef Format="2">
+          <ClassDef glyph="pakannada" class="1"/>
+          <ClassDef glyph="pevowelkannada" class="1"/>
+          <ClassDef glyph="phakannada" class="1"/>
+          <ClassDef glyph="phevowelkannada" class="1"/>
+          <ClassDef glyph="vakannada" class="1"/>
+          <ClassDef glyph="vevowelkannada" class="1"/>
+        </BacktrackClassDef>
+        <InputClassDef Format="1">
+          <ClassDef glyph="uuvowelsignkannada" class="1"/>
+          <ClassDef glyph="uvowelsignkannada" class="1"/>
+        </InputClassDef>
+        <!-- ChainSubClassSetCount=2 -->
+        <ChainSubClassSet index="0" empty="1"/>
+        <ChainSubClassSet index="1">
+          <!-- ChainSubClassRuleCount=1 -->
+          <ChainSubClassRule index="0">
+            <!-- BacktrackGlyphCount=1 -->
+            <Backtrack index="0" value="1"/>
+            <!-- InputGlyphCount=1 -->
+            <!-- LookAheadGlyphCount=0 -->
+            <!-- SubstCount=1 -->
+            <SubstLookupRecord index="0">
+              <SequenceIndex value="0"/>
+              <LookupListIndex value="1"/>
+            </SubstLookupRecord>
+          </ChainSubClassRule>
+        </ChainSubClassSet>
+      </ChainContextSubst>
+    </Lookup>
+    <Lookup index="1">
+      <!-- LookupType=1 -->
+      <LookupFlag value="0"/>
+      <!-- SubTableCount=1 -->
+      <SingleSubst index="0" Format="1">
+        <Substitution in="uuvowelsignkannada" out="uuvowelsignaltkannada"/>
+        <Substitution in="uvowelsignkannada" out="uvowelsignaltkannada"/>
+      </SingleSubst>
+    </Lookup>
+  </LookupList>
+</GSUB>
diff --git a/Lib/fontTools/mtiLib/testdata/mixed-toplevels.txt b/Lib/fontTools/mtiLib/testdata/mixed-toplevels.txt
new file mode 100644
index 000000000..dc090575d
--- /dev/null
+++ b/Lib/fontTools/mtiLib/testdata/mixed-toplevels.txt
@@ -0,0 +1,36 @@
+
+lookup	0	chained
+
+backtrackclass definition begin
+pakannada	1
+phakannada	1
+vakannada	1
+pevowelkannada	1
+phevowelkannada	1
+vevowelkannada	1
+class definition end
+
+class definition begin
+uvowelsignkannada	1
+uuvowelsignkannada	1
+class definition end
+
+class-chain	1	1		1,1
+
+lookup end
+
+script table begin
+telu	default		0, 1
+script table end
+
+lookup	1	single
+
+uvowelsignkannada	uvowelsignaltkannada
+uuvowelsignkannada	uuvowelsignaltkannada
+
+lookup end
+
+feature table begin
+0	akhn	1
+1	akh2	0
+feature table end