aws-amplify / amplify-swift

A declarative library for application development using cloud services.
Apache License 2.0
456 stars 198 forks source link

chore(storage): resolve swiftformat errors and warnings #3852

Closed phantumcode closed 1 month ago

phantumcode commented 2 months ago

Issue \

Description

Resolving SwiftFormat errors and warnings for the Storage plugin.

General Checklist

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 75.68882% with 150 lines in your changes missing coverage. Please review.

Project coverage is 68.54%. Comparing base (ca6c65a) to head (2326834). Report is 11 commits behind head on chore/resolve_swiftformat.

Files with missing lines Patch % Lines
...lugin/AWSS3StoragePlugin+AsyncClientBehavior.swift 53.73% 31 Missing :warning:
.../AWSS3StorageService+MultiPartUploadBehavior.swift 0.00% 18 Missing :warning:
...Plugin/Dependency/UploadPartInput+presignURL.swift 0.00% 17 Missing :warning:
...Storage/AWSS3StorageService+DownloadBehavior.swift 0.00% 17 Missing :warning:
...gin/Support/Internal/StorageTransferResponse.swift 12.50% 14 Missing :warning:
...oragePlugin/Configuration/AWSS3PluginOptions.swift 0.00% 6 Missing :warning:
...port/Internal/DefaultStorageTransferDatabase.swift 80.64% 6 Missing :warning:
...agePlugin/Support/Internal/StorageUploadPart.swift 71.42% 6 Missing :warning:
...ugin/Support/Internal/StorageUploadPartEvent.swift 33.33% 6 Missing :warning:
...ePlugin/Support/Internal/StorageTransferType.swift 50.00% 5 Missing :warning:
... and 11 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## chore/resolve_swiftformat #3852 +/- ## ============================================================= + Coverage 68.51% 68.54% +0.03% ============================================================= Files 1080 1023 -57 Lines 37608 35703 -1905 ============================================================= - Hits 25766 24474 -1292 + Misses 11842 11229 -613 ``` | [Flag](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | Coverage Δ | | |---|---|---| | [API_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `68.99% <ø> (+0.03%)` | :arrow_up: | | [AWSPluginsCore](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `?` | | | [Amplify](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `47.56% <ø> (ø)` | | | [Analytics_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `84.52% <ø> (ø)` | | | [Auth_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `79.69% <ø> (+0.01%)` | :arrow_up: | | [DataStore_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `81.30% <ø> (+0.08%)` | :arrow_up: | | [Geo_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `72.00% <ø> (ø)` | | | [Logging_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `62.95% <ø> (ø)` | | | [Predictions_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `37.32% <ø> (ø)` | | | [PushNotifications_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `86.21% <ø> (ø)` | | | [Storage_plugin_unit_test](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `76.64% <75.68%> (-0.17%)` | :arrow_down: | | [unit_tests](https://app.codecov.io/gh/aws-amplify/amplify-swift/pull/3852/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify) | `68.54% <75.68%> (+0.03%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=aws-amplify#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.