-
Notifications
You must be signed in to change notification settings - Fork 10
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RI-136: archive une offre #167
Conversation
mathusha-sdv
commented
Nov 9, 2021
•
edited
Loading
edited
- tests
- Route pour archiver les offres
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
...licaction/src/main/java/com/dynonuggets/refonteimplicaction/repository/StatusRepository.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai proposé une simplification en passant par le champ archive
du modèle Job.
Il faudrait que tu délimites tes tests comme je l'ai fait en mode
// given
...
// when
...
// then
pour plus de lisibilité, sinon c'est vite le bordel 😆
Mais gg quand même et merci pour les test 👍
...ction/src/main/java/com/dynonuggets/refonteimplicaction/controller/JobPostingController.java
Outdated
Show resolved
Hide resolved
...licaction/src/main/java/com/dynonuggets/refonteimplicaction/repository/StatusRepository.java
Outdated
Show resolved
Hide resolved
...mplicaction/src/main/java/com/dynonuggets/refonteimplicaction/service/JobPostingService.java
Outdated
Show resolved
Hide resolved
...mplicaction/src/main/java/com/dynonuggets/refonteimplicaction/service/JobPostingService.java
Outdated
Show resolved
Hide resolved
...mplicaction/src/main/java/com/dynonuggets/refonteimplicaction/service/JobPostingService.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
...licaction/src/main/java/com/dynonuggets/refonteimplicaction/repository/StatusRepository.java
Outdated
Show resolved
Hide resolved
76ee475
to
b59b3ae
Compare
...ction/src/main/java/com/dynonuggets/refonteimplicaction/controller/JobPostingController.java
Show resolved
Hide resolved
backend-implicaction/src/main/java/com/dynonuggets/refonteimplicaction/dto/JobPostingDto.java
Outdated
Show resolved
Hide resolved
...mplicaction/src/main/java/com/dynonuggets/refonteimplicaction/service/JobPostingService.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
.../src/test/java/com/dynonuggets/refonteimplicaction/controller/JobsPostingControllerTest.java
Outdated
Show resolved
Hide resolved
backend-implicaction/src/main/java/com/dynonuggets/refonteimplicaction/dto/JobPostingDto.java
Outdated
Show resolved
Hide resolved
backend-implicaction/src/main/java/com/dynonuggets/refonteimplicaction/model/JobPosting.java
Outdated
Show resolved
Hide resolved
* code review