Add first GitHub entities and automatically fetch them for the leaderboard

.gitignore

@@ -1,6 +1,9 @@
 .venv
 .DS_Store
 
+# Idea files
+.idea/
+
 # Byte-compiled / optimized / DLL files
 __pycache__/
 *.py[cod]

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/base/BaseGitServiceEntityConverter.java

@@ -2,13 +2,15 @@
 
 import org.kohsuke.github.GHObject;
 import org.springframework.core.convert.converter.Converter;
+import org.springframework.data.convert.ReadingConverter;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import java.io.IOException;
 import java.time.OffsetDateTime;
 import java.time.ZoneOffset;
 import java.util.Date;
 
+@ReadingConverter
 public abstract class BaseGitServiceEntityConverter<S extends GHObject, T extends BaseGitServiceEntity>
         implements Converter<S, T> {
 

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/comment/IssueComment.java

@@ -7,9 +7,11 @@
 import de.tum.in.www1.hephaestus.codereview.base.BaseGitServiceEntity;
 import de.tum.in.www1.hephaestus.codereview.pullrequest.PullRequest;
 import de.tum.in.www1.hephaestus.codereview.user.User;
+import jakarta.persistence.Basic;
 import jakarta.persistence.Entity;
 import jakarta.persistence.FetchType;
 import jakarta.persistence.JoinColumn;
+import jakarta.persistence.Lob;
 import jakarta.persistence.ManyToOne;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
@@ -24,6 +26,8 @@
 @ToString(callSuper = true)
 public class IssueComment extends BaseGitServiceEntity {
     @NonNull
+    @Lob
+    @Basic(fetch = FetchType.EAGER)
     private String body;
 
     @ManyToOne(fetch = FetchType.LAZY)

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/comment/IssueCommentConverter.java

@@ -2,9 +2,11 @@
 
 import org.kohsuke.github.GHIssueComment;
 import org.springframework.lang.NonNull;
+import org.springframework.stereotype.Service;
 
 import de.tum.in.www1.hephaestus.codereview.base.BaseGitServiceEntityConverter;
 
+@Service
 public class IssueCommentConverter extends BaseGitServiceEntityConverter<GHIssueComment, IssueComment> {
 
     @Override

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/pullrequest/PullRequestController.java

@@ -1,7 +1,9 @@
 package de.tum.in.www1.hephaestus.codereview.pullrequest;
 
+import java.util.Optional;
 import java.util.Set;
 
+import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.RequestMapping;
@@ -17,8 +19,9 @@ public PullRequestController(PullRequestService pullRequestService) {
     }
 
     @GetMapping("/{id}")
-    public PullRequest getPullRequest(@PathVariable Long id) {
-        return pullRequestService.getPullRequestById(id);
+    public ResponseEntity<PullRequest> getPullRequest(@PathVariable Long id) {
+        Optional<PullRequest> pullRequest =pullRequestService.getPullRequestById(id);
+        return pullRequest.map(ResponseEntity::ok).orElseGet(() -> ResponseEntity.notFound().build());
     }
 
     @GetMapping("/author/{login}")

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/pullrequest/PullRequestConverter.java

@@ -1,14 +1,12 @@
 package de.tum.in.www1.hephaestus.codereview.pullrequest;
 
 import org.kohsuke.github.GHPullRequest;
-import org.springframework.data.convert.ReadingConverter;
 import org.springframework.lang.NonNull;
 import org.springframework.stereotype.Service;
 
 import de.tum.in.www1.hephaestus.codereview.base.BaseGitServiceEntityConverter;
 
 @Service
-@ReadingConverter
 public class PullRequestConverter extends BaseGitServiceEntityConverter<GHPullRequest, PullRequest> {
     @Override
     public PullRequest convert(@NonNull GHPullRequest source) {

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/pullrequest/PullRequestService.java

@@ -1,5 +1,6 @@
 package de.tum.in.www1.hephaestus.codereview.pullrequest;
 
+import java.util.Optional;
 import java.util.Set;
 
 import org.slf4j.Logger;
@@ -16,13 +17,13 @@ public PullRequestService(PullRequestRepository pullRequestRepository) {
         this.pullRequestRepository = pullRequestRepository;
     }
 
-    public PullRequest getPullRequestById(Long id) {
-        logger.info("Getting pullRequest with id: " + id);
-        return pullRequestRepository.findById(id).orElse(null);
+    public Optional<PullRequest> getPullRequestById(Long id) {
+        logger.info("Getting pullRequest with id: {}", id);
+        return pullRequestRepository.findById(id);
     }
 
     public Set<PullRequest> getPullRequestsByAuthor(String login) {
-        logger.info("Getting pullRequest by author: " + login);
+        logger.info("Getting pullRequest by author: {}", login);
         return pullRequestRepository.findByAuthor_Login(login);
     }
 

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/repository/RepositoryConverter.java

@@ -2,15 +2,13 @@
 
 import org.kohsuke.github.GHRepository;
 import org.kohsuke.github.GHRepository.Visibility;
-import org.springframework.data.convert.ReadingConverter;
 import org.springframework.lang.NonNull;
 import org.springframework.lang.Nullable;
 import org.springframework.stereotype.Service;
 
 import de.tum.in.www1.hephaestus.codereview.base.BaseGitServiceEntityConverter;
 
 @Service
-@ReadingConverter
 public class RepositoryConverter extends BaseGitServiceEntityConverter<GHRepository, Repository> {
 
     @Override

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/user/UserController.java

@@ -2,13 +2,12 @@
 
 import java.util.Optional;
 
+import org.springframework.http.ResponseEntity;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RestController;
 
-import de.tum.in.www1.hephaestus.errors.EntityNotFoundException;
-
 @RestController
 @RequestMapping("/user")
 public class UserController {
@@ -19,12 +18,9 @@ public UserController(UserService actorService) {
     }
 
     @GetMapping("/{login}")
-    public UserDTO getUser(@PathVariable String login) {
+    public ResponseEntity<UserDTO> getUser(@PathVariable String login) {
         Optional<UserDTO> user = userService.getUserDTO(login);
-        if (user.isEmpty()) {
-            throw new EntityNotFoundException("Actor with login " + login + " not found!");
-        }
-        return user.get();
+        return user.map(ResponseEntity::ok).orElseGet(() -> ResponseEntity.notFound().build());
     }
 
 }

server/application-server/src/main/java/de/tum/in/www1/hephaestus/codereview/user/UserConverter.java

@@ -5,9 +5,11 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.lang.NonNull;
+import org.springframework.stereotype.Service;
 
 import de.tum.in.www1.hephaestus.codereview.base.BaseGitServiceEntityConverter;
 
+@Service
 public class UserConverter extends BaseGitServiceEntityConverter<org.kohsuke.github.GHUser, User> {
 
     protected static final Logger logger = LoggerFactory.getLogger(UserConverter.class);