Skip to content

test1248: use %NOLISTENPORT instead of a port of a non-running server - #5318

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/nolisten
Closed

test1248: use %NOLISTENPORT instead of a port of a non-running server#5318
bagder wants to merge 2 commits into
masterfrom
bagder/nolisten

Conversation

@bagder

@bagder bagder commented May 1, 2020

Copy link
Copy Markdown
Member

Otherwise the test fails when run stand-alone!

@bagder bagder added the tests label May 1, 2020

@cvengler cvengler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch!

@bagder

bagder commented May 2, 2020

Copy link
Copy Markdown
Member Author

I wrote a shell script that runs each available test one by one. It takes forever but it finds these problems pretty well...

@bagder bagder closed this in f9c6feb May 2, 2020
@bagder
bagder deleted the bagder/nolisten branch May 10, 2020 21:34
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