Skip to content

RUMM-2335 Add Session Replay Public API tests#985

Merged
mariusc83 merged 1 commit into
feature/session-replay-vofrom
mconstantin/rumm-2335/add-session-replay-public-api-unit-tests
Jul 15, 2022
Merged

RUMM-2335 Add Session Replay Public API tests#985
mariusc83 merged 1 commit into
feature/session-replay-vofrom
mconstantin/rumm-2335/add-session-replay-public-api-unit-tests

Conversation

@mariusc83

Copy link
Copy Markdown
Member

What does this PR do?

In this PR we are adding the unit tests for the Public API of Session Replay feature.
Please not that the nightly tests coverage was dropped to 0.90 due to the newly introduced 2 methods. These will be covered later in another PR once Session Replay alpha will be ready.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

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)

@mariusc83 mariusc83 self-assigned this Jul 13, 2022
@mariusc83
mariusc83 force-pushed the mconstantin/rumm-2335/add-session-replay-public-api-unit-tests branch 3 times, most recently from 808b3eb to 149b7aa Compare July 13, 2022 15:33
@mariusc83
mariusc83 marked this pull request as ready for review July 13, 2022 15:34
@mariusc83
mariusc83 requested a review from a team as a code owner July 13, 2022 15:34
@mariusc83
mariusc83 force-pushed the mconstantin/rumm-2335/add-session-replay-public-api-unit-tests branch 2 times, most recently from 4047334 to 8ccd839 Compare July 13, 2022 16:18
@xgouchet xgouchet added the size-medium This PR is medium sized label Jul 13, 2022
@mariusc83
mariusc83 force-pushed the mconstantin/rumm-2335/add-session-replay-public-api-unit-tests branch from 8ccd839 to 0ddd1c1 Compare July 15, 2022 07:28
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (feature/session-replay-vo@68da559). Click here to learn what that means.
The diff coverage is n/a.

@@                     Coverage Diff                      @@
##             feature/session-replay-vo     #985   +/-   ##
============================================================
  Coverage                             ?   82.70%           
============================================================
  Files                                ?      299           
  Lines                                ?     9835           
  Branches                             ?     1622           
============================================================
  Hits                                 ?     8134           
  Misses                               ?     1216           
  Partials                             ?      485           

@mariusc83
mariusc83 merged commit 3e96e13 into feature/session-replay-vo Jul 15, 2022
@mariusc83
mariusc83 deleted the mconstantin/rumm-2335/add-session-replay-public-api-unit-tests branch July 15, 2022 11:11
@mariusc83
mariusc83 restored the mconstantin/rumm-2335/add-session-replay-public-api-unit-tests branch July 21, 2022 12:24
@mariusc83
mariusc83 deleted the mconstantin/rumm-2335/add-session-replay-public-api-unit-tests branch July 21, 2022 12:25
@xgouchet xgouchet added this to the internal milestone Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size-medium This PR is medium sized

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants