search

Rantanen / intercom

Object based cross-language FFI for Rust
MIT License
63 stars 7 forks source link

Separated tests for cpp interface wrappers into separate project #63

Closed Fluxie closed 6 years ago

Fluxie commented 6 years ago

Support for interface wrappers in different environments may lag behind so it is better that they are in a separate project.

codecov-io commented 6 years ago

Codecov Report

Merging #63 into master will increase coverage by 0.26%. The diff coverage is 79.9%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
+ Coverage   76.31%   76.57%   +0.26%     
==========================================
  Files          49       52       +3     
  Lines        3120     3215      +95     
==========================================
+ Hits         2381     2462      +81     
- Misses        739      753      +14
Impacted Files Coverage Δ
intercom-common/src/attributes/com_class.rs 100% <ø> (ø) :arrow_up:
intercom-attributes/src/lib.rs 56% <ø> (ø) :arrow_up:
intercom-cpp/src/posix/dlwrapper.h 83.33% <ø> (ø) :arrow_up:
intercom-cpp/src/guiddef.h 0% <0%> (ø)
intercom-attributes/tests/lib.rs 86.36% <100%> (ø) :arrow_up:
intercom-common/src/generators/cpp.rs 80.59% <100%> (+0.36%) :arrow_up:
intercom-cpp/src/posix/detail/memory.h 100% <100%> (ø)
intercom-cpp/src/activator.h 86.66% <100%> (+3.33%) :arrow_up:
intercom-cpp/src/detail/bstr_buffer.h 100% <100%> (ø)
intercom-cpp/src/detail/char_buffer.h 100% <100%> (ø)
... and 21 more

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 66632ae...49697a2. Read the comment docs.

Rantanen commented 6 years ago

Looks good! This also allows us to get a better feel of how the raw COM interface looks like on the low level even if the recommended way is to use the C++ wrappers.