Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

[#6292] Update CosmosDbStorage from Functional to Unit tests - #6303

Merged
Gabo Gilabert (gabog) merged 5 commits into
mainfrom
southworks/update/cosmos-tests
Jun 8, 2022
Merged

Gabo Gilabert (gabog) merged 5 commits into
mainfrom
southworks/update/cosmos-tests

Conversation

@sw-joelmut

Copy link
Copy Markdown
Collaborator

Addresses #6292
#minor

Description

This PR removes the existing Functional tests and as a replacement, it adds support for internal testing to the CosmosDbStorage class, and introduces all the necessary unit tests, increasing the code coverage up to 86%.

Specific Changes

  • Updates the CosmosDbStorage.DocumentStoreItem class accessibility from private to internal, so it can be accessed from the tests.
  • Removed all the existing tests in CosmosDbStorageTests class.
  • Adds new unit tests to the CosmosDbStorageTests class for each method, constructor, method validations, etc.

Testing

The following image shows the new unit tests and the code coverage.
image

@sw-joelmut Joel Mut (sw-joelmut) added the Automation: No parity PR does not need to be applied to other languages. label Apr 25, 2022
@sw-joelmut
Joel Mut (sw-joelmut) requested a review from a team as a code owner April 25, 2022 17:34
@coveralls

Coveralls (coveralls) commented Apr 25, 2022

Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 304882

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 78.955%

Totals Coverage Status
Change from base Build 304862: 0%
Covered Lines: 25376
Relevant Lines: 32140

💛 - Coveralls

@BruceHaley

Copy link
Copy Markdown
Contributor

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll

@BruceHaley

Copy link
Copy Markdown
Contributor

✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Automation: No parity PR does not need to be applied to other languages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants