Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29321036: Noissue - Simplify notification handler tests (Closed)

Created:
June 23, 2015, 2:04 p.m. by Felix Dahlke
Modified:
June 23, 2015, 2:26 p.m.
Reviewers:
Sebastian Noack
CC:
Wladimir Palant
Visibility:
Public.

Description

Noissue - Simplify notification handler tests

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -41 lines) Patch
M sitescripts/notifications/test/notification.py View 12 chunks +33 lines, -41 lines 0 comments Download

Messages

Total messages: 3
Felix Dahlke
Realised we can get rid of some duplication here, didn't seem worth an issue.
June 23, 2015, 2:16 p.m. (2015-06-23 14:16:14 UTC) #1
Sebastian Noack
LGTM
June 23, 2015, 2:19 p.m. (2015-06-23 14:19:44 UTC) #2
Felix Dahlke
June 23, 2015, 2:26 p.m. (2015-06-23 14:26:27 UTC) #3

Powered by Google App Engine
This is Rietveld