Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Commit

Permalink
fix: test_bad_senderidlist now requires --gcm_enabled.
Browse files Browse the repository at this point in the history
bandaid handling of make_settings failures for now

closes #544
  • Loading branch information
pjenvey committed Jul 25, 2016
1 parent a2e1f8c commit 949f8d3
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
4 changes: 4 additions & 0 deletions autopush/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -424,6 +424,8 @@ def connection_main(sysargs=None, use_files=True):
hello_timeout=args.hello_timeout,
preflight_uaid="deadbeef000000000deadbeef" + postfix,
)
if not settings:
return 1 # pragma: nocover

r = RouterHandler
r.ap_settings = settings
Expand Down Expand Up @@ -525,6 +527,8 @@ def endpoint_main(sysargs=None, use_files=True):
bear_hash_key=args.auth_key,
preflight_uaid="deadbeef000000000deadbeef" + postfix,
)
if not settings:
return 1

# Endpoint HTTP router
site = cyclone.web.Application([
Expand Down
3 changes: 2 additions & 1 deletion autopush/tests/test_main.py
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,8 @@ def test_ssl(self):
], False)

def test_bad_senderidlist(self):
endpoint_main([
assert endpoint_main([
"--gcm_enabled",
"--senderid_list='[Invalid'"
], False)

Expand Down

0 comments on commit 949f8d3

Please sign in to comment.