Skip to content

RUMM-2336: Bring tests back#977

Merged
0xnm merged 1 commit into
feature/sdkv2from
nogorodnikov/rumm-2236/bring-unit-tests-back
Jul 11, 2022
Merged

RUMM-2336: Bring tests back#977
0xnm merged 1 commit into
feature/sdkv2from
nogorodnikov/rumm-2236/bring-unit-tests-back

Conversation

@0xnm

@0xnm 0xnm commented Jul 8, 2022

Copy link
Copy Markdown
Member

What does this PR do?

This change fixes the tests which were broken before. This will simplify the future development on the sdkv2 branch.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm
0xnm requested a review from a team as a code owner July 8, 2022 13:38
@0xnm
0xnm force-pushed the nogorodnikov/rumm-2236/bring-unit-tests-back branch from 116c0d5 to 1188e20 Compare July 8, 2022 14:32

companion object {
val TEST_TRACER = AndroidTracer.Builder().build()
// TODO remove service call once Tracer gets default value

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't you need a ticket here ?

@mariusc83 mariusc83 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@0xnm
0xnm force-pushed the nogorodnikov/rumm-2236/bring-unit-tests-back branch 2 times, most recently from 94981fd to 90f2da5 Compare July 8, 2022 17:35
@0xnm
0xnm force-pushed the nogorodnikov/rumm-2236/bring-unit-tests-back branch from 90f2da5 to d5ae1b9 Compare July 8, 2022 17:37
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (nogorodnikov/rumm-2204/data-is-written-in-sdk-specific-location@2a67f75). Click here to learn what that means.
The diff coverage is n/a.

@@                                        Coverage Diff                                         @@
##             nogorodnikov/rumm-2204/data-is-written-in-sdk-specific-location     #977   +/-   ##
==================================================================================================
  Coverage                                                                   ?   82.42%           
==================================================================================================
  Files                                                                      ?      291           
  Lines                                                                      ?     9482           
  Branches                                                                   ?     1573           
==================================================================================================
  Hits                                                                       ?     7815           
  Misses                                                                     ?     1199           
  Partials                                                                   ?      468           

@xgouchet xgouchet added the size-large This PR is large sized label Jul 8, 2022
@0xnm 0xnm changed the title RUMM-2236: Bring tests back RUMM-2336: Bring tests back Jul 11, 2022
Base automatically changed from nogorodnikov/rumm-2204/data-is-written-in-sdk-specific-location to feature/sdkv2 July 11, 2022 15:35
@0xnm
0xnm merged commit 268239e into feature/sdkv2 Jul 11, 2022
@0xnm
0xnm deleted the nogorodnikov/rumm-2236/bring-unit-tests-back branch July 11, 2022 15:35
@xgouchet xgouchet added this to the 1.16.0 milestone Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size-large This PR is large sized

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants