Skip to content

Commit

Permalink
Merge pull request #133 from slafs/remove-port-assertion
Browse files Browse the repository at this point in the history
remove port assertion
  • Loading branch information
msabramo committed Feb 20, 2015
2 parents 996a3cf + 07e213c commit d1998a8
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 6 deletions.
3 changes: 0 additions & 3 deletions honcho/environ.py
Original file line number Diff line number Diff line change
Expand Up @@ -110,9 +110,6 @@ def expand_processes(processes, concurrency=None, env=None, quiet=None, port=Non
if env is not None and env.get("PORT") is not None:
port = int(env.get("PORT"))

if port is not None:
assert port % 1000 == 0, "port must be multiple of 1000"

if quiet is None:
quiet = []

Expand Down
3 changes: 0 additions & 3 deletions honcho/test/unit/test_environ.py
Original file line number Diff line number Diff line change
Expand Up @@ -206,9 +206,6 @@ def test_port(self):
p = ep(("foo", "some command"), port=8000)
self.assertEqual({"PORT": "8000"}, p[0].env)

def test_port_invalid(self):
self.assertRaises(AssertionError, ep, ("foo", "test"), port=5123)

def test_port_multiple(self):
p = ep(("foo", "some command"),
("bar", "another command"),
Expand Down

0 comments on commit d1998a8

Please sign in to comment.