arduino-gmock-sample

arduino-gmock-sample copied to clipboard

Hello, Theses changes where needed to make it compile on my machine. Thanks

philfifi

Add ../TEST_DIR to CPPFLAGS includes

1

Users might have .h files in the ../test directory that need to be included. ``` -I../$(TEST_DIR) ```

jacobrosenthal

../Test Dir seems like bad form

1

Is there a reason you went away from googltest naming of USER_DIR to TEST_DIR which require $(TEST_DIR)../ Before all user paths? It seems like bad form and can easily be...

jacobrosenthal

Makefile CPPFLAGS should use root of include directory

1

Currently requires you to #include "arduinomock/Arduino.h" which breaks all usual include paths. Instead include like ``` -I$(ARDUINO_MOCK_DIR)/include/arduino-mock/ ```

jacobrosenthal

In testing I've found I need to bring in a few other libraries. I'm currently doing this like so ``` DEP_DIR = ../../ringbuffer # Flags passed to the preprocessor. #...

jacobrosenthal