Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump up dependencies versions #413

Merged
merged 3 commits into from
Sep 27, 2022
Merged

Conversation

mhyeon-lee
Copy link
Collaborator

No description provided.

@mhyeon-lee
Copy link
Collaborator Author

@seongahjo @SooKim1110 님 이 PR 에서 라이브러리 버전업 하니
FixtureMonkeyJacksonArbitraryGeneratorTest 의 giveMeJsonNodeReturnsNull 테스트가 깨지는데요
jackson 버전업 하면서 JsonNode 에 대한 null 처리 방식이 바뀐거 같습니다.

then(actual.getValue().isNull()).isTrue(); 여기에서 JsonNode 타입인 getValue() 가 기존엔 jsonNode.isNull() 이었는데 value 자체가 null 이 되네요

jackson 버전 업에 따른 단순 null 처리 스펙 변경인지, 아니면 다른 스펙 변경인데 fixture monkey 의 처리방식과 물려서 결과가 바뀐건지 한번 검토 부탁드릴께요

@seongahjo
Copy link
Contributor

seongahjo commented Sep 27, 2022

@mhyeon-lee
FasterXML/jackson-databind#3214
jackson이 2.13으로 넘어가면서 아무것도 없는 값 ({}) 을 deserialize했을 때 결과NullNode -> null로 변경했습니다.
자세히는 못봤는데 코틀린에서 ?을 지원하기 위해서인걸로 보입니다.

@mhyeon-lee
Copy link
Collaborator Author

@seongahjo jackson 의 스펙이 바뀐거군요
그러면 저 테스트 코드의 assertion 을 null 검사로 변경하면되겠네요

확인 감사합니다.

@mhyeon-lee mhyeon-lee merged commit 3b1fde7 into main Sep 27, 2022
@mhyeon-lee mhyeon-lee deleted the bump-up-dependencies-versions branch September 27, 2022 05:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants