verify that conflicts with Space names are being checked for compliantUsername #798
Conversation
|
|
||
| // then | ||
| compliantUsername := userSignup.Status.CompliantUsername | ||
| require.Equal(s.T(), fmt.Sprintf("%s-2", conflictingSpace.Name), compliantUsername) |
There was a problem hiding this comment.
I'm a bit lost: why is the usersignup's CompliantUserName already set to something like conflicting-2 at the beginning of the test?
There was a problem hiding this comment.
look at the // given part - it creates a Space with "conflicting" name there.
|
/retest |
|
Kudos, SonarCloud Quality Gate passed!
|
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alexeykazakov, MatousJobanek The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
paired with codeready-toolchain/host-operator#867