search

awslabs / dynein

DynamoDB CLI written in Rust.
https://github.com/awslabs/dynein
Apache License 2.0
360 stars 37 forks source link

test: add integration tests for import #230

Closed StoneDot closed 4 months ago

StoneDot commented 4 months ago

Issue #, if available:

143

Description of changes: This pull request adds integration tests for import command.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

ryota-sakamoto commented 4 months ago

Would you add test for --enable-set-inference?

https://github.com/awslabs/dynein/blob/f24d513d02370f4c9c38ad0d95a413e6e41ae8f8/src/cmd.rs#L386-L388

StoneDot commented 4 months ago

@ryota-sakamoto Thank you for pointing it out. I have added a test for --enable-set-inference. Could you please check it?