Skip to content

Fix for SonarQube

Xiangliang Meng requested to merge DavidMeng-master-patch-21556 into master

Type of change

  • Bug Fix
  • Feature

Please provide link to gitlab issue or ADR(Architecture Decision Record)

Does this introduce a change in the core logic?

  • [YES]

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Does this introduce a breaking change?

  • [NO]

What is the current behavior?

SonarQube complains the test case should_authenticateRequest_when_isFromPubsubAndUserIdentityIsCorrect, for there is no assertion side

What is the new/expected behavior?

SonarQube will not complain about this test case.

Have you added/updated Unit Tests and Integration Tests?

Yes

Any other useful information

Merge request reports