Repository navigation

Use a port picker to prevent port reuse #49

Closed

Labels

opened

on Dec 14, 2015

When running npm test, at least one test appears to hold on to port 8080 causing later tests to fail.

Rather than using port 8080 for all tests a port picker should be used for each test so that they don't interfere with each other.

jmdobry commented on Dec 14, 2015
jmdobry
on Dec 14, 2015
Member
What platform are you running the tests on? Also, how fast of a machine? I wonder if it isn't a race condition.
jmdobry commented on Dec 14, 2015
jmdobry
on Dec 14, 2015
Member
So I figured it out (and this is why I need to add instructions), but one of the tests attempts to connect to Redis, and another attempts to connect to memcached. Without both of those running, you'll get the Error: bind EADDRINUSE null:8080 error.
jmdobry
added a commit that references this issue on Dec 14, 2015
Added instructions for running the tests. Fixes #49.
03f438e
jmdobry
closed this as completedin 03f438e on Dec 14, 2015
jmdobry
added a commit that references this issue on Dec 14, 2015
Merge pull request #50 from GoogleCloudPlatform/test-instructions
dade986
sethbergman
added a commit that references this issue on Jun 4, 2017
Refactor OAuth2 to use Passport.js (GoogleCloudPlatform#49)
d8adabf
yoshi-automation
added
triage meI really want to be triaged.

🚨This issue needs some love.
on Apr 7, 2020
kweinmeister
added 3 commits that reference this issue on Nov 7, 2022
chore(deps): update dependency mocha to v8 (#49)
1834906
chore(deps): update dependency mocha to v8 (#49)
5163fde
chore(deps): update dependency mocha to v8 (#49)
678ce5c
telpirion
added a commit that references this issue on Nov 9, 2022
samples: create topic model (#49)
ab4b34d
gcf-merge-on-green
added a commit that references this issue on Nov 9, 2022
chore(contact-center): migrate code from googleapis/nodejs-contact-ce…
bef7132
grayside
added 2 commits that reference this issue on Nov 10, 2022
chore: release 1.3.2 (#49)
03c0d9e
chore: release 1.3.2 (#49)
007b696
6 remaining items
ace-n
added 3 commits that reference this issue on Nov 17, 2022
chore: update sample lockfiles (#49)
c7ddc77
release 1.3.0 (#49)
8efab69
chore(deps): lock file maintenance (#49)
1521be5
grayside
added a commit that references this issue on Nov 17, 2022
chore(deps): lock file maintenance (#49)
ef284e4
ahrarmonsur
added 2 commits that reference this issue on Nov 17, 2022
chore: update sample lockfiles (#49)
9e31bbc
chore(deps): lock file maintenance (#49)
888dda5
ace-n
added 2 commits that reference this issue on Nov 17, 2022
migrate code from googleapis/nodejs-speech (#2857)
624d200
chore: release 2.2.0 (#45)
1cd4e6e
ahrarmonsur
added 2 commits that reference this issue on Nov 18, 2022
fix(deps): update dependency yargs to v11 (#49)
0c66ec1
fix(deps): update dependency yargs to v11 (#49)
2ce36a7
NimJay
added 3 commits that reference this issue on Nov 18, 2022
fix(deps): update dependency yargs to v11 (#49)
16ef0ab
chore: release 1.3.0 (#49)
eade272
chore: release 1.3.0 (#49)
1837abc
ace-n
added a commit that references this issue on Nov 21, 2022
migrate code from googleapis/nodejs-dialogflow-cx (#2854)
c0020e4
jsimonweb
added a commit that references this issue on Dec 12, 2022
chore: bump to next dev version (GoogleCloudPlatform#49)
cf4b4a6

to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

No one assigned

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests