-
Notifications
You must be signed in to change notification settings - Fork 168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[tx] Make CharString length check a warning rather than a fatal error #446
Conversation
@cjchapman I think I know why Travis didn't kick in. Looking into it now. |
(also incorporated some feedback from @miguelsousa, re: test name, comments, etc.)
Tests/tx_test.py
Outdated
actual_path = runner(CMD + ['-o', '0', '-f', 'CJK-VarTest.otf']) | ||
expected_path = _get_expected_path('CJK-VarTest.txt') | ||
skip = ['-s', '## Filename'] | ||
assert differ([expected_path, actual_path] + skip) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you don't need to define skip
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I do. If I comment it out, I get:
def test_long_charstring_bug444():
# read a CFF2 VF with a charstring longer that 65535, check output
actual_path = runner(CMD + ['-o', '0', '-f', 'CJK-VarTest.otf'])
expected_path = _get_expected_path('CJK-VarTest.txt')
# skip = ['-s', '## Filename']
> assert differ([expected_path, actual_path] + skip)
E NameError: global name 'skip' is not defined
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry what I meant was, just do
assert differ([expected_path, actual_path, '-s', '## Filename'])
@cjchapman it just occurred to me that |
This is the writing counterpart of #446
This is the writing counterpart of #446
This is the writing counterpart of #446
Fixes #444