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.

Sign up for GitHub

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

Jump to bottom

Reduce test dependencies & Make running tests easily #10

Merged
merged 23 commits into from
Jan 26, 2020
Merged

Reduce test dependencies & Make running tests easily #10

merged 23 commits into from
Jan 26, 2020

Conversation

civitaspo
Copy link
Owner

This PR must be reviewed after #9 is merged because this branch's root is the branch of #9.
When you check this PR before #9 is merged, please see this file changes: civitaspo:57d951c...civitaspo:38ed8f9

  • Write README about how to run tests.
  • Generate test data when tests running, instead of preparing before tests.
  • Generate testing config in test code, instead of a static one.
  • Support an option to skip tests that require a real aws account.
  • Use the official dynamodb-local image instead of tray/dynamodb-local.

@civitaspo
Use latest gradle (v6.1) with jruby-gradle v1.5.0
1373bc7
@civitaspo
Add dependencies for hacking dependency resolution until all dependen…
64d6ae0 
…cies are updated
@civitaspo
Use scala (2.13.1)
abea545
@civitaspo
Fix deprecation warnings: object JavaConverters in package collection…
18bd79b 
… is deprecated (since 2.13.0): Use `scala.jdk.CollectionConverters` instead
@civitaspo
Fix deprecation warnings: procedure syntax is deprecated: instead, ad…
0532bec 
…d `: Unit =` to explicitly declare `setAuthMethod_Profile`'s return type
@civitaspo
Fix deprecation warnings: method assertThat in class Assert is deprec…
32b6a6f 
…ated: see corresponding Javadoc for more information.
@civitaspo
Use aws-java-sdk-dynamodb 1.11.711
6818483
@civitaspo
Fix deprecation warnings: constructor InstanceProfileCredentialsProvi…
8c597cf 
…der in class InstanceProfileCredentialsProvider is deprecated: see corresponding Javadoc for more information.
@civitaspo
Comment TODO: Fix the deprecation warnings: Be careful as the behavio…
261662e 
…r of the new SDK interface may change significantly.
@civitaspo
Use embulk (0.9.23) & Remove hack dependencies
0a25c8d
@civitaspo
Fix deprecation warnings: method initializeCloseable in class Bootstr…
a56fa76 
…ap is deprecated: see corresponding Javadoc for more information.
@civitaspo
Introduce embulk-test instead of using EmbulkEmbed directly because E…
57d951c 
…mbulkEmbed's interface may changes in the future.
@civitaspo
Test only generatinig credentials not running completely when testing…
cf305d9 
… AwsCredentialsTest
@civitaspo
Throw the exception as to the required env vars when testing
8b4c221
@civitaspo
Use fixed values as possible
22818c7
@civitaspo
Remove yaml dependencies in resources.
beeaca4
@civitaspo
Gather common config as defaultInConfig (AwsCredentialTest)
6f46cd8
@civitaspo
Remove yaml resource dependencies in *OperationTest
f7e4c9f
@civitaspo
Generate data in test code for readability
69a02a6
@civitaspo
Remove test data generators
14688ac
@civitaspo
Use the dynamodb-local official image
859a1bf
@civitaspo
Add the way to skip credentials tests because the tests need a real a…
aeb4819 
…ws account.
@civitaspo
Write how to run tests
38ed8f9
@civitaspo civitaspo requested a review from higarin January 26, 2020 07:21
@higarin higarin merged commit 0877d29 into civitaspo:master Jan 26, 2020
@civitaspo civitaspo mentioned this pull request Mar 8, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@higarin higarin Awaiting requested review from higarin

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@civitaspo @higarin