eranpeer / FakeIt

C++ mocking made easy. A simple yet very expressive, headers only library for c++ mocking.

MIT License

1.22k stars 170 forks source link

Imrpove CI #269

Open FranckRJ opened 2 years ago

FranckRJ commented 2 years ago

[x] Test more versions of current compilers
[x] Test with more standards of C++
[x] Test with sanitizers enabled
[x] Test on Apple Clang
[ ] Test on MinGW
[x] Move the MSVC tests in GitHub actions
[x] Run MSVC tests, currently it's only built but not run
[x] Run MSVC tests in 64 bits
[ ] Run RTTI-dependent tests in MSVC 64 bits.
[x] Run C++14 specific tests for MSVC

Take inspiration from range-v3 CI configuration, which looks great : https://github.com/ericniebler/range-v3/blob/master/.github/workflows/range-v3-ci.yml

malcolmdavey commented 1 year ago

That would be good. Just tried to get it to compile with VS 2019/v142 with the existing appveyor, but seems to need more steps to enable toolsets/compiler versions. Also would need to get the tests running for MSVC/VS

Edit - looks like I do have it working for VS 2015(v140) with C++14, and VS 2019 (v142) with C++20 and __cplusplus correct. But it would be better with github actions etc.

FranckRJ commented 1 year ago

313 improved things but there's still some things to fix to properly be able to test MSVC.

I'll probably remove the line "add more compilers" because I don't really want to invest a lot of time into supporting them.

FranckRJ commented 1 year ago

317 fixed C++14 tests on MSVC. The only known issue remaining is with RTTI stuff on MSVC x64.

FranckRJ commented 1 year ago

With #318 now there's an action to run tests with sanitizers enabled.

ClausKlein commented 1 year ago

With https://github.com/eranpeer/FakeIt/pull/319 CI test with AppleClang on OSX may be added.

FranckRJ commented 1 month ago

With #335 now tests are run for MSVC 64 bits (expect tests that depend on RTTI as it isn't properly supported in fakeit for this configuration yet).