Fix bug where diff and diff_full were switched in notification templates. (#380)

pull/385/head
Travis Howse 3 years ago committed by GitHub
parent af8efbb6d2
commit abe88192f4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -123,7 +123,8 @@ def test_check_notification(client, live_server):
assert test_url in notification_submission assert test_url in notification_submission
# Diff was correctly executed # Diff was correctly executed
assert "Diff Full: (changed) Which is across multiple lines" in notification_submission assert "Diff Full: Some initial text" in notification_submission
assert "Diff: (changed) Which is across multiple lines" in notification_submission
assert "(-> into) which has this one new line" in notification_submission assert "(-> into) which has this one new line" in notification_submission

@ -127,8 +127,8 @@ class update_worker(threading.Thread):
'watch_url': watch['url'], 'watch_url': watch['url'],
'uuid': uuid, 'uuid': uuid,
'current_snapshot': contents.decode('utf-8'), 'current_snapshot': contents.decode('utf-8'),
'diff_full': diff.render_diff(prev_fname, fname, line_feed_sep=line_feed_sep), 'diff': diff.render_diff(prev_fname, fname, line_feed_sep=line_feed_sep),
'diff': diff.render_diff(prev_fname, fname, True, line_feed_sep=line_feed_sep) 'diff_full': diff.render_diff(prev_fname, fname, True, line_feed_sep=line_feed_sep)
}) })
self.notification_q.put(n_object) self.notification_q.put(n_object)

Loading…
Cancel
Save