emit only a WARNing for bad fonts on test/163

(issue fonttools#1621)
felipesanches · Oct 26, 2017 · bf5df19 · bf5df19
1 parent 1052757
commit bf5df19
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/Lib/fontbakery/specifications/googlefonts.py b/Lib/fontbakery/specifications/googlefonts.py
@@ -4680,7 +4680,7 @@ def com_google_fonts_test_163(ttFont):
                                                 platformID=plat):
       if len(familyname_str + stylename_str) > 20:
         failed = True
-        yield FAIL, ("The combined length of family and style"
+        yield WARN, ("The combined length of family and style"
                      " exceeds 20 chars in the following '{}' entries:"
                      " FONT_FAMILY_NAME = '{}' / SUBFAMILY_NAME = '{}'"
                      "").format(PLATID_STR[plat],

diff --git a/Lib/fontbakery/specifications/googlefonts_test.py b/Lib/fontbakery/specifications/googlefonts_test.py
@@ -2187,7 +2187,7 @@ def test_id_163():
   status, message = list(test(ttFont))[-1]
   assert status == PASS
 
-  # Then we FAIL with the long family/style names
+  # Then we emit a WARNing with the long family/style names
   # that were used as an example on the glyphs tutorial
   # (at https://glyphsapp.com/tutorials/multiple-masters-part-3-setting-up-instances):
   for index, name in enumerate(ttFont["name"].names):
@@ -2200,9 +2200,9 @@ def test_id_163():
       ttFont["name"].names[index].string = "WithAVeryLongStyleName".encode(name.getEncoding())
       break
 
-  print ("Test FAIL with a bad font...")
+  print ("Test WARN with a bad font...")
   status, message = list(test(ttFont))[-1]
-  assert status == FAIL
+  assert status == WARN
 
 
 def test_id_164():