meekrosoft / fff

A testing micro framework for creating function test doubles
Other
749 stars 163 forks source link

Support custom function signatures #113

Closed yperess closed 1 year ago

yperess commented 1 year ago

Add support for other function signature types (like std::function and pw::Function)

AppVeyorBot commented 1 year ago

:x: Build fff 1.0.132 failed (commit https://github.com/meekrosoft/fff/commit/1ec2947b8b by @yperess)

AppVeyorBot commented 1 year ago

:x: Build fff 1.0.133 failed (commit https://github.com/meekrosoft/fff/commit/f850bd0f58 by @yperess)

AppVeyorBot commented 1 year ago

:x: Build fff 1.0.134 failed (commit https://github.com/meekrosoft/fff/commit/fa09bebc86 by @yperess)

AppVeyorBot commented 1 year ago

:x: Build fff 1.0.135 failed (commit https://github.com/meekrosoft/fff/commit/393b9304ff by @yperess)

yperess commented 1 year ago

I added an update to the documentation for the new feature

meekrosoft commented 1 year ago

This test seems to fail on my laptop:

image
yperess commented 1 year ago

@meekrosoft all the tests are passing now, thoughts?

yperess commented 1 year ago

@meekrosoft I've updated the PR to use the same include path in Windows as in MacOS and Linux (this makes things more consistent). I also included a link to the bug that's causing this problem.

yperess commented 1 year ago

@meekrosoft my DMs don't seem to be getting through for some reason. Zephyr's cutoff for features is coming up in 7 days (https://github.com/zephyrproject-rtos/zephyr/wiki/Release-Management#release-milestone-dates). I would like to get this feature merged and pulled into Zephyr by then otherwise it'll be locked until the end of the month).