Skip to content

Commit

Permalink
feat: 출결번호가 네 자리 수가 되도록 String 타입으로 수정 (#858)
Browse files Browse the repository at this point in the history
* feat: 스터디회차의 출석번호를 String으로 변경

* feat: 출석번호 생성기 인터페이스 및 구현체 수정
  • Loading branch information
uwoobeat authored Jan 30, 2025
1 parent 621c9a2 commit dd3d474
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 10 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
package com.gdschongik.gdsc.domain.studyv2.domain;

public interface AttendanceNumberGenerator {
int generate();
String generate();
}
Original file line number Diff line number Diff line change
Expand Up @@ -13,10 +13,10 @@ public class RandomAttendanceNumberGenerator implements AttendanceNumberGenerato

@Override
@SneakyThrows
public int generate() {
return SecureRandom.getInstanceStrong()
public String generate() {
return String.valueOf(SecureRandom.getInstanceStrong()
.ints(MIN_ORIGIN, MAX_BOUND)
.findFirst()
.orElseThrow();
.orElseThrow());
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public class StudySessionV2 extends BaseEntity {
// 수업 관련 필드

@Comment("수업 출석 번호")
private Integer lessonAttendanceNumber;
private String lessonAttendanceNumber;

@Embedded
@AttributeOverride(name = "startDate", column = @Column(name = "lesson_start_at"))
Expand Down Expand Up @@ -76,7 +76,7 @@ private StudySessionV2(
Integer position,
String title,
String description,
Integer lessonAttendanceNumber,
String lessonAttendanceNumber,
Period lessonPeriod,
String assignmentDescriptionLink,
Period assignmentPeriod,
Expand All @@ -92,7 +92,7 @@ private StudySessionV2(
studyV2.getStudySessions().add(this);
}

public static void createEmpty(Integer position, Integer lessonAttendanceNumber, StudyV2 studyV2) {
public static void createEmpty(Integer position, String lessonAttendanceNumber, StudyV2 studyV2) {
StudySessionV2.builder()
.position(position)
.lessonAttendanceNumber(lessonAttendanceNumber)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ class StudyFactoryTest {

static class FixedAttendanceNumberGenerator implements AttendanceNumberGenerator {
@Override
public int generate() {
return 1000;
public String generate() {
return "0000";
}
}

Expand Down Expand Up @@ -104,6 +104,6 @@ public int generate() {
// then
assertThat(study.getStudySessions())
.extracting(StudySessionV2::getLessonAttendanceNumber)
.containsOnly(1000);
.containsOnly("0000");
}
}

0 comments on commit dd3d474

Please sign in to comment.