diff --git a/README.adoc b/README.adoc index 00dda29..7e3552b 100644 --- a/README.adoc +++ b/README.adoc @@ -14,7 +14,7 @@ endif::[] == Gruppenbildung Für das Abschlussprojekt bilden Sie bitte eine Gruppe im GitHub Classroom, die -aus 7 bis 9 Personen besteht und laden eine entsprechende `gruppe.yml` in AUAS +aus 7 bis 9 Personen besteht und laden eine entsprechende `group.yml` in AUAS hoch. Im Gegensatz zu den normalen Blättern haben Sie volle Administrationsrechte und alle Projekte sind öffentlich um den Austausch mit anderen Gruppen bei Integrationen zwischen Anwendungen einfacher zu gestalten. @@ -22,18 +22,18 @@ anderen Gruppen bei Integrationen zwischen Anwendungen einfacher zu gestalten. IMPORTANT: Bitte entfernen Sie nicht die Organisatoren aus Ihrem Projekt. NOTE: Sie finden neben dieser README eine -link:gruppe.yml[Beispiel-Gruppe-YML-Datei]. Füllen Sie diese aus, checken Sie +link:group.yml[Beispiel-Gruppe-YML-Datei]. Füllen Sie diese aus, checken Sie sie ein. IMPORTANT: Erstellen Sie direkt eine Abgabe in AUAS. Diese ist *essentiell* für die Teilnahme am Praktikum. Diesmal muss die Abgabe eine ZIP-Datei sein. Die -ZIP-Datei soll die `gruppe.yml` und eine _optionale_ Beschreibung eines eigenen +ZIP-Datei soll die `group.yml` und eine _optionale_ Beschreibung eines eigenen Systems für MOPS beinhalten. Sie können natürlich auch einfach die vorgeschlagenen Systeme wählen und müssen keine eigenen Systeme entwerfen. == Systeme -*Die folgenden Beschreibungen sind keine Spezifikationen, die den Funktionsumfang der Projekte festlegen,* sondern nur ein "Braindump" von den Ideen, die mir zu dem entsprechenden System gekommen sind. Die Beschreibungen sollen Ihnen primär helfen, Systeme auszusuchen, die Sie umsetzen möchten. Wenn Sie einen eigenen Vorschlag für ein System haben, können Sie das auch gerne vorschlagen. Sie müssen dann zusammen mit der `gruppe.yml` Datei einen kurzen Pitch (in etwa wie in diesem Dokument) einreichen, damit wir uns vorstellen können, was Sie machen wollen und den Umfang begutachten. +*Die folgenden Beschreibungen sind keine Spezifikationen, die den Funktionsumfang der Projekte festlegen,* sondern nur ein "Braindump" von den Ideen, die mir zu dem entsprechenden System gekommen sind. Die Beschreibungen sollen Ihnen primär helfen, Systeme auszusuchen, die Sie umsetzen möchten. Wenn Sie einen eigenen Vorschlag für ein System haben, können Sie das auch gerne vorschlagen. Sie müssen dann zusammen mit der `group.yml` Datei einen kurzen Pitch (in etwa wie in diesem Dokument) einreichen, damit wir uns vorstellen können, was Sie machen wollen und den Umfang begutachten. === Online Modulhandbuch Im Modulhandbuch werden die Beschreibungen für Veranstaltungen angegeben. Die Beschreibungen werden von den Dozierenden angelegt und werden von einer Verantwortlichen geprüft und dann freigeschaltet. diff --git a/build.gradle b/build.gradle index 18cce31..0872831 100644 --- a/build.gradle +++ b/build.gradle @@ -4,11 +4,18 @@ plugins { id 'org.springframework.boot' version '2.2.5.RELEASE' id 'io.spring.dependency-management' version '1.0.9.RELEASE' id 'java' - id 'com.github.spotbugs' version "3.0.0" + + id 'com.github.spotbugs' version '3.0.0' id 'checkstyle' } + +group = 'mops' +version = '0.0.1-SNAPSHOT' +sourceCompatibility = '11' + spotbugs{ ignoreFailures = true + reportLevel = "high" effort = "max" toolVersion = '4.0.0-RC1' } @@ -26,10 +33,6 @@ checkstyle { ignoreFailures = true } -group = 'mops' -version = '0.0.1-SNAPSHOT' -sourceCompatibility = '11' - configurations { developmentOnly runtimeClasspath { @@ -56,21 +59,25 @@ dependencies { implementation 'org.springframework.boot:spring-boot-starter-security' implementation 'org.springframework.boot:spring-boot-starter-thymeleaf' implementation 'org.springframework.boot:spring-boot-starter-web' + implementation 'org.keycloak:keycloak-spring-boot-starter:9.0.0' implementation 'org.keycloak.bom:keycloak-adapter-bom:3.3.0.Final' implementation 'mops:styleguide:2.1.0' - compile group: 'io.springfox', name: 'springfox-swagger2', version: '2.7.0' - compile group: 'io.springfox', name: 'springfox-swagger-ui', version: '2.7.0' + implementation 'io.springfox:springfox-swagger2:2.9.2' + implementation 'io.springfox:springfox-swagger-ui:2.9.2' + implementation 'com.github.javafaker:javafaker:1.0.2' + compileOnly 'org.projectlombok:lombok' - developmentOnly 'org.springframework.boot:spring-boot-devtools' - testImplementation 'org.assertj:assertj-core:3.15.0' - runtimeOnly 'com.h2database:h2' annotationProcessor 'org.projectlombok:lombok' + developmentOnly 'org.springframework.boot:spring-boot-devtools' + runtimeOnly 'com.h2database:h2' + + testImplementation 'org.assertj:assertj-core:3.15.0' testImplementation('org.springframework.boot:spring-boot-starter-test') { exclude group: 'org.junit.vintage', module: 'junit-vintage-engine' } testImplementation 'org.springframework.security:spring-security-test' - compile('org.springframework.boot:spring-boot-starter-web') + testImplementation 'com.tngtech.archunit:archunit-junit5:0.13.1' } test { diff --git a/config/checkstyle/checkstyle.xml b/config/checkstyle/checkstyle.xml index 0e70ecf..4506630 100644 --- a/config/checkstyle/checkstyle.xml +++ b/config/checkstyle/checkstyle.xml @@ -229,13 +229,6 @@ - - - - - diff --git a/src/main/java/mops/gruppen2/Gruppen2Application.java b/src/main/java/mops/gruppen2/Gruppen2Application.java index 2e7599a..0861b85 100644 --- a/src/main/java/mops/gruppen2/Gruppen2Application.java +++ b/src/main/java/mops/gruppen2/Gruppen2Application.java @@ -3,11 +3,16 @@ package mops.gruppen2; import org.springframework.boot.SpringApplication; import org.springframework.boot.autoconfigure.SpringBootApplication; import org.springframework.context.annotation.Bean; +import springfox.documentation.builders.PathSelectors; import springfox.documentation.builders.RequestHandlerSelectors; +import springfox.documentation.service.ApiInfo; +import springfox.documentation.service.Contact; import springfox.documentation.spi.DocumentationType; import springfox.documentation.spring.web.plugins.Docket; import springfox.documentation.swagger2.annotations.EnableSwagger2; +import java.util.Collections; + @SpringBootApplication @EnableSwagger2 public class Gruppen2Application { @@ -16,8 +21,25 @@ public class Gruppen2Application { } @Bean - public Docket productAPI(){ - return new Docket(DocumentationType.SWAGGER_2).select() - .apis(RequestHandlerSelectors.basePackage("mops.gruppen2")).build(); + public Docket productAPI() { + return new Docket(DocumentationType.SWAGGER_2) + .select() + .paths(PathSelectors.ant("/products/**")) + .apis(RequestHandlerSelectors.basePackage("mops.gruppen2")) + .build() + .apiInfo(apiMetadata()); + } + + private ApiInfo apiMetadata() { + return new ApiInfo( + "Gruppenbildung API", + "API zum anfragen/aktualisieren der Gruppendaten.", + "0.0.1", + "Free to use", + new Contact("gruppen2", "https://github.com/hhu-propra2/abschlussprojekt-it-bois", ""), + "", + "", + Collections.emptyList() + ); } } diff --git a/src/main/java/mops/gruppen2/controllers/Gruppen2Controller.java b/src/main/java/mops/gruppen2/controller/Gruppen2Controller.java similarity index 82% rename from src/main/java/mops/gruppen2/controllers/Gruppen2Controller.java rename to src/main/java/mops/gruppen2/controller/Gruppen2Controller.java index eea8b70..e5ac048 100644 --- a/src/main/java/mops/gruppen2/controllers/Gruppen2Controller.java +++ b/src/main/java/mops/gruppen2/controller/Gruppen2Controller.java @@ -1,13 +1,11 @@ -package mops.gruppen2.controllers; +package mops.gruppen2.controller; -import mops.gruppen2.services.KeyCloakService; +import mops.gruppen2.service.KeyCloakService; import org.keycloak.adapters.springsecurity.token.KeycloakAuthenticationToken; -import org.springframework.beans.factory.annotation.Autowired; import org.springframework.stereotype.Controller; import org.springframework.ui.Model; import org.springframework.web.bind.annotation.GetMapping; import org.springframework.web.bind.annotation.RequestMapping; -import org.springframework.web.context.annotation.SessionScope; import javax.annotation.security.RolesAllowed; diff --git a/src/main/java/mops/gruppen2/controller/SwaggerAPIControllerExample.java b/src/main/java/mops/gruppen2/controller/SwaggerAPIControllerExample.java new file mode 100644 index 0000000..b186b9e --- /dev/null +++ b/src/main/java/mops/gruppen2/controller/SwaggerAPIControllerExample.java @@ -0,0 +1,47 @@ +package mops.gruppen2.controller; + + +import com.github.javafaker.Faker; +import io.swagger.annotations.ApiOperation; +import io.swagger.annotations.ApiParam; +import mops.gruppen2.domain.ProductSwaggerExample; +import org.springframework.web.bind.annotation.*; + +import java.util.ArrayList; +import java.util.List; + +/** + * Ein Beispiel für eine API mit Swagger. + */ +@RestController +@RequestMapping("/products") +public class SwaggerAPIControllerExample { + + private final Faker faker = new Faker(); + private final List products = new ArrayList<>(); + + @GetMapping("/get/all") + @ApiOperation(value = "Erzeugt eine Liste mit allen gespeicherten Produkten") + public List getProducts() { + return products; + } + + @GetMapping("/get/{index}") + public ProductSwaggerExample getProduct(@ApiParam("Produkt Index") @PathVariable int index) { + return products.get(index); + } + + @PostMapping("/save") + public String saveProduct(@RequestBody ProductSwaggerExample product) { + products.add(product); + + return "Product saved successfully"; + } + + @PostMapping("/random") + public String saveRandomProduct() { + products.add(new ProductSwaggerExample(faker.food().ingredient(), "Empty")); + + return "Product saved successfully"; + } +} diff --git a/src/main/java/mops/gruppen2/controllers/SwaggerAPIController.java b/src/main/java/mops/gruppen2/controllers/SwaggerAPIController.java deleted file mode 100644 index 6811f25..0000000 --- a/src/main/java/mops/gruppen2/controllers/SwaggerAPIController.java +++ /dev/null @@ -1,24 +0,0 @@ -package mops.gruppen2.controllers; - - -import org.springframework.web.bind.annotation.RequestMapping; -import org.springframework.web.bind.annotation.RequestMethod; -import org.springframework.web.bind.annotation.RestController; - -import java.util.ArrayList; -import java.util.List; - -@RestController -public class SwaggerAPIController { - @RequestMapping(value = "/products", method = RequestMethod.GET) - public List getProducts(){ - List productList = new ArrayList<>(); - productList.add("Honey"); - productList.add("Almond"); - return productList; - } - @RequestMapping(value = "/products", method = RequestMethod.POST) - public String createProduct() { - return "Product is saved successfully"; - } -} diff --git a/src/main/java/mops/gruppen2/entities/Aggregat.java b/src/main/java/mops/gruppen2/domain/Aggregate.java similarity index 62% rename from src/main/java/mops/gruppen2/entities/Aggregat.java rename to src/main/java/mops/gruppen2/domain/Aggregate.java index a259113..294cc60 100644 --- a/src/main/java/mops/gruppen2/entities/Aggregat.java +++ b/src/main/java/mops/gruppen2/domain/Aggregate.java @@ -1,15 +1,22 @@ -package mops.gruppen2.entities; +package mops.gruppen2.domain; -import mops.gruppen2.events.Event; +import lombok.Getter; +import mops.gruppen2.domain.event.Event; import java.lang.reflect.Method; -public abstract class Aggregat { +/** + * Repräsentiert viele Events als aggregiertes Objekt. + */ +@Getter +public abstract class Aggregate { + + protected long id; /** * Ruft die spezifische applyEvent-Methode im entsprechenden Aggregat auf. * - * @param event + * @param event Event, welches aggregiert wird */ public void applyEvent(Event event) { try { diff --git a/src/main/java/mops/gruppen2/domain/Group.java b/src/main/java/mops/gruppen2/domain/Group.java new file mode 100644 index 0000000..2716aeb --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/Group.java @@ -0,0 +1,74 @@ +package mops.gruppen2.domain; + +import lombok.EqualsAndHashCode; +import lombok.Getter; +import mops.gruppen2.domain.event.*; + +import java.util.*; + +/** + * Repräsentiert den aggregierten Zustand einer Gruppe. + */ +@EqualsAndHashCode(callSuper = false) +@Getter +public class Group extends Aggregate { + private String title; + private String description; + private final List members; + private final Map roles; + + public Group() { + this.members = new ArrayList<>(); + this.roles = new HashMap<>(); + } + + private void applyEvent(CreateGroupEvent event) { + title = event.getGroupTitle(); + description = event.getGroupDescription(); + id = event.getGroup_id(); + } + + private void applyEvent(UpdateRoleEvent event) { + User user; + + Optional userOptional = members.stream() + .filter(u -> u.getUser_id().equals(event.getUser_id())) + .findFirst(); + + if (userOptional.isPresent()) { + user = userOptional.get(); + } else { + System.out.println("UserNotFoundException"); + return; + } + + if (roles.containsKey(user) && event.getNewRole() == Role.STUDENT) { + roles.remove(user); + } else { + roles.put(user, event.getNewRole()); + } + } + + private void applyEvent(AddUserEvent event) { + User user = new User(event.getUser_id(), event.getGivenname(), event.getFamilyname(), event.getEmail()); + + this.members.add(user); + } + + private void applyEvent(UpdateGroupTitleEvent event) { + this.title = event.getNewGroupTitle(); + } + + private void applyEvent(UpdateGroupDescriptionEvent event) { + this.description = event.getNewGroupDescription(); + } + + private void applyEvent(DeleteUserEvent event) { + for (User user : members) { + if (user.getUser_id().equals(event.getUser_id())) { + this.members.remove(user); + break; + } + } + } +} diff --git a/src/main/java/mops/gruppen2/domain/ProductSwaggerExample.java b/src/main/java/mops/gruppen2/domain/ProductSwaggerExample.java new file mode 100644 index 0000000..14f50dd --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/ProductSwaggerExample.java @@ -0,0 +1,12 @@ +package mops.gruppen2.domain; + +import lombok.Value; + +// @ApiModelProperty +@Value +public class ProductSwaggerExample { + + // @ApiModelProperty + String name; + String description; +} diff --git a/src/main/java/mops/gruppen2/domain/Role.java b/src/main/java/mops/gruppen2/domain/Role.java new file mode 100644 index 0000000..91a6429 --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/Role.java @@ -0,0 +1,5 @@ +package mops.gruppen2.domain; + +public enum Role { + ORGA, ADMIN, STUDENT +} diff --git a/src/main/java/mops/gruppen2/domain/User.java b/src/main/java/mops/gruppen2/domain/User.java new file mode 100644 index 0000000..d162b94 --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/User.java @@ -0,0 +1,14 @@ +package mops.gruppen2.domain; + +import lombok.AllArgsConstructor; +import lombok.Data; + +@Data +@AllArgsConstructor +public class User { + + String user_id; + String givenname; + String familyname; + String email; +} diff --git a/src/main/java/mops/gruppen2/domain/event/AddUserEvent.java b/src/main/java/mops/gruppen2/domain/event/AddUserEvent.java new file mode 100644 index 0000000..fd66ee9 --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/event/AddUserEvent.java @@ -0,0 +1,30 @@ +package mops.gruppen2.domain.event; + +import lombok.EqualsAndHashCode; +import lombok.Value; +import mops.gruppen2.domain.User; + +/** + * Fügt einen einzelnen Nutzer einer Gruppe hinzu. + */ +@EqualsAndHashCode(callSuper = true) +@Value +public class AddUserEvent extends Event { + String givenname; + String familyname; + String email; + + public AddUserEvent(long event_id, long group_id, String user_id, String givenname, String familyname, String email) { + super(event_id, group_id, user_id); + this.givenname = givenname; + this.familyname = familyname; + this.email = email; + } + + public AddUserEvent(long event_id, long group_id, User user) { + super(event_id, group_id, user.getUser_id()); + this.givenname = user.getGivenname(); + this.familyname = user.getFamilyname(); + this.email = user.getEmail(); + } +} diff --git a/src/main/java/mops/gruppen2/domain/event/CreateGroupEvent.java b/src/main/java/mops/gruppen2/domain/event/CreateGroupEvent.java new file mode 100644 index 0000000..1f2a3e1 --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/event/CreateGroupEvent.java @@ -0,0 +1,17 @@ +package mops.gruppen2.domain.event; + +import lombok.EqualsAndHashCode; +import lombok.Value; + +@EqualsAndHashCode(callSuper = true) +@Value +public class CreateGroupEvent extends Event { + String groupTitle; + String groupDescription; + + public CreateGroupEvent(long event_id, long group_id, String user_id, String groupTitle, String groupDescription) { + super(event_id, group_id, user_id); + this.groupTitle = groupTitle; + this.groupDescription = groupDescription; + } +} diff --git a/src/main/java/mops/gruppen2/domain/event/DeleteUserEvent.java b/src/main/java/mops/gruppen2/domain/event/DeleteUserEvent.java new file mode 100644 index 0000000..366dae2 --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/event/DeleteUserEvent.java @@ -0,0 +1,16 @@ +package mops.gruppen2.domain.event; + +import lombok.EqualsAndHashCode; +import lombok.Value; + +/** + * Entfernt ein einzelnes Mitglied einer Gruppe. + */ +@EqualsAndHashCode(callSuper = true) +@Value +public class DeleteUserEvent extends Event { + + public DeleteUserEvent(long event_id, long group_id, String user_id) { + super(event_id, group_id, user_id); + } +} diff --git a/src/main/java/mops/gruppen2/domain/event/Event.java b/src/main/java/mops/gruppen2/domain/event/Event.java new file mode 100644 index 0000000..6835d96 --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/event/Event.java @@ -0,0 +1,12 @@ +package mops.gruppen2.domain.event; + +import lombok.Value; +import lombok.experimental.NonFinal; + +@Value +@NonFinal +public class Event { + long event_id; + long group_id; + String user_id; +} diff --git a/src/main/java/mops/gruppen2/domain/event/UpdateGroupDescriptionEvent.java b/src/main/java/mops/gruppen2/domain/event/UpdateGroupDescriptionEvent.java new file mode 100644 index 0000000..d4bc16c --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/event/UpdateGroupDescriptionEvent.java @@ -0,0 +1,18 @@ +package mops.gruppen2.domain.event; + +import lombok.EqualsAndHashCode; +import lombok.Value; + +/** + * Ändert nur die Gruppenbeschreibung. + */ +@EqualsAndHashCode(callSuper = true) +@Value +public class UpdateGroupDescriptionEvent extends Event { + String newGroupDescription; + + public UpdateGroupDescriptionEvent(long event_id, long group_id, String user_id, String newGroupDescription) { + super(event_id, group_id, user_id); + this.newGroupDescription = newGroupDescription; + } +} diff --git a/src/main/java/mops/gruppen2/domain/event/UpdateGroupTitleEvent.java b/src/main/java/mops/gruppen2/domain/event/UpdateGroupTitleEvent.java new file mode 100644 index 0000000..4f78f17 --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/event/UpdateGroupTitleEvent.java @@ -0,0 +1,18 @@ +package mops.gruppen2.domain.event; + +import lombok.EqualsAndHashCode; +import lombok.Value; + +/** + * Ändert nur den Gruppentitel. + */ +@EqualsAndHashCode(callSuper = true) +@Value +public class UpdateGroupTitleEvent extends Event { + String newGroupTitle; + + public UpdateGroupTitleEvent(long event_id, long group_id, String user_id, String newGroupTitle) { + super(event_id, group_id, user_id); + this.newGroupTitle = newGroupTitle; + } +} diff --git a/src/main/java/mops/gruppen2/domain/event/UpdateRoleEvent.java b/src/main/java/mops/gruppen2/domain/event/UpdateRoleEvent.java new file mode 100644 index 0000000..ae8bd5e --- /dev/null +++ b/src/main/java/mops/gruppen2/domain/event/UpdateRoleEvent.java @@ -0,0 +1,21 @@ +package mops.gruppen2.domain.event; + +import lombok.EqualsAndHashCode; +import lombok.Value; +import mops.gruppen2.domain.Role; + +/** + * Aktualisiert die Gruppenrolle eines Teilnehmers. + */ +@EqualsAndHashCode(callSuper = true) +@Value +public class UpdateRoleEvent extends Event { + + Role newRole; + + public UpdateRoleEvent(long event_id, long group_id, String user_id, Role newRole) { + super(event_id, group_id, user_id); + + this.newRole = newRole; + } +} diff --git a/src/main/java/mops/gruppen2/entities/Admin.java b/src/main/java/mops/gruppen2/entities/Admin.java deleted file mode 100644 index dcc21d8..0000000 --- a/src/main/java/mops/gruppen2/entities/Admin.java +++ /dev/null @@ -1,4 +0,0 @@ -package mops.gruppen2.entities; - -public class Admin extends Rolle { -} diff --git a/src/main/java/mops/gruppen2/entities/Gruppe.java b/src/main/java/mops/gruppen2/entities/Gruppe.java deleted file mode 100644 index 0a7ba4e..0000000 --- a/src/main/java/mops/gruppen2/entities/Gruppe.java +++ /dev/null @@ -1,69 +0,0 @@ -package mops.gruppen2.entities; - -import lombok.Data; -import lombok.EqualsAndHashCode; -import mops.gruppen2.events.*; -import mops.gruppen2.events.AddUser; -import mops.gruppen2.events.CreateGroupEvent; -import mops.gruppen2.events.UpdateGroupDescriptionEvent; -import mops.gruppen2.events.UpdateGroupTitleEvent; -import mops.gruppen2.events.DeleteUserEvent; - -import java.util.ArrayList; -import java.util.HashMap; -import java.util.List; -import java.util.Map; - -@EqualsAndHashCode(callSuper=false) -@Data -public class Gruppe extends Aggregat { - long id; - String titel; - String beschreibung; - List teilnehmersList; - Map rollenList; - - public void applyEvent(CreateGroupEvent event){ - this.id = event.getGruppe_id(); - this.titel = event.getTitel(); - this.beschreibung = event.getBeschreibung(); - this.teilnehmersList = new ArrayList<>(); - this.rollenList = new HashMap<>(); - } - - public void applyEvent(UpdateRoleEvent event) { - teilnehmersList.stream() - .filter(teilnehmer -> teilnehmer.getId().equals(event.getUser_id())) - .findFirst() - .ifPresentOrElse(teilnehmer -> rollenList.put(teilnehmer, event.getRole()), - () -> System.out.println("UserNotFoundException")); - } - - public void applyEvent(AddUser event){ - Teilnehmer teilnehmer = new Teilnehmer(); - - teilnehmer.setId(event.getUser_id()); - teilnehmer.setVorname(event.getVorname()); - teilnehmer.setNachname(event.getNachname()); - teilnehmer.setEmail(event.getEmail()); - - this.teilnehmersList.add(teilnehmer); - } - - public void applyEvent(UpdateGroupTitleEvent event) { - this.titel = event.getTitel(); - } - - public void applyEvent(UpdateGroupDescriptionEvent event) { - this.beschreibung = event.getBeschreibung(); - } - - public void applyEvent(DeleteUserEvent event) { - for (Teilnehmer teilnehmer: teilnehmersList) { - if (teilnehmer.getId().equals(event.getUser_id())) { - this.teilnehmersList.remove(teilnehmer); - break; - } - } - } -} diff --git a/src/main/java/mops/gruppen2/entities/Orga.java b/src/main/java/mops/gruppen2/entities/Orga.java deleted file mode 100644 index ad86b27..0000000 --- a/src/main/java/mops/gruppen2/entities/Orga.java +++ /dev/null @@ -1,4 +0,0 @@ -package mops.gruppen2.entities; - -public class Orga extends Rolle { -} diff --git a/src/main/java/mops/gruppen2/entities/Rolle.java b/src/main/java/mops/gruppen2/entities/Rolle.java deleted file mode 100644 index 1a7bb04..0000000 --- a/src/main/java/mops/gruppen2/entities/Rolle.java +++ /dev/null @@ -1,5 +0,0 @@ -package mops.gruppen2.entities; - -public class Rolle { - -} diff --git a/src/main/java/mops/gruppen2/entities/Teilnehmer.java b/src/main/java/mops/gruppen2/entities/Teilnehmer.java deleted file mode 100644 index 8eff602..0000000 --- a/src/main/java/mops/gruppen2/entities/Teilnehmer.java +++ /dev/null @@ -1,14 +0,0 @@ -package mops.gruppen2.entities; - -import lombok.Data; - -import java.util.List; - -@Data -public class Teilnehmer { - String id; - String vorname; - String nachname; - String email; - List Gruppen; -} diff --git a/src/main/java/mops/gruppen2/events/AddUser.java b/src/main/java/mops/gruppen2/events/AddUser.java deleted file mode 100644 index bb705b5..0000000 --- a/src/main/java/mops/gruppen2/events/AddUser.java +++ /dev/null @@ -1,15 +0,0 @@ -package mops.gruppen2.events; - -import lombok.Getter; - -@Getter -public class AddUser extends Event{ - String vorname, nachname, email; - - public AddUser(long id, long gruppe_id, String user_id, String vorname, String nachname, String email) { - super(id, gruppe_id, user_id); - this.vorname = vorname; - this.nachname = nachname; - this.email = email; - } -} diff --git a/src/main/java/mops/gruppen2/events/CreateGroupEvent.java b/src/main/java/mops/gruppen2/events/CreateGroupEvent.java deleted file mode 100644 index cae95c3..0000000 --- a/src/main/java/mops/gruppen2/events/CreateGroupEvent.java +++ /dev/null @@ -1,15 +0,0 @@ -package mops.gruppen2.events; - -import lombok.Getter; - -@Getter -public class CreateGroupEvent extends Event { - String titel; - String beschreibung; - - public CreateGroupEvent(long id, long gruppe_id, String user_id, String titel, String beschreibung) { - super(id, gruppe_id, user_id); - this.titel = titel; - this.beschreibung = beschreibung; - } -} diff --git a/src/main/java/mops/gruppen2/events/DeleteUserEvent.java b/src/main/java/mops/gruppen2/events/DeleteUserEvent.java deleted file mode 100644 index 97924ce..0000000 --- a/src/main/java/mops/gruppen2/events/DeleteUserEvent.java +++ /dev/null @@ -1,11 +0,0 @@ -package mops.gruppen2.events; - -import lombok.Getter; - -@Getter -public class DeleteUserEvent extends Event{ - - public DeleteUserEvent(long id, long gruppe_id, String user_id) { - super(id, gruppe_id, user_id); - } -} diff --git a/src/main/java/mops/gruppen2/events/Event.java b/src/main/java/mops/gruppen2/events/Event.java deleted file mode 100644 index da9c4e6..0000000 --- a/src/main/java/mops/gruppen2/events/Event.java +++ /dev/null @@ -1,12 +0,0 @@ -package mops.gruppen2.events; - -import lombok.AllArgsConstructor; -import lombok.Getter; - -@Getter -@AllArgsConstructor -public class Event { - long id; - long gruppe_id; - String user_id; -} diff --git a/src/main/java/mops/gruppen2/events/UpdateGroupDescriptionEvent.java b/src/main/java/mops/gruppen2/events/UpdateGroupDescriptionEvent.java deleted file mode 100644 index 232fe56..0000000 --- a/src/main/java/mops/gruppen2/events/UpdateGroupDescriptionEvent.java +++ /dev/null @@ -1,13 +0,0 @@ -package mops.gruppen2.events; - -import lombok.Getter; - -@Getter -public class UpdateGroupDescriptionEvent extends Event { - String beschreibung; - - public UpdateGroupDescriptionEvent(long id, long gruppe_id, String user_id, String beschreibung) { - super(id, gruppe_id, user_id); - this.beschreibung = beschreibung; - } -} diff --git a/src/main/java/mops/gruppen2/events/UpdateGroupTitleEvent.java b/src/main/java/mops/gruppen2/events/UpdateGroupTitleEvent.java deleted file mode 100644 index 4829561..0000000 --- a/src/main/java/mops/gruppen2/events/UpdateGroupTitleEvent.java +++ /dev/null @@ -1,13 +0,0 @@ -package mops.gruppen2.events; - -import lombok.Getter; - -@Getter -public class UpdateGroupTitleEvent extends Event { - String titel; - - public UpdateGroupTitleEvent(long id, long gruppe_id, String user_id, String titel) { - super(id, gruppe_id, user_id); - this.titel = titel; - } -} diff --git a/src/main/java/mops/gruppen2/events/UpdateRoleEvent.java b/src/main/java/mops/gruppen2/events/UpdateRoleEvent.java deleted file mode 100644 index a9d5141..0000000 --- a/src/main/java/mops/gruppen2/events/UpdateRoleEvent.java +++ /dev/null @@ -1,16 +0,0 @@ -package mops.gruppen2.events; - -import lombok.Getter; -import mops.gruppen2.entities.Rolle; - -@Getter -public class UpdateRoleEvent extends Event { - - private final Rolle role; - - public UpdateRoleEvent(long id, long gruppe_id, String user_id, Rolle newRole) { - super(id, gruppe_id, user_id); - - this.role = newRole; - } -} diff --git a/src/main/java/mops/gruppen2/repositories/EventRepository.java b/src/main/java/mops/gruppen2/repository/EventRepository.java similarity index 64% rename from src/main/java/mops/gruppen2/repositories/EventRepository.java rename to src/main/java/mops/gruppen2/repository/EventRepository.java index a3b329d..0e04761 100644 --- a/src/main/java/mops/gruppen2/repositories/EventRepository.java +++ b/src/main/java/mops/gruppen2/repository/EventRepository.java @@ -1,6 +1,6 @@ -package mops.gruppen2.repositories; +package mops.gruppen2.repository; -import mops.gruppen2.events.Event; +import mops.gruppen2.domain.event.Event; import org.springframework.data.repository.CrudRepository; public interface EventRepository extends CrudRepository { diff --git a/src/main/java/mops/gruppen2/security/SecurityConfig.java b/src/main/java/mops/gruppen2/security/SecurityConfig.java index e3299e7..8816b24 100644 --- a/src/main/java/mops/gruppen2/security/SecurityConfig.java +++ b/src/main/java/mops/gruppen2/security/SecurityConfig.java @@ -6,7 +6,11 @@ import org.keycloak.adapters.springsecurity.authentication.KeycloakAuthenticatio import org.keycloak.adapters.springsecurity.config.KeycloakWebSecurityConfigurerAdapter; import org.keycloak.representations.AccessToken; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.context.annotation.*; +import org.springframework.context.annotation.Bean; +import org.springframework.context.annotation.ComponentScan; +import org.springframework.context.annotation.Configuration; +import org.springframework.context.annotation.Scope; +import org.springframework.context.annotation.ScopedProxyMode; import org.springframework.security.config.annotation.authentication.builders.AuthenticationManagerBuilder; import org.springframework.security.config.annotation.method.configuration.EnableGlobalMethodSecurity; import org.springframework.security.config.annotation.method.configuration.GlobalMethodSecurityConfiguration; @@ -70,8 +74,6 @@ class SecurityConfig extends KeycloakWebSecurityConfigurerAdapter { http.headers().frameOptions().disable(); } - - /** * Declaring this class enables us to use the Spring specific * {@link org.springframework.security.access.annotation.Secured} annotation diff --git a/src/main/java/mops/gruppen2/service/GroupService.java b/src/main/java/mops/gruppen2/service/GroupService.java new file mode 100644 index 0000000..42e9d31 --- /dev/null +++ b/src/main/java/mops/gruppen2/service/GroupService.java @@ -0,0 +1,26 @@ +package mops.gruppen2.service; + +import mops.gruppen2.domain.Group; +import mops.gruppen2.domain.event.Event; +import org.springframework.stereotype.Service; + +import java.util.List; + +@Service +public class GroupService { + + /** + * Konstruiert eine vollständige Gruppe aus Events, welche dieselbe Gruppe betreffen. + * + * @param event Initiales CreateGroup-Event + * @param eventList Die restlichen Events für diese Gruppe + * @return Gruppe auf aktuellem Stand + */ + Group buildGroupFromEvents(List eventList) { + Group newGroup = new Group(); + + eventList.forEach(newGroup::applyEvent); + + return newGroup; + } +} diff --git a/src/main/java/mops/gruppen2/services/KeyCloakService.java b/src/main/java/mops/gruppen2/service/KeyCloakService.java similarity index 96% rename from src/main/java/mops/gruppen2/services/KeyCloakService.java rename to src/main/java/mops/gruppen2/service/KeyCloakService.java index 99d2548..825d734 100644 --- a/src/main/java/mops/gruppen2/services/KeyCloakService.java +++ b/src/main/java/mops/gruppen2/service/KeyCloakService.java @@ -1,4 +1,4 @@ -package mops.gruppen2.services; +package mops.gruppen2.service; import mops.gruppen2.security.Account; import org.keycloak.KeycloakPrincipal; diff --git a/src/main/java/mops/gruppen2/service/SerializationService.java b/src/main/java/mops/gruppen2/service/SerializationService.java new file mode 100644 index 0000000..8d3452c --- /dev/null +++ b/src/main/java/mops/gruppen2/service/SerializationService.java @@ -0,0 +1,10 @@ +package mops.gruppen2.service; + +import org.springframework.stereotype.Service; + +/** + * Übersetzt JSON-Event-Payloads zu Java-Event-Repräsentationen und zurück. + */ +@Service +public class SerializationService { +} diff --git a/src/main/java/mops/gruppen2/services/GruppenService.java b/src/main/java/mops/gruppen2/services/GruppenService.java deleted file mode 100644 index 9e72148..0000000 --- a/src/main/java/mops/gruppen2/services/GruppenService.java +++ /dev/null @@ -1,16 +0,0 @@ -package mops.gruppen2.services; - -import mops.gruppen2.events.Event; -import mops.gruppen2.entities.Gruppe; -import org.springframework.stereotype.Service; -import java.util.List; - -@Service -public class GruppenService { - - Gruppe buildGroup(List eventList){ - Gruppe newGroup = new Gruppe(); - eventList.forEach(newGroup::applyEvent); - return newGroup; - } -} diff --git a/src/main/java/mops/gruppen2/services/SerializationService.java b/src/main/java/mops/gruppen2/services/SerializationService.java deleted file mode 100644 index b8de09a..0000000 --- a/src/main/java/mops/gruppen2/services/SerializationService.java +++ /dev/null @@ -1,10 +0,0 @@ -package mops.gruppen2.services; - -import org.springframework.stereotype.Service; - -/** - * Übersetzt und baut - */ -@Service -public class SerializationService { -} diff --git a/src/main/resources/application-dev.properties b/src/main/resources/application-dev.properties index a7ff497..3ad4107 100644 --- a/src/main/resources/application-dev.properties +++ b/src/main/resources/application-dev.properties @@ -14,4 +14,3 @@ keycloak.auth-server-url=https://keycloak.cs.hhu.de/auth keycloak.realm=MOPS keycloak.resource=demo keycloak.public-client=true - diff --git a/src/main/resources/data.sql b/src/main/resources/data.sql deleted file mode 100644 index 0132c78..0000000 --- a/src/main/resources/data.sql +++ /dev/null @@ -1,4 +0,0 @@ --- noinspection SqlNoDataSourceInspectionForFile - -insert into TEILNEHMER (VORNAME, NACHNAME, EMAIL) values - ('Peter', 'Müller', 'Peter@123.de'); diff --git a/src/main/resources/schema.sql b/src/main/resources/schema.sql index 767c37a..51b59eb 100644 --- a/src/main/resources/schema.sql +++ b/src/main/resources/schema.sql @@ -1,25 +1,11 @@ -- noinspection SqlNoDataSourceInspectionForFile -DROP TABLE IF EXISTS teilnehmer; -CREATE TABLE teilnehmer ( - teilnehmer_id INT PRIMARY KEY AUTO_INCREMENT, - vorname VARCHAR(50) NOT NULL, - nachname VARCHAR(50) NOT NULL , - email VARCHAR(255) NOT NULL -); +DROP TABLE IF EXISTS event; -DROP TABLE IF EXISTS gruppe; -CREATE TABLE gruppe +CREATE TABLE event ( - gruppe_id INTEGER PRIMARY KEY auto_increment, - titel TEXT NOT NULL, - beschreibung TEXT NOT NULL -); - -DROP TABLE IF EXISTS teilnahme; -CREATE TABLE teilnahme -( - id INTEGER PRIMARY KEY auto_increment, - teilnehmer_dto INTEGER REFERENCES teilnehmer(teilnehmer_id), - gruppe_dto INTEGER REFERENCES gruppe(gruppe_id) + event_id INT PRIMARY KEY AUTO_INCREMENT, + group_id INT NOT NULL, + user_id VARCHAR(50), + event_payload VARCHAR(255) ); diff --git a/src/test/java/mops/gruppen2/Gruppen2ApplicationTests.java b/src/test/java/mops/gruppen2/Gruppen2ApplicationTests.java index e0b4ab2..5b437f6 100644 --- a/src/test/java/mops/gruppen2/Gruppen2ApplicationTests.java +++ b/src/test/java/mops/gruppen2/Gruppen2ApplicationTests.java @@ -6,8 +6,8 @@ import org.springframework.boot.test.context.SpringBootTest; @SpringBootTest class Gruppen2ApplicationTests { - @Test - void contextLoads() { - } + @Test + void contextLoads() { + } } diff --git a/src/test/java/mops/gruppen2/domain/GroupTest.java b/src/test/java/mops/gruppen2/domain/GroupTest.java new file mode 100644 index 0000000..3901fe0 --- /dev/null +++ b/src/test/java/mops/gruppen2/domain/GroupTest.java @@ -0,0 +1,78 @@ +package mops.gruppen2.domain; + +import mops.gruppen2.domain.event.AddUserEvent; +import mops.gruppen2.domain.event.CreateGroupEvent; +import mops.gruppen2.domain.event.UpdateRoleEvent; +import org.junit.jupiter.api.BeforeEach; +import org.junit.jupiter.api.Disabled; +import org.junit.jupiter.api.Test; + +import static org.assertj.core.api.Assertions.assertThat; + +class GroupTest { + + @BeforeEach + public void setUp(){ + } + + + @Disabled + @Test + void applyEvent() { + } + + @Test + void createSingleGroup() { + CreateGroupEvent createGroupEvent = new CreateGroupEvent(1,2, "asd", "hello", "foo"); + + Group group = new Group(); + group.applyEvent(createGroupEvent); + + assertThat(group.getDescription()).isEqualTo("foo"); + assertThat(group.getTitle()).isEqualTo("hello"); + assertThat(group.getId()).isEqualTo(2); + } + + // Verwendet CreateGroupEvent + @Test + void addSingleUser() { + CreateGroupEvent createGroupEvent = new CreateGroupEvent(1,1,"prof1", "hi", "foo"); + Group group = new Group(); + group.applyEvent(createGroupEvent); + + User user = new User("prof", "jens", "bendi", "hi@gmail.com"); + AddUserEvent addUserEvent = new AddUserEvent(1,1, user); + group.applyEvent(addUserEvent); + + assertThat(group.getMembers().get(0)).isEqualTo(user); + } + + // Verwendet CreateGroupEvent und AddUserEvent + @Test + void updateRoleForExistingUser() { + // Arrange + CreateGroupEvent createGroupEvent = new CreateGroupEvent(1L, 1L, "1L", "gruppe1", "Eine Testgruppe"); + AddUserEvent addUserEvent = new AddUserEvent(1L, 1L, "5L", "Peter", "Pan", "123@mail.de"); + + Group group = new Group(); + group.applyEvent(createGroupEvent); + group.applyEvent(addUserEvent); + + UpdateRoleEvent updateRoleEvent = new UpdateRoleEvent(1L, 1L, "5L", Role.ORGA); + + // Act + group.applyEvent(updateRoleEvent); + + // Assert + assertThat(group.getRoles()) + .containsOnlyKeys(group.getMembers().get(0)) + .containsValue(Role.ORGA); + } + + @Disabled + @Test + void updateRoleForNonExistingUser() { + + } + +} diff --git a/src/test/java/mops/gruppen2/entities/GruppeTest.java b/src/test/java/mops/gruppen2/entities/GruppeTest.java deleted file mode 100644 index ce7da2f..0000000 --- a/src/test/java/mops/gruppen2/entities/GruppeTest.java +++ /dev/null @@ -1,90 +0,0 @@ -package mops.gruppen2.entities; - -import mops.gruppen2.events.AddUser; -import mops.gruppen2.events.CreateGroupEvent; -import mops.gruppen2.events.UpdateRoleEvent; -import org.junit.jupiter.api.BeforeEach; -import org.junit.jupiter.api.Test; - -import java.util.ArrayList; -import java.util.HashMap; -import java.util.List; - -import static org.assertj.core.api.Assertions.assertThat; -import static org.junit.jupiter.api.Assertions.assertEquals; - -class GruppeTest { - - AddUser addUser; - CreateGroupEvent createGroupEvent; - - @BeforeEach - public void setUp(){ - - } - - - @Test - void applyEvent() { - } - - @Test - void applyAddUserEvent(){ - Gruppe gruppe = new Gruppe(); - Gruppe testGruppe = new Gruppe(); - Teilnehmer teilnehmer = new Teilnehmer(); - addUser = new AddUser(1L,1L,"prof","jens","bendi", "hi@gmail.com"); - createGroupEvent = new CreateGroupEvent(1L,1L,"prof1", "hi", "foo"); - - gruppe.applyEvent(createGroupEvent); - gruppe.applyEvent(addUser); - testGruppe.applyEvent(createGroupEvent); - teilnehmer.setId("prof"); - teilnehmer.setVorname("jens"); - teilnehmer.setNachname("bendi"); - teilnehmer.setEmail("hi@gmail.com"); - List testTeil= new ArrayList<>(); - testTeil.add(teilnehmer); - testGruppe.setTeilnehmersList(testTeil); - - assertEquals(testGruppe,gruppe); - } - - // Verwendet CreateGroupEvent und AddUserEvent - @Test - void updateRoleForExistingUser() { - // Arrange - Gruppe gruppe = new Gruppe(); - Orga orga = new Orga(); - - gruppe.applyEvent(new CreateGroupEvent(1L, 1L, "1L", "gruppe1", "Eine Testgruppe")); - gruppe.applyEvent(new AddUser(1L, 1L, "5L", "Peter", "Pan", "123@mail.de")); - - // Act - gruppe.applyEvent(new UpdateRoleEvent(1L, 1L, "5L", orga)); - - // Assert - assertThat(gruppe.getRollenList()) - .containsOnlyKeys(gruppe.getTeilnehmersList().get(0)) - .containsValue(orga); - } - - @Test - void applyCreteGroupEvent() { - String userId = "asd"; - CreateGroupEvent event = new CreateGroupEvent(1L,2,userId, "hello", "foo"); - - Gruppe gruppe1 = new Gruppe(); - gruppe1.applyEvent(event); - - Gruppe gruppe2 = new Gruppe(); - gruppe2.id = 2L; - gruppe2.titel = "hello"; - gruppe2.beschreibung = "foo"; - gruppe2.teilnehmersList = new ArrayList<>(); - gruppe2.rollenList = new HashMap<>(); - - assertEquals(gruppe2, gruppe1); - } - -} \ No newline at end of file