Adding several serial functions and unit tests for all print and println functions

[frznlogic]

Added mocking for:

• Serial::read,
• Serial::available,
• Serial::write
• A new helper class stringCapture which simplifies getting all data from Serial functions, or other functions for that matter. This helps when testing code with arbitrary number of calls to Serial functions, but where the "sent" string is known (ie, you know how the protocol should look, but don't care how it is created).

Unit tests are added for almost all functionality, and also updated to cover all print/println functions.

[ikeyasu]

Why do we need stringCapture here?
I think ... it is not mock for Arduino.

It is a helper class for our UnitTest, isn't it? If so, please move this one to '/test' directory.

[frznlogic]

Hi Ikeasu, I wrote a longer response below. The stringCapture stuff is supposed to be used by external testers. The naming is a bit bad, any thoughts on a new name?

[ikeyasu]

Good job!
My comment is only one for a position where we should have stringCapture.
It seems a helper function for arduino-mock's unittest, so it should be locate in /test?

[frznlogic]

Hi Ikeyasu!

Thanks! I really appreciate the project fyi, I've been looking for a good mocking system of the arduino platform for a long time as it is fairly hard to test on the platform itself :).

Anyways, back to topic, the stringCapture class is supposed to be used by external users. I'm currently using it in a private project to verify functions which calls Serial::write arbitrary number of times and where I don't want to rely on the internal implementation of the function, but the outcoming stream format is really important. In this testcase, I encode it using the arduino functions, then I decode it using the PC side decoder within the unit test/mock framework.

[ikeyasu]

Thank you for your reply.
I would like to improve Arduino testing framework. (You know the current Arduino testing is very poor..)

I'm currently using it in a private project to verify functions which calls Serial::write arbitrary number of times and where I don't want to rely on the internal implementation of the function

You mean you are using it as a library for your Arduino project?
If so, please consider to separate the class to another header file.
I understood that It is necessary but I would like to keep a structure of 'serial.h' as same as Arduino's library.

[frznlogic]

Fair enough. I've moved the functions into a file called serialHelper.cc/h and serialHelper_unittest.cc. Makefiles have been updated. Is this better?

[ikeyasu]

LGTM. It was merged manually.
Thanks a lot.

[frznlogic]

Thank you :)