last changes

[Klyvux]

added comments to search by ing files
added tests to liked recipe list

[github-actions]

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

checkstyle

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'lastSavedRecipe' must be private and have accessor methods.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 54 in 82260a0

Recipe lastSavedRecipe;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'liked' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 62 in 82260a0

List<Recipe> liked = likedByUser.computeIfAbsent(username, u -> new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.LambdaParameterNameCheck> _{reported by reviewdog 🐶}
Name 'u' must match pattern '^(id)|([a-z][a-z0-9][a-zA-Z0-9]+)$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 62 in 82260a0

List<Recipe> liked = likedByUser.computeIfAbsent(username, u -> new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'liked' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 73 in 82260a0

List<Recipe> liked = likedByUser.get(username);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.LambdaParameterNameCheck> _{reported by reviewdog 🐶}
Name 'r' must match pattern '^(id)|([a-z][a-z0-9][a-zA-Z0-9]+)$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 75 in 82260a0

liked.removeIf(r -> r.getRecipeId() == recipeId);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.ReturnCountCheck> _{reported by reviewdog 🐶}
Return count is 3 (max allowed for non-void methods/lambdas is 1).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 79 in 82260a0

@Override

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'liked' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 84 in 82260a0

List<Recipe> liked = likedByUser.get(username);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'u' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 88 in 82260a0

User u = usersByName.get(username);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.FinalClassCheck> _{reported by reviewdog 🐶}
Class FakePresenter should be declared as final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 131 in 82260a0

private static class FakePresenter implements LikedRecipeOutputBoundary {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'lastOutput' must be private and have accessor methods.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 133 in 82260a0

LikedRecipeOutputData lastOutput;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'lastHandsfreeInput' must be private and have accessor methods.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 134 in 82260a0

StepByStepInputData lastHandsfreeInput;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'lastError' must be private and have accessor methods.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 135 in 82260a0

String lastError;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.ExplicitInitializationCheck> _{reported by reviewdog 🐶}
Variable 'likedViewCalls' explicitly initialized to '0' (default value for its type).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 137 in 82260a0

int likedViewCalls = 0;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'likedViewCalls' must be private and have accessor methods.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 137 in 82260a0

int likedViewCalls = 0;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.ExplicitInitializationCheck> _{reported by reviewdog 🐶}
Variable 'handsfreeCalls' explicitly initialized to '0' (default value for its type).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 138 in 82260a0

int handsfreeCalls = 0;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'handsfreeCalls' must be private and have accessor methods.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 138 in 82260a0

int handsfreeCalls = 0;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.ExplicitInitializationCheck> _{reported by reviewdog 🐶}
Variable 'failCalls' explicitly initialized to '0' (default value for its type).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 139 in 82260a0

int failCalls = 0;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'failCalls' must be private and have accessor methods.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 139 in 82260a0

int failCalls = 0;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 161 in 82260a0

private User newUser(String username) {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 169 in 82260a0

private Recipe newRecipe(int id, String name, List<Ingredient> ingredients) {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'nutr' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 170 in 82260a0

Map<String, Double> nutr = new HashMap<>();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 177 in 82260a0

private LikedRecipeInputData inputFor(int id) {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 183 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addLikedRecipe_successNewRecipe' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 184 in 82260a0

public void addLikedRecipe_successNewRecipe() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 185 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 186 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'user' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 188 in 82260a0

User user = newUser("user1");

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'recipe' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 191 in 82260a0

Recipe recipe = newRecipe(1, "R1", new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 194 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.TrailingCommentCheck> _{reported by reviewdog 🐶}
Don't use trailing comments.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 203 in 82260a0

assertNotNull(presenter.lastOutput); // from loadLikedRecipes()

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 206 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addLikedRecipe_recipeNotFound' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 207 in 82260a0

public void addLikedRecipe_recipeNotFound() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 208 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 209 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 214 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 223 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addLikedRecipe_userNotFound' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 224 in 82260a0

public void addLikedRecipe_userNotFound() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 225 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 226 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'recipe' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 228 in 82260a0

Recipe recipe = newRecipe(1, "R1", new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 232 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 241 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addLikedRecipe_alreadySaved' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 242 in 82260a0

public void addLikedRecipe_alreadySaved() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 243 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'presenter' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 244 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 244 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'user' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 246 in 82260a0

User user = newUser("user1");

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'recipe' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 247 in 82260a0

Recipe recipe = newRecipe(1, "R1", new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.TrailingCommentCheck> _{reported by reviewdog 🐶}
Don't use trailing comments.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 248 in 82260a0

user.getSavedRecipes().add(recipe); // already saved

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 253 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 264 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addLikedRecipe_exceptionHandled' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 265 in 82260a0

public void addLikedRecipe_exceptionHandled() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 266 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 267 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 271 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 281 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'deleteLikedRecipe_success' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 282 in 82260a0

public void deleteLikedRecipe_success() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 283 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'presenter' declaration and its first usage is 5, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 284 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 284 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'user' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 286 in 82260a0

User user = newUser("user1");

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'r1' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 287 in 82260a0

Recipe r1 = newRecipe(1, "R1", new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'r2' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 288 in 82260a0

Recipe r2 = newRecipe(2, "R2", new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 295 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.TrailingCommentCheck> _{reported by reviewdog 🐶}
Don't use trailing comments.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 303 in 82260a0

assertNotNull(presenter.lastOutput); // from loadLikedRecipes()

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 307 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'deleteLikedRecipe_userNotFound' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 308 in 82260a0

public void deleteLikedRecipe_userNotFound() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 309 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 310 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 313 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 321 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'deleteLikedRecipe_exceptionHandled' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 322 in 82260a0

public void deleteLikedRecipe_exceptionHandled() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 323 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 324 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 328 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 338 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'loadLikedRecipes_populatesOutput' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 339 in 82260a0

public void loadLikedRecipes_populatesOutput() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'dao' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 340 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 340 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'presenter' declaration and its first usage is 6, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 341 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 341 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'user' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 343 in 82260a0

User user = newUser("user1");

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'ingredients' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 345 in 82260a0

List<Ingredient> ingredients = new ArrayList<>();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'r1' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 349 in 82260a0

Recipe r1 = newRecipe(10, "Pizza", ingredients);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 355 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'out' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 362 in 82260a0

LikedRecipeOutputData out = presenter.lastOutput;

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'ingStrings' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 369 in 82260a0

List<String[]> ingStrings = out.getRecipeIngredients().get(0);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 376 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'loadLikedRecipes_exceptionHandled' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 377 in 82260a0

public void loadLikedRecipes_exceptionHandled() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 378 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 379 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 383 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 393 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'handsfree_success' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 394 in 82260a0

public void handsfree_success() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 395 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 396 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 403 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'result' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 405 in 82260a0

List<InstructionStep> result = interactor.handsfree(inputFor(123));

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'instructions' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 411 in 82260a0

RecipeInstructions instructions = presenter.lastHandsfreeInput.instructions();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 416 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'handsfree_exceptionHandled' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 417 in 82260a0

public void handsfree_exceptionHandled() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 418 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 419 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 423 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'result' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 425 in 82260a0

List<InstructionStep> result = interactor.handsfree(inputFor(123));

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 434 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addIngredientsToGrocery_recipeNotFound' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 435 in 82260a0

public void addIngredientsToGrocery_recipeNotFound() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 436 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 437 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 442 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 451 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addIngredientsToGrocery_noIngredients' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 452 in 82260a0

public void addIngredientsToGrocery_noIngredients() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 453 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'presenter' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 454 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 454 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'user' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 456 in 82260a0

User user = newUser("user1");

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'recipe' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 457 in 82260a0

Recipe recipe = newRecipe(10, "Empty", new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 463 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 472 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addIngredientsToGrocery_success' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 473 in 82260a0

public void addIngredientsToGrocery_success() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'dao' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 474 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 474 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'presenter' declaration and its first usage is 6, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 475 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 475 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'user' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 477 in 82260a0

User user = newUser("user1");

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'ing' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 478 in 82260a0

List<Ingredient> ing = new ArrayList<>();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'recipe' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 481 in 82260a0

Recipe recipe = newRecipe(10, "WithIngredients", ing);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 487 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.TrailingCommentCheck> _{reported by reviewdog 🐶}
Don't use trailing comments.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 494 in 82260a0

assertEquals(1, presenter.likedViewCalls); // from loadLikedRecipes()

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 498 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addIngredientsToGrocery_exceptionHandled' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 499 in 82260a0

public void addIngredientsToGrocery_exceptionHandled() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 500 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 501 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 505 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 513 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addLikedRecipe_alreadyHasDifferentRecipeId' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 514 in 82260a0

public void addLikedRecipe_alreadyHasDifferentRecipeId() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 515 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'presenter' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 516 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 516 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'user' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 518 in 82260a0

User user = newUser("user1");

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'existing' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 521 in 82260a0

Recipe existing = newRecipe(99, "Other", new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'newOne' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 525 in 82260a0

Recipe newOne = newRecipe(1, "New", new ArrayList<>());

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 530 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.TrailingCommentCheck> _{reported by reviewdog 🐶}
Don't use trailing comments.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 538 in 82260a0

assertNotNull(presenter.lastOutput); // loadLikedRecipes called

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 542 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addIngredientsToGrocery_nonMatchingRecipeIdInList' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 543 in 82260a0

public void addIngredientsToGrocery_nonMatchingRecipeIdInList() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 544 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'presenter' declaration and its first usage is 5, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 545 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 545 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'user' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 547 in 82260a0

User user = newUser("user1");

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'ing' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 550 in 82260a0

List<Ingredient> ing = new ArrayList<>();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'r' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 552 in 82260a0

Recipe r = newRecipe(5, "SomeRecipe", ing); // ID 5

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.TrailingCommentCheck> _{reported by reviewdog 🐶}
Don't use trailing comments.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 552 in 82260a0

Recipe r = newRecipe(5, "SomeRecipe", ing); // ID 5

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 558 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.InnerTypeLastCheck> _{reported by reviewdog 🐶}
Init blocks, constructors, fields and methods should be before inner types.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 569 in 82260a0

@Test

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.MethodNameCheck> _{reported by reviewdog 🐶}
Name 'addIngredientsToGrocery_ingredientsNull' must match pattern '^[a-z][a-zA-Z0-9]*$'.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 570 in 82260a0

public void addIngredientsToGrocery_ingredientsNull() {

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'dao' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 571 in 82260a0

FakeDataAccess dao = new FakeDataAccess();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'presenter' declaration and its first usage is 5, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 572 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'presenter' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 572 in 82260a0

FakePresenter presenter = new FakePresenter();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'user' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 574 in 82260a0

User user = newUser("user1");

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'nutr' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 577 in 82260a0

Map<String, Double> nutr = new HashMap<>();

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'recipeWithNullIngredients' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 579 in 82260a0

Recipe recipeWithNullIngredients =

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.FinalLocalVariableCheck> _{reported by reviewdog 🐶}
Variable 'interactor' should be declared final.

CSC207_Project/src/test/java/use_case/liked_recipe_list/LikedRecipeInteractorTest.java

Line 587 in 82260a0

LikedRecipeInteractor interactor = new LikedRecipeInteractor(dao, presenter);

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

CSC207_Project/src/main/java/use_case/search_by_ingr/SearchByIngredientInputData.java

Line 26 in 82260a0

/**

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

CSC207_Project/src/main/java/use_case/search_by_ingr/SearchByIngredientInputData.java

Line 33 in 82260a0

/**

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

CSC207_Project/src/main/java/use_case/search_by_ingr/SearchByIngredientOutputData.java

Line 25 in 82260a0

/**

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

CSC207_Project/src/main/java/use_case/search_by_ingr/SearchByIngredientOutputData.java

Line 32 in 82260a0

/**

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

CSC207_Project/src/main/java/interface_adapter/search_by_ingr/SearchByIngredientState.java

Line 23 in 82260a0

/**

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

CSC207_Project/src/main/java/interface_adapter/search_by_ingr/SearchByIngredientState.java

Line 39 in 82260a0

/**

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

CSC207_Project/src/main/java/interface_adapter/search_by_ingr/SearchByIngredientState.java

Line 55 in 82260a0

/**

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

CSC207_Project/src/main/java/interface_adapter/search_by_ingr/SearchByIngredientState.java

Line 71 in 82260a0

/**

---

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck> _{reported by reviewdog 🐶}
Summary javadoc is missing.

CSC207_Project/src/main/java/view/SearchByIngredientView.java

Line 305 in 82260a0

/**

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.naming.PackageNameCheck> _{reported by reviewdog 🐶}
Name 'use_case.liked_recipe_list' must match pattern '^[a-z]+(.[a-zA-Z_]\w*)*$'.

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck> _{reported by reviewdog 🐶}
Import statement for 'org.junit.Test' is in the wrong order. Should be in the 'THIRD_PARTY_PACKAGE' group, expecting not assigned imports on this line.

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.imports.ImportOrderCheck> _{reported by reviewdog 🐶}
Wrong order for 'org.junit.Test' import.

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.imports.ImportOrderCheck> _{reported by reviewdog 🐶}
'use_case.step_by_step.StepByStepInputData' should be separated from previous imports.

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck> _{reported by reviewdog 🐶}
Import statement for 'java.util.*' is in the wrong order. Should be in the 'STANDARD_JAVA_PACKAGE' group, expecting not assigned imports on this line.

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.ExplicitInitializationCheck> _{reported by reviewdog 🐶}
Variable 'throwOnGetAnalyzedInstructions' explicitly initialized to 'false' (default value for its type).

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'throwOnGetAnalyzedInstructions' must be private and have accessor methods.

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.ExplicitInitializationCheck> _{reported by reviewdog 🐶}
Variable 'throwOnGetLikedRecipesInGrocery' explicitly initialized to 'false' (default value for its type).

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'throwOnGetLikedRecipesInGrocery' must be private and have accessor methods.

[github-actions]

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.design.VisibilityModifierCheck> _{reported by reviewdog 🐶}
Variable 'lastSavedUsername' must be private and have accessor methods.

[Adish-Singh]

Thank you for adding the comments. The tests have 100% line coverage so great job.