feat(storage): hide operation APIs that don't take any effect on upload

Author: HuiSF

packages/amplify_core/lib/src/types/storage/base/storage_controllable_operation.dart

@@ -12,14 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-abstract class StorageControllableOperation {
-  /// {@template amplify_core.storage.controllable_operation.cancel}
-  /// Cancels the operation.
-  ///
-  /// A cancelled operation cannot be resumed.
-  /// {@endtemplate}
-  Future<void> cancel();
-
+abstract class StorageResumableOperation {
   /// {@template amplify_core.storage.controllable_operation.pause}
   /// Pauses the operation that is in progress.
   /// {@endtemplate}
@@ -30,3 +23,12 @@ abstract class StorageControllableOperation {
   /// {@endtemplate}
   Future<void> resume();
 }
+
+abstract class StorageCancelableOperation {
+  /// {@template amplify_core.storage.controllable_operation.cancel}
+  /// Cancels the operation.
+  ///
+  /// A cancelled operation cannot be resumed.
+  /// {@endtemplate}
+  Future<void> cancel();
+}

packages/amplify_core/lib/src/types/storage/download_data_operation.dart

@@ -24,7 +24,7 @@ abstract class StorageDownloadDataOperation<
     extends StorageOperation<Request, Result>
 // TODO(HuiSF): re-enable controllable APIs when SmithyOperation.cancel
 // can cancel underlying http request.
-/* implements StorageControllableOperation */ {
+/* implements StorageResumableOperation, StorageCancelableOperation */ {
   /// {@macro amplify_core.storage.download_data_operation}
   StorageDownloadDataOperation({
     required super.request,

packages/amplify_core/lib/src/types/storage/download_file_operation.dart

@@ -24,7 +24,7 @@ abstract class StorageDownloadFileOperation<
     extends StorageOperation<Request, Result>
 // TODO(HuiSF): re-enable controllable APIs when SmithyOperation.cancel
 // can cancel underlying http request.
-/* implements StorageControllableOperation */ {
+/* implements StorageResumableOperation, StorageCancelableOperation */ {
   /// {@macro amplify_core.storage.download_file_operation}
   StorageDownloadFileOperation({
     required super.request,

packages/amplify_core/lib/src/types/storage/upload_data_operation.dart

@@ -23,7 +23,7 @@ abstract class StorageUploadDataOperation<
         Request extends StorageUploadDataRequest,
         Result extends StorageUploadDataResult>
     extends StorageOperation<Request, Result>
-    implements StorageControllableOperation {
+    implements StorageCancelableOperation {
   /// {@macro amplify_core.storage.upload_data_operation}
   StorageUploadDataOperation({
     required super.request,

packages/amplify_core/lib/src/types/storage/upload_file_operation.dart

@@ -23,7 +23,7 @@ abstract class StorageUploadFileOperation<
         Request extends StorageUploadFileRequest,
         Result extends StorageUploadFileResult>
     extends StorageOperation<Request, Result>
-    implements StorageControllableOperation {
+    implements StorageResumableOperation, StorageCancelableOperation {
   /// {@macro amplify_core.storage.upload_file_operation}
   StorageUploadFileOperation({
     required super.request,

packages/storage/amplify_storage_s3_dart/lib/src/amplify_storage_s3_dart_impl.dart

@@ -331,8 +331,6 @@ class AmplifyStorageS3Dart extends StoragePluginInterface<
       result: uploadTask.result.then(
         (uploadedItem) => S3UploadDataResult(uploadedItem: uploadedItem),
       ),
-      resume: uploadTask.resume,
-      pause: uploadTask.pause,
       cancel: uploadTask.cancel,
     );
   }

packages/storage/amplify_storage_s3_dart/lib/src/model/s3_upload_data_operation.dart

@@ -24,25 +24,11 @@ class S3UploadDataOperation extends StorageUploadDataOperation<
   S3UploadDataOperation({
     required super.request,
     required super.result,
-    required Future<void> Function() resume,
-    required Future<void> Function() pause,
     required Future<void> Function() cancel,
-  })  : _resume = resume,
-        _pause = pause,
-        _cancel = cancel;
+  }) : _cancel = cancel;
 
-  final Future<void> Function() _resume;
-  final Future<void> Function() _pause;
   final Future<void> Function() _cancel;
 
-  /// Resume takes no effect for a [S3UploadDataOperation].
-  @override
-  Future<void> resume() => _resume();
-
-  /// Pause takes no effect for a [S3UploadDataOperation] .
-  @override
-  Future<void> pause() => _pause();
-
   @override
   Future<void> cancel() => _cancel();
 }