Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests/pkg/lwip: Add README.md #19949

Merged
merged 1 commit into from
Sep 27, 2023
Merged

Conversation

miri64
Copy link
Member

@miri64 miri64 commented Sep 27, 2023

Contribution description

While tests/pkg/lwip is automated, it is only so for native and IPv6. This adds some doc how to test for other modes, e.g. 6LoWPAN and IPv4.

Testing procedure

Read, follow the instructions, give feedback what can be improved :-).

Issues/PRs references

None

@miri64 miri64 added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: network Area: Networking Area: doc Area: Documentation Area: tests Area: tests and testing framework CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Sep 27, 2023
@github-actions github-actions bot removed the Area: network Area: Networking label Sep 27, 2023
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Sep 27, 2023
@riot-ci
Copy link

riot-ci commented Sep 27, 2023

Murdock results

✔️ PASSED

cb2287d tests/pkg/lwip: Add README.md

Success Failures Total Runtime
1 0 1 01m:06s

Artifacts


```console
$ BOARD="<board>" make flash test
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we also add

# Instead of simulating an Ethernet connection, we can also simulate
# an IEEE 802.15.4 radio using ZEP
USE_ZEP ?= 0

# set the ZEP port for native
ZEP_PORT_BASE ?= 17754
ifeq (1,$(USE_ZEP))
  TERMFLAGS += -z [::1]:$(ZEP_PORT_BASE)
  USEMODULE += socket_zep

  ifneq (,$(ZEP_MAC))
    TERMFLAGS += --eui64=$(ZEP_MAC)
  endif
endif

to the Makefile so we can just set USE_ZEP=1 for 6LoWPAN tests on native

Copy link
Member Author

Choose a reason for hiding this comment

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

You can try, but let's do it in a separate PR :-).

@miri64
Copy link
Member Author

miri64 commented Sep 27, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Sep 27, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 71232ed into RIOT-OS:master Sep 27, 2023
34 checks passed
@miri64 miri64 deleted the tests/doc/lwip-README branch September 27, 2023 16:53
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.10 milestone Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: doc Area: Documentation 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 CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants