Skip to content

[nrf fromtree] Fix TestInetEndPoint to pass even when computer is offline#612

Merged
LuDuda merged 1 commit into
nrfconnect:masterfrom
adigie:fix-failing-test
May 16, 2025
Merged

[nrf fromtree] Fix TestInetEndPoint to pass even when computer is offline#612
LuDuda merged 1 commit into
nrfconnect:masterfrom
adigie:fix-failing-test

Conversation

@adigie
Copy link
Copy Markdown
Member

@adigie adigie commented May 16, 2025

Description:

  • TestInetEndPoint was not hermetic, it required at least one network interface connected to a link and up.
  • There was a comment before to skip tests if the link was not up, but it was not actually skipping the test. This made it fail for me when working offline and trying to run unit tests.

Testing done:

  • Test passes with and without link up.

(cherry picked from commit b2a86bd)

…line (#38015)

Description:

- TestInetEndPoint was not hermetic, it required at least one network
  interface connected to a link and up.
- There was a comment before to skip tests if the link was not up,
  but it was not actually skipping the test. This made it fail for
  me when working offline and trying to run unit tests.

Testing done:
- Test passes with and without link up.

(cherry picked from commit b2a86bd)
Signed-off-by: Adrian Gielniewski <adrian.gielniewski@nordicsemi.no>
@adigie adigie requested a review from a team as a code owner May 16, 2025 11:10
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@LuDuda LuDuda merged commit 00413ab into nrfconnect:master May 16, 2025
9 of 10 checks passed
@adigie adigie deleted the fix-failing-test branch May 16, 2025 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants