Add macOS CPU unittest

[mthrok]

No description provided.

[codecov]

Codecov Report

Merging #777 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #777   +/-   ##
=======================================
  Coverage   89.53%   89.53%           
=======================================
  Files          32       32           
  Lines        2617     2617           
=======================================
  Hits         2343     2343           
  Misses        274      274           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c9142fd...2f3c1f1. Read the comment docs.

[peterjc123]

@mthrok So actually you are sharing the common script for unit tests between MacOS and Linux, right?

[mthrok]

@mthrok So actually you are sharing the common script for unit tests between MacOS and Linux, right?

Yeah, I thought of splitting them since I asked you to split Windows from Linux, but I felt that it's more beneficial to keep *nix systems same. (this is a vague line and subjective, splitting macOS and Linux feels like later when we need to apply changes to Linux, we might forget to apply the same changes to macOS) However, my goal is to add address sanitizer on macOS unit test, so depending on how it is done, I might split Linux and macOS later.

[peterjc123]

That's okay for me. But I'm just a bit worried that the same thing could happen to Windows, too.

[mthrok]

That's okay for me. But I'm just a bit worried that the same thing could happen to Windows, too.

Yes, it's already happening. In #664 I forgot to remove style check from windows job, so I am doing it in #778 haha.

[mthrok]

Thanks!