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/net: add stub test for gcoap_forward_proxy #20909

Merged
merged 2 commits into from
Oct 14, 2024

Conversation

maribu
Copy link
Member

@maribu maribu commented Oct 14, 2024

Contribution description

This adds enough to do compile testing for gcoap_forward_proxy. A proper test would still be needed, though.

This still already exposed compilation issues.

Testing procedure

The test is a stub to at least get compilation testing. It is not intended to be flashed and run.

So: Green CI

Issues/PRs references

A step towards fixing #20910

@maribu maribu added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 14, 2024
@github-actions github-actions bot added Area: doc Area: Documentation Area: tests Area: tests and testing framework labels Oct 14, 2024
@riot-ci
Copy link

riot-ci commented Oct 14, 2024

Murdock results

✔️ PASSED

a671229 tests/net: add stub test for gcoap_forward_proxy

Success Failures Total Runtime
10214 0 10215 13m:30s

Artifacts

@benpicco benpicco requested a review from miri64 October 14, 2024 12:31
@maribu maribu force-pushed the tests/net/gcoap_forward_proxy branch from 7b86592 to 7c32c39 Compare October 14, 2024 14:14
@github-actions github-actions bot added Area: network Area: Networking Area: CoAP Area: Constrained Application Protocol implementations Area: sys Area: System labels Oct 14, 2024
tests/net/gcoap_forward_proxy/Makefile Outdated Show resolved Hide resolved
tests/net/gcoap_forward_proxy/Makefile Outdated Show resolved Hide resolved
This adds enough to do compile testing for gcoap_forward_proxy.
A proper test would still be needed, though.

This still already exposed compilation issues.
@maribu maribu force-pushed the tests/net/gcoap_forward_proxy branch from 7c32c39 to a671229 Compare October 14, 2024 14:33
@maribu
Copy link
Member Author

maribu commented Oct 14, 2024

Cleaned up copy-pasta leftovers in the Makefile and squashed

@miri64 miri64 enabled auto-merge October 14, 2024 14:50
@miri64 miri64 added this pull request to the merge queue Oct 14, 2024
@miri64 miri64 added the Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation label Oct 14, 2024
Merged via the queue into RIOT-OS:master with commit 5094f03 Oct 14, 2024
27 checks passed
@maribu maribu deleted the tests/net/gcoap_forward_proxy branch October 14, 2024 20:27
@benpicco benpicco added this to the Release 2024.10 milestone Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: CoAP Area: Constrained Application Protocol implementations Area: doc Area: Documentation Area: network Area: Networking Area: sys Area: System 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: 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