new tests for percent sub command for otf and ttf files

Author: chrissimpkins

README.md

@@ -1,4 +1,4 @@
-## font-line [![Build Status](https://travis-ci.org/source-foundry/font-line.svg?branch=master)](https://travis-ci.org/source-foundry/font-line)  [![Build status](https://ci.appveyor.com/api/projects/status/2s4725o5mxh2298c/branch/master?svg=true)](https://ci.appveyor.com/project/chrissimpkins/font-line/branch/master)
+## font-line [![Build Status](https://travis-ci.org/source-foundry/font-line.svg?branch=master)](https://travis-ci.org/source-foundry/font-line)  [![Build status](https://ci.appveyor.com/api/projects/status/2s4725o5mxh2298c/branch/master?svg=true)](https://ci.appveyor.com/project/chrissimpkins/font-line/branch/master) [![codecov.io](https://codecov.io/github/source-foundry/font-line/coverage.svg?branch=master)](https://codecov.io/github/source-foundry/font-line?branch=master)
 
 
 ### About

coverage.sh

@@ -4,5 +4,5 @@ coverage run --source fontline -m py.test
 coverage report -m
 coverage html
 
-#coverage xml
-#codecov --token=$CODECOV_{{fontline}}
+coverage xml
+codecov --token=$CODECOV_FONTLINE

tests/test_commands.py

@@ -7,7 +7,7 @@
 from fontline.commands import get_linegap_percent_filepath
 
 
-def test_commands_function_getlg_percent_filepath():
+def test_commands_module_function_getlg_percent_filepath():
     response = get_linegap_percent_filepath("Test.ttf", "20")
     assert response == "Test-linegap20.ttf"
 

tests/test_percent_cmd.py

@@ -5,10 +5,33 @@
 import pytest
 import os.path
 import os
+import shutil
+from fontTools import ttLib
+
+from fontline.utilities import file_exists
+
+# ///////////////////////////////////////////////////////
+#
+# utility functions for module tests
+#
+# ///////////////////////////////////////////////////////
+
+
+def create_test_file(filepath):
+    filepath_list = filepath.split(".")
+    testpath = filepath_list[0] + "-test." + filepath_list[1]
+    shutil.copyfile(filepath, testpath)
+    return True
+
+
+def erase_test_file(filepath):
+    os.remove(filepath)
+    return True
+
 
 # ///////////////////////////////////////////////////////
 #
-# report sub-command tests
+# percent sub-command command line logic tests
 #
 # ///////////////////////////////////////////////////////
 
@@ -72,3 +95,101 @@ def test_percent_cmd_font_file_wrong_filetype(capsys):
     main()
     out, err = capsys.readouterr()
     assert ("does not appear to be a supported font file type." in err) is True
+
+
+# ///////////////////////////////////////////////////////
+#
+# percent sub-command command functionality tests
+#
+# ///////////////////////////////////////////////////////
+
+def test_percent_cmd_ttf_file_10_percent(capsys):
+    try:
+        from fontline.app import main
+        fontpath = os.path.join('tests', 'testingfiles', 'Hack-Regular.ttf')
+        testpath = os.path.join('tests', 'testingfiles', 'Hack-Regular-test.ttf')
+        newfont_path = os.path.join('tests', 'testingfiles', 'Hack-Regular-test-linegap10.ttf')
+        create_test_file(fontpath)
+        assert file_exists(testpath) is True
+        sys.argv = ['font-line', 'percent', '10', testpath]
+        main()
+
+        assert file_exists(newfont_path)
+
+        tt = ttLib.TTFont(newfont_path)
+
+        os2_typo_ascender = tt['OS/2'].__dict__['sTypoAscender']
+        os2_typo_descender = tt['OS/2'].__dict__['sTypoDescender']
+        os2_win_ascent = tt['OS/2'].__dict__['usWinAscent']
+        os2_win_descent = tt['OS/2'].__dict__['usWinDescent']
+        os2_typo_linegap = tt['OS/2'].__dict__['sTypoLineGap']
+        hhea_ascent = tt['hhea'].__dict__['ascent']
+        hhea_descent = tt['hhea'].__dict__['descent']
+        hhea_linegap = tt['hhea'].__dict__['lineGap']
+        units_per_em = tt['head'].__dict__['unitsPerEm']
+
+        assert os2_typo_ascender == 1556
+        assert os2_typo_descender == -492
+        assert os2_win_ascent == 1658
+        assert os2_win_descent == 594
+        assert units_per_em == 2048
+        assert os2_typo_linegap == 204
+        assert hhea_ascent == 1658
+        assert hhea_descent == -594
+        assert hhea_linegap == 0
+
+        erase_test_file(testpath)
+        erase_test_file(newfont_path)
+    except Exception as e:
+        # cleanup test files
+        if file_exists(testpath):
+            erase_test_file(testpath)
+        if file_exists(newfont_path):
+            erase_test_file(newfont_path)
+        raise e
+
+
+def test_percent_cmd_otf_file_10_percent(capsys):
+    try:
+        from fontline.app import main
+        fontpath = os.path.join('tests', 'testingfiles', 'Hack-Regular.otf')
+        testpath = os.path.join('tests', 'testingfiles', 'Hack-Regular-test.otf')
+        newfont_path = os.path.join('tests', 'testingfiles', 'Hack-Regular-test-linegap10.otf')
+        create_test_file(fontpath)
+        assert file_exists(testpath) is True
+        sys.argv = ['font-line', 'percent', '10', testpath]
+        main()
+
+        assert file_exists(newfont_path)
+
+        tt = ttLib.TTFont(newfont_path)
+
+        os2_typo_ascender = tt['OS/2'].__dict__['sTypoAscender']
+        os2_typo_descender = tt['OS/2'].__dict__['sTypoDescender']
+        os2_win_ascent = tt['OS/2'].__dict__['usWinAscent']
+        os2_win_descent = tt['OS/2'].__dict__['usWinDescent']
+        os2_typo_linegap = tt['OS/2'].__dict__['sTypoLineGap']
+        hhea_ascent = tt['hhea'].__dict__['ascent']
+        hhea_descent = tt['hhea'].__dict__['descent']
+        hhea_linegap = tt['hhea'].__dict__['lineGap']
+        units_per_em = tt['head'].__dict__['unitsPerEm']
+
+        assert os2_typo_ascender == 1556
+        assert os2_typo_descender == -492
+        assert os2_win_ascent == 1658
+        assert os2_win_descent == 594
+        assert units_per_em == 2048
+        assert os2_typo_linegap == 204
+        assert hhea_ascent == 1658
+        assert hhea_descent == -594
+        assert hhea_linegap == 0
+
+        erase_test_file(testpath)
+        erase_test_file(newfont_path)
+    except Exception as e:
+        # cleanup test files
+        if file_exists(testpath):
+            erase_test_file(testpath)
+        if file_exists(newfont_path):
+            erase_test_file(newfont_path)
+        raise e

tests/testingfiles/backup/Hack-Regular.otf

@@ -0,0 +1 @@
+This is a file for testing.