Skip to content

Commit 9e2b3b1

Browse files
committed
refactor(course): add debug log level, logging Course
1 parent b9cc4e0 commit 9e2b3b1

File tree

4 files changed

+25
-19
lines changed

4 files changed

+25
-19
lines changed

src/main/java/lms/domain/Course.java

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -9,20 +9,23 @@ public class Course {
99

1010
private Long creatorId;
1111

12-
private LocalDateTime createdDate;
12+
private LocalDateTime createdAt;
1313

14-
private LocalDateTime updatedDate;
14+
private LocalDateTime updatedAt;
15+
16+
public Course() {
17+
}
1518

1619
public Course(String title, Long creatorId) {
1720
this(0L, title, creatorId, LocalDateTime.now(), null);
1821
}
1922

20-
public Course(Long id, String title, Long creatorId, LocalDateTime createdDate, LocalDateTime updatedDate) {
23+
public Course(Long id, String title, Long creatorId, LocalDateTime createdAt, LocalDateTime updatedAt) {
2124
this.id = id;
2225
this.title = title;
2326
this.creatorId = creatorId;
24-
this.createdDate = createdDate;
25-
this.updatedDate = updatedDate;
27+
this.createdAt = createdAt;
28+
this.updatedAt = updatedAt;
2629
}
2730

2831
public String getTitle() {
@@ -33,8 +36,8 @@ public Long getCreatorId() {
3336
return creatorId;
3437
}
3538

36-
public LocalDateTime getCreatedDate() {
37-
return createdDate;
39+
public LocalDateTime getCreatedAt() {
40+
return createdAt;
3841
}
3942

4043
@Override
@@ -43,8 +46,8 @@ public String toString() {
4346
"id=" + id +
4447
", title='" + title + '\'' +
4548
", creatorId=" + creatorId +
46-
", createdDate=" + createdDate +
47-
", updatedDate=" + updatedDate +
49+
", createdAt=" + createdAt +
50+
", updatedAt=" + updatedAt +
4851
'}';
4952
}
5053
}

src/main/java/lms/infrastructure/JdbcCourseRepository.java

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3,38 +3,34 @@
33
import lms.domain.Course;
44
import lms.domain.CourseRepository;
55
import org.springframework.beans.factory.annotation.Autowired;
6-
import org.springframework.jdbc.core.JdbcTemplate;
6+
import org.springframework.jdbc.core.JdbcOperations;
77
import org.springframework.jdbc.core.RowMapper;
88
import org.springframework.stereotype.Repository;
99

10-
import java.sql.ResultSet;
11-
import java.sql.SQLException;
1210
import java.sql.Timestamp;
1311
import java.time.LocalDateTime;
1412

1513
@Repository("courseRepository")
1614
public class JdbcCourseRepository implements CourseRepository {
1715
@Autowired
18-
private JdbcTemplate jdbcTemplate;
16+
private JdbcOperations jdbcTemplate;
1917

2018
@Override
2119
public int save(Course course) {
2220
String sql = "insert into course (title, creator_id, created_at) values(?, ?, ?)";
23-
return jdbcTemplate.update(
24-
sql,
25-
course.getTitle(), course.getCreatorId(), course.getCreatedDate());
21+
return jdbcTemplate.update(sql, course.getTitle(), course.getCreatorId(), course.getCreatedAt());
2622
}
2723

2824
@Override
2925
public Course findById(Long id) {
3026
String sql = "select id, title, creator_id, created_at, updated_at from course where id = ?";
31-
RowMapper<Course> courseRowMapper = (rs, rowNum) -> new Course(
27+
RowMapper<Course> rowMapper = (rs, rowNum) -> new Course(
3228
rs.getLong(1),
3329
rs.getString(2),
3430
rs.getLong(3),
3531
toLocalDateTime(rs.getTimestamp(4)),
3632
toLocalDateTime(rs.getTimestamp(5)));
37-
return jdbcTemplate.queryForObject(sql, courseRowMapper, id);
33+
return jdbcTemplate.queryForObject(sql, rowMapper, id);
3834
}
3935

4036
private LocalDateTime toLocalDateTime(Timestamp timestamp) {

src/main/resources/logback.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,8 @@
99

1010
<logger name="qna" level="DEBUG"/>
1111

12+
<logger name="lms" level="DEBUG"/>
13+
1214
<root level="INFO">
1315
<appender-ref ref="STDOUT"/>
1416
</root>

src/test/java/lms/infrastruecture/CourseRepositoryTest.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,15 @@
55
import lms.domain.Course;
66
import lms.domain.CourseRepository;
77
import org.junit.jupiter.api.Test;
8+
import org.slf4j.Logger;
9+
import org.slf4j.LoggerFactory;
810
import org.springframework.beans.factory.annotation.Autowired;
911
import org.springframework.boot.test.context.SpringBootTest;
1012

1113
@SpringBootTest
1214
public class CourseRepositoryTest {
15+
private static final Logger LOGGER = LoggerFactory.getLogger(CourseRepositoryTest.class);
16+
1317
@Autowired
1418
private CourseRepository courseRepository;
1519

@@ -19,6 +23,7 @@ void crud() {
1923
int count = courseRepository.save(course);
2024
assertThat(count).isEqualTo(1);
2125
Course savedCourse = courseRepository.findById(1L);
22-
assertThat(course.getTitle()).isEqualTo(course.getTitle());
26+
assertThat(course.getTitle()).isEqualTo(savedCourse.getTitle());
27+
LOGGER.debug("Course: {}", savedCourse);
2328
}
2429
}

0 commit comments

Comments
 (0)