Skip to content

Conversation

TRybina132
Copy link
Member

Add integration tests for Storage http client class

TRybina132 and others added 26 commits October 16, 2023 20:05
…ntegration_tests_project

# Conflicts:
#	ManagedCode.Storage.IntegrationTests/ManagedCode.Storage.IntegrationTests.csproj
…ntegration_tests_project

# Conflicts:
#	ManagedCode.Storage.Client/StorageClient.cs
# Conflicts:
#	ManagedCode.Storage.Client/IStorageClient.cs
#	ManagedCode.Storage.Client/StorageClient.cs
#	ManagedCode.Storage.IntegrationTests/Constants/ApiEndpoints.cs
#	ManagedCode.Storage.IntegrationTests/Helpers/Crc32Helper.cs
#	ManagedCode.Storage.IntegrationTests/TestApp/Controllers/AzureTestController.cs
#	ManagedCode.Storage.IntegrationTests/TestApp/Controllers/Base/BaseFileController.cs
#	ManagedCode.Storage.IntegrationTests/TestApp/Controllers/Base/BaseTestController.cs
#	ManagedCode.Storage.IntegrationTests/Tests/BaseDownloadControllerTests.cs
#	ManagedCode.Storage.IntegrationTests/Tests/BaseUploadControllerTests.cs
@TRybina132 TRybina132 changed the base branch from main to develop October 18, 2023 15:48
@github-actions
Copy link

NDepend Analysis
Quality Gates Failed Failed because at least one Quality Gate failed

Name Status Trend Baseline Value Baseline Status Currrent Value
Percentage Coverage PassPass NoChange N/A % PassPass N/A %
Percentage Coverage on New Code PassPass NoChange N/A % PassPass N/A %
Percentage Coverage on Refactored Code PassPass NoChange N/A % PassPass N/A %
Blocker Issues PassPass NoChange 0 issues PassPass 0 issues
Critical Issues PassPass NoChange 0 issues PassPass 0 issues
New Blocker / Critical / High Issues PassPass NoTrend 0 issues
Critical Rules Violated FailedFailed NoChange 1 rules FailedFailed 1 rules
Percentage Debt PassPass NoChange 4.64 % PassPass 4.64 %
New Debt since Baseline PassPass NoTrend 0 man-days
Debt Rating per Namespace FailedFailed NoChange 1 namespaces FailedFailed 1 namespaces
New Annual Interest since Baseline PassPass NoTrend 0 man-days

Issues:

Severity Count Added Issues Fixed Issues
All Issues 89 +0 -0
Blocker Blocker 0 +0 -0
Critical Critical 0 +0 -0
High High 19 +0 -0
Medium Medium 69 +0 -0
Low Low 1 +0 -0

To have more details about the analysis you can Download the detailled report here ("https://github.com/managedcode/Storage/actions/runs/6563357842#artifacts")

@TRybina132 TRybina132 requested a review from KSemenenko October 18, 2023 17:38
@KSemenenko KSemenenko merged commit cb4f311 into develop Oct 18, 2023
@KSemenenko KSemenenko deleted the integration_tests_project branch October 18, 2023 21:26
KSemenenko added a commit that referenced this pull request Oct 20, 2023
* aws

* Added Managed Identity for Azure Storage

* Release alpha version

* base controller

* Add methods from StorageFromFileExt and StorageExt (#46)

Co-authored-by: Hurko Volodymyr <hurko@managed-code.com>

* prototype

* test containsers azure + aws

* checks

* actions

* test containsers

* test

* rename

* #54

* refactoring

* tests

* tests

* tests

* fix all tests

* naming

* fix test container for google

* ndepend

* Upload chunks (#69)

* add integration tests project

* add client

* add test controller and controller tests

* fix

* add IStorageClient

* fix storage client

* accept file and content names in client method

* add content name property

* return status code when request is failed

* add test for file upload from file info

* move tests

* add tests for upload methods with azure

* fix tests

* return downloaded file as stream

* return localFile from download method

* return result on download

* add base test classes for controller tests

* add methods upload in chunk

---------

Co-authored-by: TRybina132 <rybina@managed-code.com>

* add stream tests classes

* Add integration tests (#70)

* add integration tests project

* add client

* add test controller and controller tests

* fix

* add IStorageClient

* fix storage client

* accept file and content names in client method

* add content name property

* return status code when request is failed

* add test for file upload from file info

* move tests

* add tests for upload methods with azure

* fix tests

* return downloaded file as stream

* return localFile from download method

* return result on download

* add base test classes for controller tests

* use another controller for base controller

* use another method for crc calculation

* remove duplicated controller

* remove local folder path and replace it with path from method

---------

Co-authored-by: TRybina132 <rybina@managed-code.com>
Co-authored-by: Olexandr Ivanitskyi <6saaanchik6@gmail.com>

* fix azure blob stream lenght

* fix tests

* add write test for azure blob

* add seek test

* Updated code coverage in pipeline (#74)

* Temporarily removed some code from dotnet.yml pipeline

* codecoverage

* Modified test csproj

* Rollbacked changes to csproj

* Added stopIfQGFailed: false

* Updated dotnet.yml

* Updated dotnet.yml

* Modified pipeline

* Fixed type and temporarily removed some code

* Added ndepend

* Brought back missing code

* Added --no-build flag

---------

Co-authored-by: Kostiantyn Bondarenko <bondarenko@managed-code.com>
Co-authored-by: ksemenenko <KSemenenko@users.noreply.github.com>

* add properties variable

---------

Co-authored-by: dtymakhov <dtymakhov@gmail.com>
Co-authored-by: Alexandr Ivanitskyi <6saaanchik6@gmail.com>
Co-authored-by: Hurko Volodymyr <92858780+Hurko-Volodymyr@users.noreply.github.com>
Co-authored-by: Hurko Volodymyr <hurko@managed-code.com>
Co-authored-by: Eduard <eduard@managed-code.com>
Co-authored-by: Alexandr Ivanitskyi <91984393+Sanyyazzz@users.noreply.github.com>
Co-authored-by: TRybina132 <rybina@managed-code.com>
Co-authored-by: TRybina132 <98330987+TRybina132@users.noreply.github.com>
Co-authored-by: Kostiantyn Bondarenko <92333402+techcat18@users.noreply.github.com>
Co-authored-by: Kostiantyn Bondarenko <bondarenko@managed-code.com>
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.

3 participants