Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

fix: "$in needs an array" error from mongo FindByIDs #142

Merged
merged 1 commit into from
May 11, 2022

Conversation

rot1024
Copy link
Member

@rot1024 rot1024 commented May 11, 2022

What I've done

An error "$in needs an array" returns from MongoDB when ids arg of internal/infrastrucutre/mongo.*.FindByIDs methods is nil.

How I tested

@codecov
Copy link

codecov bot commented May 11, 2022

Codecov Report

Merging #142 (4b701f3) into main (35f9db5) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #142      +/-   ##
==========================================
- Coverage   34.61%   34.57%   -0.04%     
==========================================
  Files         349      349              
  Lines       31755    31788      +33     
==========================================
  Hits        10992    10992              
- Misses      19760    19793      +33     
  Partials     1003     1003              
Impacted Files Coverage Δ
internal/infrastructure/mongo/asset.go 24.48% <0.00%> (-0.78%) ⬇️
internal/infrastructure/mongo/dataset.go 0.00% <0.00%> (ø)
internal/infrastructure/mongo/dataset_schema.go 0.00% <0.00%> (ø)
internal/infrastructure/mongo/layer.go 0.00% <0.00%> (ø)
internal/infrastructure/mongo/plugin.go 0.00% <0.00%> (ø)
internal/infrastructure/mongo/project.go 0.00% <0.00%> (ø)
internal/infrastructure/mongo/property.go 0.00% <0.00%> (ø)
internal/infrastructure/mongo/property_schema.go 0.00% <0.00%> (ø)
internal/infrastructure/mongo/scene.go 0.00% <0.00%> (ø)
internal/infrastructure/mongo/tag.go 0.00% <0.00%> (ø)
... and 4 more

@rot1024 rot1024 merged commit 58e1b02 into main May 11, 2022
@rot1024 rot1024 deleted the fix-mongo-findbyids branch May 11, 2022 03:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant