Skip to content

tests: add %NOLISTENPORT and use it#5270

Closed
bagder wants to merge 2 commits intomasterfrom
bagder/tests-nolistenport
Closed

tests: add %NOLISTENPORT and use it#5270
bagder wants to merge 2 commits intomasterfrom
bagder/tests-nolistenport

Conversation

@bagder
Copy link
Copy Markdown
Member

@bagder bagder commented Apr 20, 2020

The purpose with this variable is to provide a port number that is
reasonably likely to not have a listener on the local host so that tests
can try connect failures against it. It uses port 2.

Updated six tests to use it instead of the previous different ports.

The purpose with this variable is to provide a port number that is
reasonably likely to not have a listener on the local host so that tests
can try connect failures against it. It uses port 2.

Updated six tests to use it instead of the previous different ports.
@bagder bagder added the tests label Apr 20, 2020
@bagder bagder closed this in 018dd77 Apr 20, 2020
@bagder
Copy link
Copy Markdown
Member Author

bagder commented Apr 20, 2020

thanks @emilengler

@bagder bagder deleted the bagder/tests-nolistenport branch April 20, 2020 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants