Skip to content

Commit

Permalink
jinja diff is not right so note that
Browse files Browse the repository at this point in the history
  • Loading branch information
git-user committed Mar 17, 2021
1 parent d0e165e commit e582f7c
Showing 1 changed file with 11 additions and 3 deletions.
14 changes: 11 additions & 3 deletions pykern/pkunit.py
Expand Up @@ -167,15 +167,23 @@ def file_eq(expect_path, *args, **kwargs):
if e == actual:
return
c = f"diff '{expect_path}' '{actual_path}'"
if j:
x = '''
Implementation restriction: The jinja values are not filled in the diff
so the actual can't be copied to the expected to fix things.
'''
else:
x = '''
to update test data:
cp '{actual_path}' '{expect_path}'
'''
with os.popen(c) as f:
pkfail(
'{}',
f'''expect != actual:
{c}
{f.read()}
to update test data:
cp '{actual_path}' '{expect_path}'
{x}
'''
)

Expand Down

0 comments on commit e582f7c

Please sign in to comment.