Skip to content
This repository was archived by the owner on Aug 29, 2024. It is now read-only.

Int test updates#482

Merged
kushagraThapar merged 4 commits into
microsoft:masterfrom
blackbaud:int-test-updates
Feb 7, 2020
Merged

Int test updates#482
kushagraThapar merged 4 commits into
microsoft:masterfrom
blackbaud:int-test-updates

Conversation

@Blackbaud-MikeLueders

Copy link
Copy Markdown
Contributor

The TestRepositoryConfig configuration allows for specifying the cosmos database to use for integration tests via the cosmosdb.database property name. This minimizes the number of collections created which is very useful if you don't have unlimited Azure credits. Instead of injecting cosmosdb.uri and cosmosdb.key and manually creating CosmosDbConfig, this PR updates those files to apply TestRepositoryConfig and autowire the CosmosDbConfig.

In addition, RoleRepositoryCollectionIT is updated to add assertions.

@Incarnation-p-lee

Copy link
Copy Markdown
Contributor

Codacy Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 4
           

See the complete overview on Codacy

@kushagraThapar

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@kushagraThapar kushagraThapar left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for this change, it looks good to me.

@kushagraThapar kushagraThapar merged commit b493fbb into microsoft:master Feb 7, 2020
@Blackbaud-MikeLueders Blackbaud-MikeLueders deleted the int-test-updates branch February 8, 2020 03:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants