chore: Fix unclosed databases in WorkManager tests #3070
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
Description of changes:
Fixes unclosed databases in the unit tests for cloudwatch logging plugin. This was addressed in version 2.9.0 of the WorkManager test artifact, which added the
closeWorkDatabase()function. (see here)I updated to 2.9.1, which is the latest version that didn't require adjusting our compile SDK.
I replaced the
workmanager-runtime-ktxdependency withworkmanager-runtimebecause as of version 2.9.0 the KTX functionality was promoted to the main artifact, andworkmanager-runtime-ktxis now empty.How did you test these changes?
Verified that no error logs emitted by these tests.
Documentation update required?
General Checklist
fix(storage): message,feat(auth): message,chore(all): message)By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.