SZUT/ProjectManagmentTool
5
0
Fork 0
Code Issues Pull requests Releases Wiki Activity Actions

PMT-16: Projekt Anlegen #12

Merged
SZUT-Ole merged 10 commits from story/PMT-16-projekt-anlegen into trunk 2024-10-15 08:15:23 +00:00
Conversation 7 Commits 10 Files changed 6 +291 -11
SZUT-Rajbir commented 2024-10-10 16:02:28 +00:00
Member
Copy link
No description provided.
SZUT-Rajbir added 12 commits 2024-10-10 16:02:28 +00:00
.
Some checks failed
Quality Check / Validate OAS (push) Successful in 34s
Details
Quality Check / Linting (push) Failing after 52s
Details
Quality Check / Static Analysis (push) Failing after 1m0s
Details
Quality Check / Testing (push) Successful in 1m4s
Details
d84647f9e6 
Signed-off-by: Dominik Säume <Dominik.Saeume@hmmh.de>
Signed-off-by: Dominik Säume <Dominik.Saeume@hmmh.de>
Signed-off-by: Dominik Säume <Dominik.Saeume@hmmh.de>
Co-authored-by: Rajbir Singh <rajbir.singh@hmmh.de>
Signed-off-by: Dominik Säume <Dominik.Saeume@hmmh.de>
Signed-off-by: Dominik Säume <Dominik.Saeume@hmmh.de>
testing....... (Comit needs to be edited later)
All checks were successful
Quality Check / Validate OAS (push) Successful in 32s
Details
Quality Check / Linting (push) Successful in 1m11s
Details
Quality Check / Testing (push) Successful in 1m13s
Details
Quality Check / Static Analysis (push) Successful in 1m16s
Details
57de299b99 
Co-authored-by: Rajbir Singh <rajbir.singh@hmmh.de>
Signed-off-by: Dominik Säume <Dominik.Saeume@hmmh.de>
# Conflicts:
#	api/pmt.yml
#	src/main/java/de/hmmh/pmt/ApiController.java
added shouldNotCreateProjectWhenAdministratorDoesNotExist test
All checks were successful
Quality Check / Validate OAS (push) Successful in 33s
Details
Quality Check / Linting (push) Successful in 1m7s
Details
Quality Check / Static Analysis (push) Successful in 1m15s
Details
Quality Check / Testing (push) Successful in 1m14s
Details
Quality Check / Validate OAS (pull_request) Successful in 32s
Details
Quality Check / Linting (pull_request) Successful in 1m9s
Details
Quality Check / Testing (pull_request) Successful in 1m14s
Details
Quality Check / Static Analysis (pull_request) Successful in 1m16s
Details
609cafcb71
SZUT-Rajbir requested review from SZUT-Dominik 2024-10-10 16:03:10 +00:00
SZUT-Dominik requested changes 2024-10-11 10:58:49 +00:00
Dismissed
SZUT-Dominik left a comment
Owner
Copy link

Dirty Git History

Dirty Git History
SZUT-Dominik force-pushed story/PMT-16-projekt-anlegen from 609cafcb71 to 80a8a74373 2024-10-11 11:08:08 +00:00 Compare
SZUT-Dominik requested changes 2024-10-11 11:14:03 +00:00
Dismissed
SZUT-Dominik left a comment
Owner
Copy link

Ich kann kein ordentliches Review machen, bis ihr:

  • Auf Trunk ge Rebased habt.
  • Die letzten 3 Commits für Testing wie ich euch gesagt habe Squash
Ich kann kein ordentliches Review machen, bis ihr: - [x] Auf Trunk ge Rebased habt. - [x] Die letzten 3 Commits für Testing wie ich euch gesagt habe Squash
SZUT-Ole commented 2024-10-11 11:16:33 +00:00
Member
Copy link
  • Die letzten 3 Commits für Testing wie ich euch gesagt habe Squash

Was zum bums ist Squash?

> - [ ] Die letzten 3 Commits für Testing wie ich euch gesagt habe Squash Was zum bums ist Squash?
SZUT-Dominik commented 2024-10-11 11:20:19 +00:00
Owner
Copy link

Den Inhalt der 3 Commits in eine machen und die Message anpassen.

  1. git reset --soft HEAD~3 löst die letzten 3 Commits auf
  2. git commit -m "TICKETNUMMER: Nachricht" Inhalt neu committen mit einer Nachricht
  3. git log --all --decorate --oneline --graph neue History mit Alter Vergleichen
  4. git push --force-with-lease Git Branch Upstream sicherer überschreiben
Den Inhalt der 3 Commits in eine machen und die Message anpassen. 1. `git reset --soft HEAD~3` löst die letzten 3 Commits auf 2. `git commit -m "TICKETNUMMER: Nachricht"` Inhalt neu committen mit einer Nachricht 1. `git log --all --decorate --oneline --graph` neue History mit Alter Vergleichen 3. `git push --force-with-lease` Git Branch Upstream sicherer überschreiben
SZUT-Dominik changed title from story/PMT-16-projekt-anlegen to PMT-16: Projekt Anlegen 2024-10-11 11:22:21 +00:00
SZUT-Dominik added the due date 2024-10-11 2024-10-11 11:23:16 +00:00
SZUT-Rajbir force-pushed story/PMT-16-projekt-anlegen from 80a8a74373 to bd8f92a9c6 2024-10-11 15:01:02 +00:00 Compare
SZUT-Dominik commented 2024-10-11 15:43:17 +00:00
Owner
Copy link

@SZUT-Rajbir jetzt hast du wieder was anderes gemacht, du hast wieder nicht den Stand aus dem Gut sondern deinen Lokalen genutzt.

Du hast nicht gepullt/ hard resettet.

Ich fixe das Wieder wenn ich zuhause bin.
Und wir Organisieren dir Nochmal eine Git Schulung

@SZUT-Rajbir jetzt hast du wieder was anderes gemacht, du hast wieder nicht den Stand aus dem Gut sondern deinen Lokalen genutzt. Du hast nicht gepullt/ hard resettet. Ich fixe das Wieder wenn ich zuhause bin. Und wir Organisieren dir Nochmal eine Git Schulung
SZUT-Rajbir commented 2024-10-11 15:53:34 +00:00
Author
Member
Copy link

@SZUT-Rajbir jetzt hast du wieder was anderes gemacht, du hast wieder nicht den Stand aus dem Gut sondern deinen Lokalen genutzt.

Du hast nicht gepullt/ hard resettet.

Ich fixe das Wieder wenn ich zuhause bin.
Und wir Organisieren dir Nochmal eine Git Schulung

sorry, habe gerade auch festgestellt, dass ich mit dem lokalen Stand von trunk gerebased. Wäre gut wenn du das wieder herstellen kannst. Notiere mir doch bitte die Schritte die du machst, damit ich das einmal nachvollziehen kann. Danke

> @SZUT-Rajbir jetzt hast du wieder was anderes gemacht, du hast wieder nicht den Stand aus dem Gut sondern deinen Lokalen genutzt. > > Du hast nicht gepullt/ hard resettet. > > Ich fixe das Wieder wenn ich zuhause bin. > Und wir Organisieren dir Nochmal eine Git Schulung sorry, habe gerade auch festgestellt, dass ich mit dem lokalen Stand von trunk gerebased. Wäre gut wenn du das wieder herstellen kannst. Notiere mir doch bitte die Schritte die du machst, damit ich das einmal nachvollziehen kann. Danke
SZUT-Dominik commented 2024-10-11 17:25:05 +00:00
Owner
Copy link

Ich hatte ja sogar das was gemacht werden muss nochmal genauer runtergeschrieben. Es hätte garnichts gerebased werden müssen

Ich reppariere es wenn ich zuhause bin. Ist jedes mal abhängig davon wie es kapput ist, wie man es reppariert

Ich hatte ja sogar das was gemacht werden muss nochmal genauer runtergeschrieben. Es hätte garnichts gerebased werden müssen Ich reppariere es wenn ich zuhause bin. Ist jedes mal abhängig davon wie es kapput ist, wie man es reppariert
SZUT-Dominik force-pushed story/PMT-16-projekt-anlegen from bd8f92a9c6 to fcf1371891 2024-10-11 19:03:48 +00:00 Compare
SZUT-Dominik commented 2024-10-11 19:08:39 +00:00
Owner
Copy link

@SZUT-Rajbir hier ein Video wie ich es gefixt habe:
Ich habe extra für dich keine meiner Git Aliases genutzt.

@SZUT-Rajbir hier ein Video wie ich es gefixt habe: Ich habe extra für dich keine meiner Git Aliases genutzt.
Screencast from 2024-10-11 20-58-07.mp4
54 MiB
SZUT-Dominik force-pushed story/PMT-16-projekt-anlegen from fcf1371891 to 9bf8607b43 2024-10-11 19:23:20 +00:00 Compare
SZUT-Dominik commented 2024-10-11 19:23:53 +00:00
Owner
Copy link

Ich habe die Letzte Commit nochmal ge Reworded

Ich habe die Letzte Commit nochmal ge Reworded
SZUT-Dominik requested changes 2024-10-11 19:42:30 +00:00
Dismissed
SZUT-Dominik left a comment
Owner
Copy link

Die Probleme die Ich gefunden habe im aktuellen Code müssen noch gelöst werden

Außerdem ergeben sich aus der API Doc und dem Implementierungscode,
noch fälle, die getestet werden müssen:

  • HttpClientErrorException, die kein 404 ist, sollte zu einem Unavailable führen
  • eine RestClientException und es soll nen internal Server Error zurückgegeben werden
  • Invalide Timestamps, weswegen ein Unprocessable zurückkommen sollte
Die Probleme die Ich gefunden habe im aktuellen Code müssen noch gelöst werden Außerdem ergeben sich aus der API Doc und dem Implementierungscode, noch fälle, die getestet werden müssen: - HttpClientErrorException, die kein 404 ist, sollte zu einem Unavailable führen - eine RestClientException und es soll nen internal Server Error zurückgegeben werden - Invalide Timestamps, weswegen ein Unprocessable zurückkommen sollte
src/test/java/de/hmmh/pmt/project/CreateTest.java Outdated
@ -0,0 +40,4 @@
.contentType(MediaType.APPLICATION_JSON);
this.mvc
.perform(requestBuilder)
SZUT-Dominik commented 2024-10-11 19:31:18 +00:00
Owner
Copy link

Nutzt hier doch doirekt die Methode inline, also .perform(getRequest(createDTO))

Nutzt hier doch doirekt die Methode inline, also `.perform(getRequest(createDTO))`
SZUT-Ole marked this conversation as resolved
src/test/java/de/hmmh/pmt/project/CreateTest.java Outdated
@ -0,0 +63,4 @@
.andExpect(status().isCreated())
.andExpect(jsonPath("$.id").exists());
RequestBuilder secondProjectRequestBuilder = createProjectRequestBuilder(createDTO);
SZUT-Dominik commented 2024-10-11 19:32:19 +00:00
Owner
Copy link

Useless 2 Posts, nehmt einfach Einen Der Existierenden Datensätze und Versucht Ihn erneut zu schreiben

Useless 2 Posts, nehmt einfach Einen Der Existierenden Datensätze und Versucht Ihn erneut zu schreiben
SZUT-Ole marked this conversation as resolved
src/test/java/de/hmmh/pmt/project/CreateTest.java Outdated
@ -0,0 +72,4 @@
@Test
void shouldNotCreateProjectWhenAdministratorDoesNotExist() throws Exception {
HttpClientErrorException httpClientErrorException = mock(HttpClientErrorException.class);
SZUT-Dominik commented 2024-10-11 19:37:00 +00:00
Owner
Copy link

Viel zu Komplex, Mein Ergebnis, wenn man die Spring Doc liest:

when(this.mockEmployeeApi.findById(Mockito.anyLong()))
                .thenThrow(new HttpClientErrorException(HttpStatus.NOT_FOUND));
``
Viel zu Komplex, Mein Ergebnis, wenn man die Spring Doc liest: ```java when(this.mockEmployeeApi.findById(Mockito.anyLong())) .thenThrow(new HttpClientErrorException(HttpStatus.NOT_FOUND)); ``
SZUT-Ole marked this conversation as resolved
src/test/java/de/hmmh/pmt/project/CreateTest.java Outdated
@ -0,0 +87,4 @@
createDTO.setStart(LocalDateTime.of(2000, 1, 13, 12, 51));
createDTO.setPlannedEnd(LocalDateTime.of(2002, 3, 21, 11, 42));
RequestBuilder projectRequestBuilder = createProjectRequestBuilder(createDTO);
SZUT-Dominik commented 2024-10-11 19:37:34 +00:00
Owner
Copy link

Wofür die Zwischen Variable, kann doch einfach Inline gemovt werden.

Wofür die Zwischen Variable, kann doch einfach Inline gemovt werden.
SZUT-Ole marked this conversation as resolved
src/test/java/de/hmmh/pmt/project/CreateTest.java Outdated
@ -0,0 +94,4 @@
.andExpect(status().isNotFound());
}
private RequestBuilder createProjectRequestBuilder(CreateProjectDTO createDTO) throws Exception {
SZUT-Dominik commented 2024-10-11 19:30:14 +00:00
Owner
Copy link

Das Ganze Baute Euch den Inhalt einer Request zusammen, Würde es dementsprechend
getRequest() nennen

Das Ganze Baute Euch den Inhalt einer Request zusammen, Würde es dementsprechend `getRequest()` nennen
SZUT-Ole marked this conversation as resolved
SZUT-Dominik removed the due date 2024-10-11 2024-10-11 19:48:27 +00:00
SZUT-Dominik added the due date 2024-10-14 2024-10-11 19:48:36 +00:00
SZUT-Ole added 1 commit 2024-10-14 08:59:40 +00:00
PMT-16: Edit and create more test cases
All checks were successful
Quality Check / Validate OAS (push) Successful in 53s
Details
Quality Check / Validate OAS (pull_request) Successful in 1m7s
Details
Quality Check / Linting (push) Successful in 2m7s
Details
Quality Check / Linting (pull_request) Successful in 2m12s
Details
Quality Check / Testing (push) Successful in 2m21s
Details
Quality Check / Static Analysis (push) Successful in 2m25s
Details
Quality Check / Testing (pull_request) Successful in 2m16s
Details
Quality Check / Static Analysis (pull_request) Successful in 2m20s
Details
499d1279d1
SZUT-Ole added 1 commit 2024-10-14 09:07:51 +00:00
PMT-16: Cleanup Test Cases
All checks were successful
Quality Check / Validate OAS (push) Successful in 50s
Details
Quality Check / Validate OAS (pull_request) Successful in 1m6s
Details
Quality Check / Linting (push) Successful in 2m2s
Details
Quality Check / Linting (pull_request) Successful in 2m9s
Details
Quality Check / Testing (push) Successful in 2m18s
Details
Quality Check / Testing (pull_request) Successful in 2m14s
Details
Quality Check / Static Analysis (push) Successful in 2m24s
Details
Quality Check / Static Analysis (pull_request) Successful in 2m18s
Details
03e33d654e
SZUT-Dominik approved these changes 2024-10-14 09:08:02 +00:00
SZUT-Ole scheduled this pull request to auto merge when all checks succeed 2024-10-14 09:08:55 +00:00
Snoweuph force-pushed story/PMT-16-projekt-anlegen from 03e33d654e to 50d889820d 2024-10-15 08:12:50 +00:00 Compare
SZUT-Ole merged commit d38c5a1084 into trunk 2024-10-15 08:15:23 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
SZUT-Dominik
Labels
Clear labels
  
type
Bug

  
type
Story

  
type
Task

No labels
type
Bug
type
Story
type
Task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

2024-10-14

Dependencies

No dependencies set.

Reference: SZUT/ProjectManagmentTool#12
No description provided.
Delete branch "story/PMT-16-projekt-anlegen"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?