Skip to content

Conversation

DNKpp
Copy link
Owner

@DNKpp DNKpp commented Sep 9, 2025

This PR

  • deprecates the existing (MIMICPP_)MOCK_(OVERLOADED_)METHOD macros
  • adds a whole set of new facade-macros
  • Facade.hpp and macros/Facade.hpp headers
  • removes InterfaceMock.hpp

@DNKpp DNKpp added the enhancement New feature or request label Sep 9, 2025
Copy link

codacy-production bot commented Sep 9, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
+0.02% (target: -1.00%) 100.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (798c20a) 2489 2442 98.11%
Head commit (69be5a8) 2513 (+24) 2466 (+24) 98.13% (+0.02%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#134) 29 29 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Copy link

codecov bot commented Sep 9, 2025

Codecov Report

❌ Patch coverage is 96.55172% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 94.30%. Comparing base (798c20a) to head (69be5a8).

Files with missing lines Patch % Lines
include/mimic++/Facade.hpp 95.23% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           development     #134      +/-   ##
===============================================
+ Coverage        94.29%   94.30%   +0.01%     
===============================================
  Files               55       56       +1     
  Lines             2489     2513      +24     
  Branches           969      971       +2     
===============================================
+ Hits              2347     2370      +23     
  Misses              51       51              
- Partials            91       92       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@coveralls
Copy link

coveralls commented Sep 9, 2025

Pull Request Test Coverage Report for Build 18447902210

Details

  • 29 of 29 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 98.13%

Totals Coverage Status
Change from base Build 18245161934: 0.02%
Covered Lines: 2466
Relevant Lines: 2513

💛 - Coveralls

@DNKpp DNKpp force-pushed the dev/MockWithThis branch from 57aaca2 to d9d0f9e Compare October 3, 2025 19:57
@DNKpp DNKpp changed the title Feature: MOCK_METHOD_WITH_THIS Feature: New set of facade-macros Oct 12, 2025
DNKpp added 23 commits October 12, 2025 20:31
…OVERLOADED_MOCK into detail::make_interface_mock_settings
DNKpp added 29 commits October 12, 2025 20:31
…E_OVERLOADED_MEMBER_MOCK and MIMICPP_MAKE_MEMBER_FACADE to MIMICPP_MAKE_MEMBER_MOCK
…MAKE_OVERLOADED_MEMBER_MOCK_WITH_THIS and MIMICPP_MOCK_METHOD_WITH_THIS to MIMICPP_MAKE_MEMBER_MOCK_WITH_THIS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants