Changement de la valeur par défaut du tenant CAS #17
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mamakh
changed the title
vitam-prg
requested review from
a team,
nabilm2i,
thermoweb,
benemart,
nicoename and
CouzinieAtCines
and removed request for
a team,
nabilm2i,
thermoweb,
benemart,
nicoename and
CouzinieAtCines
mamakh
force-pushed
the
change-cas-tenant-value
branch
from
22bb2f9 to
b2929c4
Compare
TDevillechabrolle
requested review from
TDevillechabrolle,
croftophile,
nicoename,
a team,
benemart and
Ro3034
and removed request for
a team
TDevillechabrolle
approved these changes
Apr 6, 2020
lgheribi
reviewed
Apr 7, 2020
integration-tests/src/test/java/fr/gouv/vitamui/utils/FactoryDto.java
Outdated
Show resolved
Hide resolved
mamakh
force-pushed
the
change-cas-tenant-value
branch
3 times, most recently
from
daf3219 to
62a3013
Compare
lgheribi
approved these changes
Apr 10, 2020
mamakh
force-pushed
the
change-cas-tenant-value
branch
2 times, most recently
from
f37ab6a to
2e1c8e7
Compare
mamakh
force-pushed
the
change-cas-tenant-value
branch
from
2e1c8e7 to
119badb
Compare
Closed
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Cette Pull Request change la valeur par défaut du tenant utilisé par CAS.
Cette demande a été faite lors du comité technique.
CAS utilise un tenant dans le système de sécurité de VITAM-UI pour différencier le tenant CAS du tenant système. Ce tenant n'a pas aujourd'hui de raison d'exister dans VITAM.
Sa valeur a été changée afin d'indiquer que le tenant est virtuel.
Les tests d'intégration ont été changés afin de pouvoir variabiliser & définir des valeurs utilisées pour le tenant lors de leurs exécutions.
Les certificats utilisés en dev pour les tests d'intégration ne correspondaient pas aux certificats des modules démarrés en dev. Ils ont été changés par conséquent.
Type de changement:
Code et/ou Configuration : Refactorisation
Documentation:
Tests:
Vérification des test d'intégration effectuée :
'mvn clean integration-test -Piam'
1437 Scenarios (1437 passed)
8393 Steps (8393 passed)
5m7.351s
[INFO] Tests run: 1437, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 310.138 s - in fr.gouv.vitamui.cucumber.back.runners.IamIntegrationTest
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 1437, Failures: 0, Errors: 0, Skipped: 0
Migration:
Checklist:
[x ] Mon code suit le style de code de ce projet.
[x ] J'ai commenté mon code, en particulier dans les classes et les
méthodes difficile à comprendre.