search

melexis / mlx90632-library

MLX90632 library for the Melexis 90632 Infra Red temperature sensor.
Apache License 2.0
42 stars 15 forks source link

tearDown setUp functions #20

Closed sison54 closed 4 years ago

sison54 commented 4 years ago

Are the mock_mlx90632_depends.h and mock_mlx90632_depends.c files available? I dont see them in the repository.

Thanks

Letme commented 4 years ago

They are generated by CMock at build time.

Letme commented 4 years ago

If you install rake and run make utest it should use Ceedling (CMock and Unity) to generate all the needed things for unit testing (and run), which will produce all the mock files.

sison54 commented 4 years ago

Ah. Got it. Thanks!

sison54 commented 4 years ago

I get linker errors that the setUp and tearDown functions are missing from the test files. It works fine once I add them (or rename SetUp to setUp).

Letme commented 4 years ago

Did you install Ceedling from gem, or you run the one which is referenced here with submodules?

But him, I do see inconsistency among Test* files, so you can open a pull request with aligning all that to small setUp, if that is the only error.

sison54 commented 4 years ago

I am using the one referenced in the submodule. I will create a pull request to fix this.

sison54 commented 4 years ago

Added pull request #21

Letme commented 4 years ago

Merged. Thanks for reporting and fixing.