Job Failed #317517

[JuyeoungJun]

In GitLab by @gm2202981 on Jul 13, 2021, 11:40

개요

• Job #317517 failed for f60683e3c60fb2500fb2b463313052d396464d5c:

버그 내용

CronLogRepositoryImplTest > save FAILED
    java.lang.IllegalStateException at DefaultCacheAwareContextLoaderDelegate.java:132
        Caused by: org.springframework.beans.factory.UnsatisfiedDependencyException at ConstructorResolver.java:800
            Caused by: org.springframework.beans.factory.UnsatisfiedDependencyException at ConstructorResolver.java:800
                Caused by: org.springframework.boot.context.properties.ConfigurationPropertiesBindException at ConfigurationPropertiesBindingPostProcessor.java:92
                    Caused by: org.springframework.boot.context.properties.bind.BindException at Binder.java:384
                        Caused by: org.springframework.beans.factory.BeanCreationException at AutowiredAnnotationBeanPostProcessor.java:405
                            Caused by: java.lang.IllegalArgumentException at PropertyPlaceholderHelper.java:180

• 버그 원인 : 로컬 테스트 환경과 Gitlab CI상 테스트 환경의 차이
• 발생 환경 : G클라우드 테스트 서버

버그 재현 코드

@RunWith(SpringRunner.class)
@SpringBootTest
@ActiveProfiles("dev_env")
public class CronLogRepositoryImplTest {

    @Autowired
    CronLogRepository cronLogRepository;

    @Test
    public void save() {
        // Given
        CronLog cronLog = CronLog.builder()
            .logTime(Instant.now())
            .log("test")
            .cronProcess("test")
            .build();
        // When
        CronLog savedLog = cronLogRepository.save(cronLog);
        // Then
        assertThat(cronLog).isEqualTo(savedLog);
    }
}

해결 방안

@RunWith(SpringRunner.class)
@SpringBootTest
@ActiveProfiles("prod")
public class CronLogRepositoryImplTest {

    @Autowired
    CronLogRepository cronLogRepository;

    @Test
    public void save() {
        // Given
        CronLog cronLog = CronLog.builder()
            .logTime(Instant.now())
            .log("test")
            .cronProcess("test")
            .build();
        // When
        CronLog savedLog = cronLogRepository.save(cronLog);
        // Then
        assertThat(cronLog).isEqualTo(savedLog);
    }
}

• 코드상의 ActiveProfile 변경

  추후 고려사항

• 로컬 개발환경과 CI 환경 상의 테스트 환경 (IP 주소) 차이를 테스트 코드 상에서 동적으로 처리할 수 있는 방안 탐색
• 테스트 통과를 위해 비즈니스 로직을 분리한 KafkaListner 웹훅 서비스는 해당 프레임워크에게 테스트 책임이 존재한다고 판단하여 일단 테스트 커버리지에서 제외했으나, 추후 재검토 필요.

[JuyeoungJun]

In GitLab by @gm2202981 on Jul 13, 2021, 11:44

changed the description

[JuyeoungJun]

In GitLab by @gm2202981 on Jul 13, 2021, 11:45

changed the description

[JuyeoungJun]

In GitLab by @gm2202981 on Jul 13, 2021, 12:03

changed the description

[JuyeoungJun]

In GitLab by @gm2202981 on Jul 13, 2021, 12:56

mentioned in merge request !9