Skip to content

Commit

Permalink
fix: repair reviewdog findings
Browse files Browse the repository at this point in the history
  • Loading branch information
Hsbalazs committed Sep 23, 2024
1 parent 764fb78 commit a5720e4
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@
import com.greenfoxacademy.backend.services.user.OwnerService;
import java.security.Principal;
import java.util.UUID;

import lombok.RequiredArgsConstructor;
import org.springframework.http.HttpStatus;
import org.springframework.http.ResponseEntity;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,7 @@
import jakarta.persistence.GeneratedValue;
import jakarta.persistence.Id;
import jakarta.persistence.MappedSuperclass;

import java.util.UUID;

import lombok.AllArgsConstructor;
import lombok.Data;
import lombok.Getter;
Expand All @@ -31,6 +29,7 @@ public abstract class User implements UserDetails {
private Long id;
private String firstName;
private String lastName;

@Column(unique = true)
private String email;
private String password;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ class AddPetServiceImplTest {

@Test
void petSuccessfullyAddedToPetRepository() {
CreatePetDto pet = new CreatePetDto("Morzsi","Dog","Male",new Date(2024,9,13));
CreatePetDto pet = new CreatePetDto("Morzsi", "Dog", "Male", new Date(2024, 9, 13));
petService.addPet("owner",pet);

Check warning on line 34 in backend/src/test/java/com/greenfoxacademy/backend/services/pet/AddPetServiceImplTest.java

View workflow job for this annotation

GitHub Actions / build

[testtool] reported by reviewdog 🐶 ',' is not followed by whitespace. Raw Output: /github/workspace/backend/src/test/java/com/greenfoxacademy/backend/services/pet/AddPetServiceImplTest.java:34:30: warning: ',' is not followed by whitespace. (com.puppycrawl.tools.checkstyle.checks.whitespace.WhitespaceAfterCheck)

verify(petService, times(1)).addPet("owner", pet);
Expand Down
2 changes: 1 addition & 1 deletion frontend/src/pages/Logout.test.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { render } from "@testing-library/react";
import {describe, expect, it, vi} from "vitest";
import { describe, expect, it, vi } from "vitest";
import { usePetClinicState } from "../state";
import { Logout } from "./Logout";

Expand Down
2 changes: 1 addition & 1 deletion frontend/src/pages/PetList.test.tsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { ChakraProvider } from "@chakra-ui/react";
import { render, screen, waitFor } from "@testing-library/react";
import { BrowserRouter } from "react-router-dom";
import {describe, expect, Mock, test, vi} from "vitest";
import { type Mock, describe, expect, test, vi } from "vitest";
import { petList } from "../httpClient";
import { PetList } from "./PetList";

Expand Down
2 changes: 1 addition & 1 deletion frontend/src/pages/utils/ProtectedPage.test.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { render, screen } from "@testing-library/react";
import { BrowserRouter } from "react-router-dom";
import {describe, expect, vi, test} from "vitest";
import { describe, expect, test, vi } from "vitest";
import { usePetClinicState } from "../../state";
import { ProtectedPage } from "./ProtectedPage";

Expand Down

0 comments on commit a5720e4

Please sign in to comment.