Skip to content

RUM-1517 Create SDKCore stub classes#1734

Merged
xgouchet merged 1 commit into
developfrom
xgouchet/RUM-1517/stub_sdk_core
Nov 21, 2023
Merged

RUM-1517 Create SDKCore stub classes#1734
xgouchet merged 1 commit into
developfrom
xgouchet/RUM-1517/stub_sdk_core

Conversation

@xgouchet

@xgouchet xgouchet commented Nov 20, 2023

Copy link
Copy Markdown
Contributor

What does this PR do?

Create a few Stubbed classes to use the SDKCore in integration tests from the Feature modules.

Note

So far only a few functions in the StubSDKCore class are actually implemented, others will be added when needed

@xgouchet
xgouchet requested review from a team as code owners November 20, 2023 13:07
@xgouchet
xgouchet force-pushed the xgouchet/RUM-1517/stub_sdk_core branch from e34a781 to 714bbec Compare November 20, 2023 13:54
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #1734 (714bbec) into develop (8e099c0) will increase coverage by 0.08%.
Report is 2 commits behind head on develop.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1734      +/-   ##
===========================================
+ Coverage    83.41%   83.48%   +0.08%     
===========================================
  Files          465      465              
  Lines        16182    16179       -3     
  Branches      2412     2411       -1     
===========================================
+ Hits         13497    13507      +10     
+ Misses        2026     2022       -4     
+ Partials       659      650       -9     

see 15 files with indirect coverage changes

@xgouchet
xgouchet merged commit 764129c into develop Nov 21, 2023
@xgouchet
xgouchet deleted the xgouchet/RUM-1517/stub_sdk_core branch November 21, 2023 09:17
@xgouchet xgouchet added this to the 2.3.0 milestone Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants