Migrate to GTest

[ciband]

Move from AUnit to googletest

What kind of change does this PR introduce?

• Bugfix
• Feature
• Refactor
• Performance
• Tests
• Build
• Documentation
• Code style update
• Continuous Integration
• Other, please describe:

Does this PR introduce a breaking change?

• Yes
• No

Does this PR release a new version?

• Yes
• No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

• It's submitted to the develop branch, not the master branch
• All tests are passing
• New/updated tests are included

If adding a new feature, the PR's description includes:

• A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

[ghost]

Thanks for submitting this pull request! A maintainer will review this in the next few days and explicitly select labels so you know what's going on.

If no reviewer appears after a week, a reminder will be sent out.

[ghost]

Your pull request doesn't follow our contribution guidelines. Please review and correct it.

[sleepdefic1t]

@ciband
I’ve got a potential fix for the failing PIO in #121

utils/json.h

Remove the line that disables Arduino String.

#define ARDUINOJSON_ENABLE_ARDUINO_STRING 0 // disable 'String'

[sleepdefic1t]

Well, that fixed some of it 🤷‍♂️

GTest just won’t make it easy.

[sleepdefic1t]

Okay, @ciband
I think I've got it.

I ran pio run from the test dir, and saw that googletest wasn't actually being saved anywhere..

So, from test/platformio.ini, I changed lib_deps to explicitly use v1.8.1.

i.e. googletest >> googletest@1.8.1

Memory Usage -> http://bit.ly/pio-memory-usage
DATA:    [=         ]   7.3% (used 23916 bytes from 327680 bytes)
PROGRAM: [======    ]  59.2% (used 776301 bytes from 1310720 bytes)
esptool.py v2.6
================================================================= [SUCCESS] Took 31.54 seconds =================================================================

[ciband]

@sleepdefic1t thanks for the debugging help. I was going to get to it, you just beat me to it. Looks like that did it.

[codecov-io]

Codecov Report

Merging #131 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #131   +/-   ##
========================================
  Coverage    91.82%   91.82%           
========================================
  Files           33       33           
  Lines          856      856           
========================================
  Hits           786      786           
  Misses          70       70

---

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 4469607...3e15819. Read the comment docs.

[ghost]

A contributor has approved this PR. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait.

Thank you for your contribution!

[ghost]

Your pull request has been merged but was not assigned a bounty tier. A maintainer will assign a bounty tier to this pull request in the next few days.

[ghost]

Your pull request has been merged and marked as tier 4. It will earn you $20 USD.