Skip to content

Commit e34a213

Browse files
committed
Use a less common port than 4567 in script/test
If someone already has a Sinatra app spun up in development environment, `script/test` will fail because it will try to use the same port. Use port 3999 for testing because it's less likely to be used by another app.
1 parent 51f03dd commit e34a213

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

script/server

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
#!/usr/bin/env bash
22
set -e
33

4-
exec bundle exec ruby ./test/app.rb
4+
exec bundle exec ruby ./test/app.rb "$@"

script/test

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
#!/usr/bin/env bash
22
set -e
33

4-
port=4567
5-
script/server &>/dev/null &
4+
port=3999
5+
script/server -p "$port" &>/dev/null &
66
pid=$!
77

88
trap "kill $pid" EXIT INT

0 commit comments

Comments
 (0)