Skip to content

Commit 84b4ef1

Browse files
authored
Merge pull request #156 from DevKor-github/fix/suggestion
건의 조회 오류 수정
2 parents 56e0303 + e53775c commit 84b4ef1

File tree

3 files changed

+23
-18
lines changed

3 files changed

+23
-18
lines changed

src/main/java/devkor/com/teamcback/domain/routes/repository/NodeRepository.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
import devkor.com.teamcback.domain.routes.entity.NodeType;
66
import java.util.List;
77
import org.springframework.data.jpa.repository.JpaRepository;
8+
import org.springframework.data.jpa.repository.Query;
89

910
public interface NodeRepository extends JpaRepository<Node, Long> {
1011
List<Node> findByBuildingAndRoutingAndTypeNot(Building building, boolean routing, NodeType type);
@@ -13,4 +14,6 @@ public interface NodeRepository extends JpaRepository<Node, Long> {
1314
List<Node> findAllByBuildingAndType(Building building, NodeType nodeType);
1415
List<Node> findAllByBuildingAndFloorAndTypeIn(Building building, double floor, List<NodeType> entrance);
1516
List<Node> findByBuildingAndRoutingAndTypeIn(Building building, boolean routing, List<NodeType> type);
17+
@Query("SELECT e FROM Node e ORDER BY FUNCTION('RAND') LIMIT 20")
18+
List<Node> findRandomNodes();
1619
}

src/main/java/devkor/com/teamcback/domain/suggestion/dto/response/GetSuggestionRes.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,6 @@ public GetSuggestionRes(Suggestion suggestion) {
2525
DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm");
2626
this.createdAt = suggestion.getCreatedAt().format(formatter);
2727
this.isSolved = suggestion.isSolved();
28-
this.userId = suggestion.getUser().getUserId();
28+
this.userId = suggestion.getUser() == null ? null : suggestion.getUser().getUserId();
2929
}
3030
}

src/test/java/devkor/com/teamcback/integration/RouteServiceIntegrationTest.java

Lines changed: 19 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -37,16 +37,18 @@ public class RouteServiceIntegrationTest extends BaseMvcTest {
3737
@Order(1)
3838
@DisplayName("기본 길찾기 테스트: 노드 - 노드")
3939
void nodeRouteTest() {
40-
List<Node> nodeList = nodeRepository.findAll();
41-
for(Node start : nodeList) {
42-
for (Node end : nodeList) {
40+
List<Node> nodeList = nodeRepository.findRandomNodes(); // 무작위 노드 20개
41+
for(int i = 0; i < nodeList.size() - 1; i++) {
42+
Node start = nodeList.get(i);
43+
for (int j = i+1; j < nodeList.size(); j++) {
44+
Node end = nodeList.get(j);
4345
try {
44-
System.out.println("start_node_id: " + start.getId() + " end_node_id: " + end.getId());
46+
System.out.println("-------------------------" + "start_node_id: " + start.getId() + " end_node_id: " + end.getId());
4547
routeService.findRoute(LocationType.NODE, start.getId(), null, null, LocationType.NODE, end.getId(), null, null, null);
4648
} catch (AdminException e) {
47-
System.out.println(e.getAdminMessage());
49+
System.out.println("##########################" + e.getAdminMessage());
4850
} catch (GlobalException e) {
49-
System.out.println(e.getResultCode());
51+
System.out.println("##########################" + e.getResultCode());
5052
}
5153
}
5254
}
@@ -57,17 +59,17 @@ void nodeRouteTest() {
5759
@DisplayName("기본 길찾기 테스트: 빌딩 - 빌딩")
5860
void buildingRouteTest() {
5961
List<Building> buildingList = buildingRepository.findAll();
60-
for(Building start : buildingList) {
61-
for (Building end : buildingList) {
62-
if(start.getId() != 0 && end.getId() != 0 && !start.getId().equals(end.getId())) {
63-
try {
64-
System.out.println("start_building_id: " + start.getId() + " end_building_id: " + end.getId());
65-
routeService.findRoute(LocationType.BUILDING, start.getId(), null, null, LocationType.BUILDING, end.getId(), null, null, null);
66-
} catch (AdminException e) {
67-
System.out.println(e.getAdminMessage());
68-
} catch (GlobalException e) {
69-
System.out.println(e.getResultCode());
70-
}
62+
for(int i = 1; i < buildingList.size() - 1; i++) {
63+
Building start = buildingList.get(i);
64+
for (int j = i+1; j < buildingList.size(); j++) {
65+
Building end = buildingList.get(j);
66+
try {
67+
System.out.println("-------------------------" + "start_building_id: " + start.getId() + " end_building_id: " + end.getId());
68+
routeService.findRoute(LocationType.BUILDING, start.getId(), null, null, LocationType.BUILDING, end.getId(), null, null, null);
69+
} catch (AdminException e) {
70+
System.out.println("##########################" + e.getAdminMessage());
71+
} catch (GlobalException e) {
72+
System.out.println("##########################" + e.getResultCode());
7173
}
7274
}
7375
}

0 commit comments

Comments
 (0)