scriptoriumadmin/skillhub
1
0
Fork 0
mirror of https://github.com/iflytek/skillhub.git synced 2026-10-15 04:18:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

test(review): tighten persistence assertions

Browse source 
Signed-off-by: XiaoSeS <87064762+XiaoSeS@users.noreply.github.com>
This commit is contained in:
XiaoSeS 2026-09-01 19:51:28 +08:00
parent e7771cf208
commit 22b3a719c9
2 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
server/skillhub-app/src/test/java/com/iflytek/skillhub/controller/admin/SkillReviewModerationFlowIntegrationTest.java
View file

@ -77,6 +77,11 @@ class SkillReviewModerationFlowIntegrationTest {
.andExpect(status().isOk())
.andExpect(jsonPath("$.data.status").value("HIDDEN"));
SkillRating hidden = ratingRepository.findById(review.getId()).orElseThrow();
assertThat(hidden.getReviewStatus()).isEqualTo(SkillReviewStatus.HIDDEN);
assertThat(hidden.getModeratedBy()).isEqualTo(ADMIN_ID);
assertThat(hidden.getModerationReason()).isEqualTo("off topic");
mockMvc.perform(post("/api/v1/admin/skill-reviews/" + review.getId() + "/restore")
.with(authentication(adminAuth()))
.with(csrf()))
@ -89,6 +94,8 @@ class SkillReviewModerationFlowIntegrationTest {
List<String> actions = auditLogRepository.findAll().stream()
.filter(log -> ADMIN_ID.equals(log.getActorUserId()))
.filter(log -> review.getId().equals(log.getTargetId()))
.filter(log -> "SKILL_REVIEW".equals(log.getTargetType()))
.map(AuditLog::getAction)
.toList();
assertThat(actions).contains("SKILL_REVIEW_HIDE", "SKILL_REVIEW_RESTORE");

3
server/skillhub-app/src/test/java/com/iflytek/skillhub/repository/SkillRatingOptimisticLockingTest.java
View file

@ -8,6 +8,7 @@ import com.iflytek.skillhub.domain.social.SkillReviewStatus;
import jakarta.persistence.EntityManager;
import jakarta.persistence.EntityManagerFactory;
import jakarta.persistence.OptimisticLockException;
import org.hibernate.exception.ConstraintViolationException;
import org.junit.jupiter.api.Test;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.test.autoconfigure.orm.jpa.DataJpaTest;
@ -78,7 +79,7 @@ class SkillRatingOptimisticLockingTest {
secondManager.persist(duplicate);
secondManager.flush();
secondManager.getTransaction().commit();
}).isInstanceOf(RuntimeException.class);
}).satisfies(error -> assertThat(hasCause(error, ConstraintViolationException.class)).isTrue());
EntityManager verifier = entityManagerFactory.createEntityManager();
try {