Skip to content

tests: fix tests/gnrc_netif for changes in #6895#8255

Merged
bergzand merged 1 commit intoRIOT-OS:masterfrom
miri64:tests/fix/regression-fa64e933
Dec 13, 2017
Merged

tests: fix tests/gnrc_netif for changes in #6895#8255
bergzand merged 1 commit intoRIOT-OS:masterfrom
miri64:tests/fix/regression-fa64e933

Conversation

@miri64
Copy link
Copy Markdown
Member

@miri64 miri64 commented Dec 13, 2017

Contribution description

#6895 changed the output of gnrc_netif_hdr_print(), so these tests fail now on master since their checks were too restrictive.

Issues/PRs references

Fixes test "regression" introduced in #6895.

@miri64 miri64 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: tests Area: tests and testing framework labels Dec 13, 2017
@miri64 miri64 requested a review from bergzand December 13, 2017 14:43
@miri64 miri64 force-pushed the tests/fix/regression-fa64e933 branch from 37bffda to eb08a50 Compare December 13, 2017 14:44
Copy link
Copy Markdown
Member

@bergzand bergzand left a comment

Choose a reason for hiding this comment

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

ACK

@bergzand bergzand added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Dec 13, 2017
@bergzand
Copy link
Copy Markdown
Member

Sorry for breaking the test by the way :(

@bergzand bergzand merged commit d1aa688 into RIOT-OS:master Dec 13, 2017
@miri64 miri64 deleted the tests/fix/regression-fa64e933 branch December 13, 2017 15:33
@miri64
Copy link
Copy Markdown
Member Author

miri64 commented Dec 13, 2017

nP. It's not run, so you couldn't have known ;-).

@aabadie aabadie added this to the Release 2018.01 milestone Jan 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants