Closed Rantanen closed 5 years ago
Merging #83 into master will increase coverage by
2.06%
. The diff coverage is93.5%
.
@@ Coverage Diff @@
## master #83 +/- ##
==========================================
+ Coverage 74.05% 76.11% +2.06%
==========================================
Files 55 55
Lines 4012 4057 +45
==========================================
+ Hits 2971 3088 +117
+ Misses 1041 969 -72
Impacted Files | Coverage Δ | |
---|---|---|
intercom-common/src/methodinfo.rs | 90.77% <100%> (+5.41%) |
:arrow_up: |
intercom/src/comitf.rs | 76.47% <100%> (+18.77%) |
:arrow_up: |
intercom/src/error.rs | 25.42% <33.33%> (+11.74%) |
:arrow_up: |
intercom-common/src/tyhandlers.rs | 81.59% <80%> (+1.22%) |
:arrow_up: |
intercom/src/strings.rs | 81.81% <95%> (+13.74%) |
:arrow_up: |
intercom-common/src/attributes/com_interface.rs | 96.8% <97.36%> (-0.26%) |
:arrow_down: |
intercom-common/src/model/cominterface.rs | 93.88% <0%> (+0.55%) |
:arrow_up: |
... and 3 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 8ae1326...2bfe69b. Read the comment docs.
The type system stuff is a bit incomplete currently. Opened #84 to fix that before merging this. The tests here only take care of Automation type system. I still want to add the Raw type system tests.
The interface implementation on
ComItf
now checks for the presence of either Automation or Raw pointer and performs the delegated call using the first thing that is found available.