[PM-11405] Account Management: Prevent a verified user from changing their email address #3993
scan.yml
on: pull_request_target
Check PR run
/
Check
4s
Annotations
2 errors and 10 warnings
Bit.Api.Test.Auth.Controllers.AccountsControllerTests ► PostEmail_WithAccountDeprovisioningEnabled_WhenUserIsManagedByAnOrganization_ShouldThrowBadRequestException:
test/Api.Test/Auth/Controllers/AccountsControllerTests.cs#L263
Failed test found in:
test/Api.Test/TestResults/oss-test-results.trx
Error:
Assert.Equal() Failure: Strings differ
↓ (pos 0)
Expected: "Accounts managed by an organization canno"···
Actual: "Cannot change emails for accounts owned b"···
↑ (pos 0)
|
Bit.Api.Test.Auth.Controllers.AccountsControllerTests ► PostEmailToken_WithAccountDeprovisioningEnabled_WhenUserIsManagedByAnOrganization_ShouldThrowBadRequestException:
test/Api.Test/Auth/Controllers/AccountsControllerTests.cs#L197
Failed test found in:
test/Api.Test/TestResults/oss-test-results.trx
Error:
Assert.Equal() Failure: Strings differ
↓ (pos 0)
Expected: "Accounts managed by an organization canno"···
Actual: "Cannot change emails for accounts owned b"···
↑ (pos 0)
|
Quality scan:
src/Core/Models/Mail/Provider/ProviderInitiateDeleteModel.cs#L11
'ProviderInitiateDeleteModel.WebVaultUrl' hides inherited member 'BaseMailModel.WebVaultUrl'. Use the new keyword if hiding was intended.
|
Quality scan:
src/Core/Models/Mail/Provider/ProviderInitiateDeleteModel.cs#L14
'ProviderInitiateDeleteModel.SiteName' hides inherited member 'BaseMailModel.SiteName'. Use the new keyword if hiding was intended.
|
Quality scan:
src/Core/AdminConsole/OrganizationFeatures/OrganizationUsers/UpdateOrganizationUserCommand.cs#L121
Possible null reference argument for parameter 'collections' in 'Task IOrganizationUserRepository.ReplaceAsync(OrganizationUser obj, IEnumerable<CollectionAccessSelection> collections)'.
|
Quality scan:
src/Core/Billing/Services/Implementations/PaymentHistoryService.cs#L15
Parameter 'logger' is unread.
|
Quality scan:
src/Core/Services/Implementations/UserService.cs#L170
'_currentContext' is null on at least one execution path. (https://rules.sonarsource.com/csharp/RSPEC-2259)
|
Quality scan:
src/Core/Services/Implementations/UserService.cs#L181
'_currentContext' is null on at least one execution path. (https://rules.sonarsource.com/csharp/RSPEC-2259)
|
Quality scan:
src/Core/Services/Implementations/UserService.cs#L959
'paymentService' is null on at least one execution path. (https://rules.sonarsource.com/csharp/RSPEC-2259)
|
Quality scan:
test/Core.Test/AdminConsole/Services/OrganizationServiceTests.cs#L100
Cannot convert null literal to non-nullable reference type.
|
Quality scan:
test/Core.Test/AdminConsole/Services/OrganizationServiceTests.cs#L104
Cannot convert null literal to non-nullable reference type.
|
Quality scan:
test/Core.Test/AdminConsole/Services/OrganizationServiceTests.cs#L175
Cannot convert null literal to non-nullable reference type.
|