Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can this message be changed to
FreeBSD does not support V4MAPPED flag.
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@thefourtheye if so, perhaps squeeze both
test_lookup_ipv6_hint
andv4mapped
in there? wouldn't tap just reportinternet/test-dns.js
and not the actual test name.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jbergstroem Unfortunately, nope. It will not know the specific test :( I thought that if we specified the actual reason, it would be better to understand why it is skipped.
Now there is another interesting thing here. If we skip more than one tests in a single file, then there will be two skip TAP lines. I wonder how the output will be displayed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@thefourtheye yeah, not too much to do about "double skipping" unless we're keen to start rewriting larger parts of our tests. Long term we should look at replacing the current runner with something build on top of
node-tap
or similar. There's an issue about this somewhere..anyway, moving forward; I'm ok with either message.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cheers