Skip to content

Commit

Permalink
Merge pull request #246 from maeil-mail/236
Browse files Browse the repository at this point in the history
๊ตฌ๋… ๊ธฐ์‚ญ์ œ ์—ฌ๋ถ€ ํŒ๋‹จ ๋กœ์ง์„ ์ถ”๊ฐ€ํ•œ๋‹ค.
  • Loading branch information
le2sky authored Feb 25, 2025
2 parents 1931342 + e57f151 commit 99c39aa
Show file tree
Hide file tree
Showing 2 changed files with 40 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -63,8 +63,11 @@ private long determineSequenceByCategory(QuestionCategory category) {
return 0L;
}

// TODO: ๊ธฐ์‚ญ์ œ ์—ฌ๋ถ€ ํŒ๋‹จ
public void unsubscribe() {
if (deletedAt != null) {
throw new IllegalStateException("์ด๋ฏธ ๊ตฌ๋…์ด ์ทจ์†Œ๋˜์—ˆ์Šต๋‹ˆ๋‹ค.");
}

this.deletedAt = LocalDateTime.now();
}

Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
package maeilmail.subscribe.command.domain;

import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.assertThatThrownBy;

import maeilmail.question.QuestionCategory;
import org.junit.jupiter.api.DisplayName;
import org.junit.jupiter.api.Test;

class SubscribeTest {

@Test
@DisplayName("๊ตฌ๋…์„ ์ทจ์†Œํ•  ์ˆ˜ ์žˆ๋‹ค.")
void unsubscribe() {
Subscribe subscribe = createSubscribe();

subscribe.unsubscribe();

assertThat(subscribe.getDeletedAt()).isNotNull();
}

@Test
@DisplayName("๊ตฌ๋… ์ทจ์†Œ๋œ ์‚ฌ์šฉ์ž๋Š” ๋‹ค์‹œ ๊ตฌ๋…์„ ์ทจ์†Œํ•  ์ˆ˜ ์—†๋‹ค.")
void alreadyUnsubscribed() {
Subscribe subscribe = createSubscribe();
subscribe.unsubscribe();

assertThatThrownBy(subscribe::unsubscribe)
.isInstanceOf(IllegalStateException.class)
.hasMessage("์ด๋ฏธ ๊ตฌ๋…์ด ์ทจ์†Œ๋˜์—ˆ์Šต๋‹ˆ๋‹ค.");
}

private Subscribe createSubscribe() {
return new Subscribe("atom@test.com", QuestionCategory.BACKEND, SubscribeFrequency.DAILY);
}
}

0 comments on commit 99c39aa

Please sign in to comment.