Skip to content

Conversation

@souissimai
Copy link
Contributor

PR Summary

SOUISSI Maissa (Externe) and others added 5 commits January 22, 2026 20:31
Signed-off-by: SOUISSI Maissa (Externe) <souissimai@gm0winl878.bureau.si.interne>
TU
Signed-off-by: SOUISSI Maissa (Externe) <souissimai@gm0winl878.bureau.si.interne>
@Mathieu-Deharbe Mathieu-Deharbe self-requested a review January 28, 2026 12:06
Copy link
Contributor

@Mathieu-Deharbe Mathieu-Deharbe left a comment

Choose a reason for hiding this comment

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

UserAdminServerStubs class has changed. Unit tests have to be updated.

userProfileInfos.getPccminParameterId(), userId, userProfileInfos.getName()), e);
}
}
// no profile, or no/bad shortcircuit parameters in profile => use default values
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// no profile, or no/bad shortcircuit parameters in profile => use default values
// no profile, or no/bad pcc min parameters in profile => use default values

SOUISSI Maissa (Externe) added 2 commits January 28, 2026 16:36
Signed-off-by: SOUISSI Maissa (Externe) <souissimai@gm0winl878.bureau.si.interne>
Signed-off-by: SOUISSI Maissa (Externe) <souissimai@gm0winl878.bureau.si.interne>
souissimai and others added 4 commits January 28, 2026 23:41
Signed-off-by: SOUISSI Maissa (Externe) <souissimai@gm0winl878.bureau.si.interne>
Signed-off-by: SOUISSI Maissa (Externe) <souissimai@gm0winl878.bureau.si.interne>
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
54.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@souissimai souissimai closed this Jan 29, 2026
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.

2 participants