Skip to content

Adding CI unit testing for ArduinoCore-API #122

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.

Sign up for GitHub

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

Jump to bottom
Merged
aentinger merged 102 commits into from
Nov 6, 2020
Merged

Adding CI unit testing for ArduinoCore-API #122

aentinger merged 102 commits into from
Nov 6, 2020

Conversation

@aentinger
Copy link
Contributor

@aentinger aentinger commented Sep 25, 2020

No description provided.

@aentinger aentinger self-assigned this Sep 25, 2020
@arduino arduino deleted a comment from codecov-commenter Oct 5, 2020
@codecov-io
Copy link

codecov-io commented Oct 7, 2020 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (master@76e931e). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #122   +/-   ##
=========================================
  Coverage          ?   96.40%           
=========================================
  Files             ?       14           
  Lines             ?      835           
  Branches          ?        0           
=========================================
  Hits              ?      805           
  Misses            ?       30           
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 76e931e...c4a350a. Read the comment docs.

@arduino arduino deleted a comment from codecov-commenter Oct 7, 2020
aentinger added 24 commits October 7, 2020 17:40
@aentinger
Initial minimal commit to compile a host-run test binary
95a146f
@aentinger
Adding test code for 'isHexadecimalDigit'
1fcfa7f
@aentinger
Adding test code for 'isUpperCase'
2efc193
@aentinger
Adding test code for 'isSpace'
e35d969
@aentinger
Adding test code for 'max'
7639c5c
@aentinger
Adding test code for 'min'
3541f60
@aentinger
Adding test code for 'makeWord' and 'word'
5841c06
@aentinger
Adding test code for 'isPunct'
da4b985
@aentinger
Adding test code for 'isLowerCase'
1c0027c
@aentinger
Adding test code for 'isDigit'
09ce806
@aentinger
Adding test code for 'isControl'
c616c58
@aentinger
Adding test code for 'isWhitespace'
09d4c4c
@aentinger
Adding test code for 'RingbufferN::isFull'
4749b49
@aentinger
Fix: 'memset' requires including of 'string.h'
0c2502b
@aentinger
Adding test code for 'RingbufferN::available'
ed6bea5
@aentinger
Separating tests for RingbufferN::available from RingBufferN::availab…
33da3fd 
…leFromStore
@aentinger
Adding test code for 'RingbufferN::clear'
f4d76bd
@aentinger
Augment test for 'RingBufferN::isFull'
208f455
@aentinger
Fixing RingBufferN
b8ae7a0
@aentinger
Augment test for 'RingBufferN::store_char'
7219305
@aentinger
Adding test code for 'Ringbuffer::read_char'
b7edc11
@aentinger
Fixing bugs when attempting to read from ringbuffer
9a14dd6
@aentinger
Adding test code for 'Ringbuffer::peek'
2efb5d4
@aentinger
Cross-compiling 'String' which requires to provide implementions for …
0520927 
…dtostrf,itoa,utoa,etc.
@aentinger
Adding StreamMock class to test functions of Stream base class as wel…
b6a55a8 
…l as test code for Stream::getTimeout()
@aentinger
Disable 'int atexit(void (*func)()) __attribute__((weak));' for host …
09541b3 
…test build since ARM-GCC LIBC has a different definition of the function
@aentinger
Compiling 'Stream.cpp' for which it's necessary to provide a fake mil…
4f5332d 
…lis function
@aentinger
Renaming module 'millis' to 'MillisFake' to better reflect the actual…
716c324 
… nature of the module
@aentinger
Adding test code for Stream::readString
ff7e4a6
@aentinger
Fixing a couple of issues within StreamMock so that it's now fully fu…
abee6d1 
…nctional
@aentinger
Augmenting the MillisFake module with an automatic mode where real sy…
fbe1cd0 
…stem time is in play
@aentinger
Adding test code for Stream::readStringUntil(...)
7182f67
@aentinger
Adding test code for Stream::find(...)
08a44c9
@aentinger
Adding test code for Stream::findUntil(...)
a7e0931
@aentinger
Addding test code for Stream::parseInt(...)
52ddc44
@aentinger
Adding test code for Stream::parseFloat(...)
5c7276a
@aentinger
Reworking parseInt test code
da7100c
@aentinger
Adding test code for Stream::readBytes(...)
2c1feec
per1234 and others added 2 commits November 6, 2020 00:41
@aentinger aentinger marked this pull request as ready for review November 6, 2020 09:53
@aentinger aentinger merged commit 105276f into master Nov 6, 2020
@aentinger aentinger deleted the ci-unit-test branch November 6, 2020 09:54
@codecov-commenter
Copy link

codecov-commenter commented Oct 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (master@76e931e). Learn more about missing BASE report.

Additional details and impacted files 
@@            Coverage Diff            @@
##             master     #122   +/-   ##
=========================================
  Coverage          ?   96.40%           
=========================================
  Files             ?       14           
  Lines             ?      835           
  Branches          ?        0           
=========================================
  Hits              ?      805           
  Misses            ?       30           
  Partials          ?        0

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@aentinger aentinger

Labels

enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@aentinger @codecov-io @codecov-commenter @per1234 @giulcioffi