Skip to content
Snippets Groups Projects
Commit 2c3bc594 authored by Erik Moravec's avatar Erik Moravec Committed by Ján Dovjak
Browse files

fix: fixed warnings in facades tests

parent 981b561a
No related branches found
No related tags found
No related merge requests found
...@@ -45,13 +45,6 @@ public class LeagueManagerFacadeImplTest { ...@@ -45,13 +45,6 @@ public class LeagueManagerFacadeImplTest {
return team; return team;
} }
public TeamDto buildTeamDTO(Long id, String name) {
var team = new TeamDto();
team.setId(id);
team.setName(name);
return team;
}
public Game buildGameMock(Long id) { public Game buildGameMock(Long id) {
var game = mock(Game.class); var game = mock(Game.class);
when(game.getId()).thenReturn(id); when(game.getId()).thenReturn(id);
...@@ -111,7 +104,7 @@ public class LeagueManagerFacadeImplTest { ...@@ -111,7 +104,7 @@ public class LeagueManagerFacadeImplTest {
public void updateGameScore() { public void updateGameScore() {
var game = buildGameMock(1L); var game = buildGameMock(1L);
var gameUpdate = buildGameUpdateDTO(game, 1, 2); var gameUpdate = buildGameUpdateDTO(game, 1, 2);
when(gameService.findGame(1L)).thenReturn(Optional.ofNullable(game)); when(gameService.findGame(1L)).thenReturn(Optional.of(game));
leagueManagerFacade.updateGameScore(gameUpdate); leagueManagerFacade.updateGameScore(gameUpdate);
verify(gameService).updateGameScore(game, 1, 2); verify(gameService).updateGameScore(game, 1, 2);
} }
...@@ -122,6 +115,7 @@ public class LeagueManagerFacadeImplTest { ...@@ -122,6 +115,7 @@ public class LeagueManagerFacadeImplTest {
when(gameService.findGame(1L)).thenReturn(Optional.ofNullable(game)); when(gameService.findGame(1L)).thenReturn(Optional.ofNullable(game));
var team = buildTeamMock(1L, "edmonton"); var team = buildTeamMock(1L, "edmonton");
when(teamService.findTeam(1L)).thenReturn(Optional.ofNullable(team)); when(teamService.findTeam(1L)).thenReturn(Optional.ofNullable(team));
assert game != null;
var gameWinner = buildGameWinnerDTO(game, 1L); var gameWinner = buildGameWinnerDTO(game, 1L);
leagueManagerFacade.setGameWinner(gameWinner); leagueManagerFacade.setGameWinner(gameWinner);
verify(gameService).setGameWinner(game, team); verify(gameService).setGameWinner(game, team);
......
...@@ -216,18 +216,16 @@ class UserFacadeImplTest { ...@@ -216,18 +216,16 @@ class UserFacadeImplTest {
return game; return game;
} }
private Team addHomeGames(Team team, Game... games) { private void addHomeGames(Team team, Game... games) {
for (Game game : games) for (Game game : games)
when(game.getHomeTeam()).thenReturn(team); when(game.getHomeTeam()).thenReturn(team);
when(team.getHomeGames()).thenReturn(new HashSet<>(Arrays.asList(games))); when(team.getHomeGames()).thenReturn(new HashSet<>(Arrays.asList(games)));
return team;
} }
private Team addAwayGames(Team team, Game... games) { private void addAwayGames(Team team, Game... games) {
for (Game game : games) for (Game game : games)
when(game.getAwayTeam()).thenReturn(team); when(game.getAwayTeam()).thenReturn(team);
when(team.getAwayGames()).thenReturn(new HashSet<>(Arrays.asList(games))); when(team.getAwayGames()).thenReturn(new HashSet<>(Arrays.asList(games)));
return team;
} }
public PlayerDto buildPlayerDTO(Long id, String name) { public PlayerDto buildPlayerDTO(Long id, String name) {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment