Skip to content

tests/heatshrink: move from unittests to regular test#10202

Merged
bergzand merged 1 commit intoRIOT-OS:masterfrom
smlng:pr/tests/heatshrink
Oct 19, 2018
Merged

tests/heatshrink: move from unittests to regular test#10202
bergzand merged 1 commit intoRIOT-OS:masterfrom
smlng:pr/tests/heatshrink

Conversation

@smlng
Copy link
Copy Markdown
Member

@smlng smlng commented Oct 19, 2018

Contribution description

This moves tests for the heatshrink package from unittests to a regular
test, which should help to decrease binary size of unittests.

Testing procedure

run the tests, i.e. BOARD=<your-favourite> make -C tests/pkg_heatshrink flash test

Issues/PRs references

@smlng smlng added Area: tests Area: tests and testing framework Area: pkg Area: External package ports Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Oct 19, 2018
@smlng smlng requested a review from bergzand October 19, 2018 10:15
@smlng smlng force-pushed the pr/tests/heatshrink branch from ac12f26 to e39d7c2 Compare October 19, 2018 10:19
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.

I think tests/pkg_heatshrink/Makefile.include can be fully removed here. Otherwise ACK, feel free to directly squash the changes.

@smlng
Copy link
Copy Markdown
Member Author

smlng commented Oct 19, 2018

@bergzand yes, my bad ...

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, please squash

@bergzand
Copy link
Copy Markdown
Member

Looks like the arduino-{duemilanove,uno} and the nucleo-f031k6 have to be added to the insufficient memory list

@smlng smlng force-pushed the pr/tests/heatshrink branch from e26565c to 1e5faed Compare October 19, 2018 12:53
@bergzand
Copy link
Copy Markdown
Member

Re-ACK :)

@smlng
Copy link
Copy Markdown
Member Author

smlng commented Oct 19, 2018

okay then I'll squash

This moves tests for the heatshrink package from unittests to a regular
test, which should help to decrease binary size of unittests.
@smlng smlng force-pushed the pr/tests/heatshrink branch from fbabc73 to 24032ac Compare October 19, 2018 13:18
@bergzand bergzand merged commit 1157720 into RIOT-OS:master Oct 19, 2018
@smlng smlng deleted the pr/tests/heatshrink branch June 25, 2019 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: pkg Area: External package ports 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: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants