test: Refactor and tests Sourcemap command

bin/raygun_cli.dart

@@ -23,12 +23,12 @@ ArgParser buildParser() {
       help: 'Print the tool version.',
     )
     ..addCommand(
-      kSourcemapCommand,
-      buildParserSourcemap(),
+      sourcemapCommand.name,
+      sourcemapCommand.buildParser(),
     )
     ..addCommand(
       symbolsCommand.name,
-      symbolsCommand.buildParserSymbols(),
+      symbolsCommand.buildParser(),
     )
     ..addCommand(
       kDeploymentsCommand,
@@ -71,8 +71,8 @@ void main(List<String> arguments) {
       verbose = true;
     }
 
-    if (results.command?.name == kSourcemapCommand) {
-      parseSourcemapCommand(results.command!, verbose);
+    if (results.command?.name == sourcemapCommand.name) {
+      sourcemapCommand.execute(results.command!, verbose);
       return;
     }
 

lib/src/core/raygun_command.dart

@@ -7,11 +7,5 @@ abstract class RaygunCommand {
 
   void execute(ArgResults command, bool verbose);
 
-  Future<bool> run({
-    required ArgResults command,
-    required String appId,
-    required String token,
-  });
-
-  ArgParser buildParserSymbols();
+  ArgParser buildParser();
 }

lib/src/sourcemap/flutter/sourcemap_flutter.dart

@@ -1,18 +1,18 @@
 import 'dart:io';
 
 import 'package:raygun_cli/src/config_props.dart';
-import 'package:raygun_cli/src/sourcemap/sourcemap_api.dart';
 import 'package:raygun_cli/src/sourcemap/sourcemap_base.dart';
 
 /// Uploads a Flutter web sourcemap file to Raygun.
 class SourcemapFlutter extends SourcemapBase {
   SourcemapFlutter({
     required super.command,
     required super.verbose,
+    required super.api,
   });
 
   @override
-  Future<void> upload() async {
+  Future<bool> upload() async {
     if (command.option('uri') == null && command.option('base-uri') == null) {
       print('Provide either "uri" or "base-uri"');
       exit(2);
@@ -28,16 +28,11 @@ class SourcemapFlutter extends SourcemapBase {
       print('input-map: $path');
       print('uri: $uri');
     }
-    final out = await uploadSourcemap(
+    return await api.uploadSourcemap(
       appId: appId,
       token: token,
       path: path,
       uri: uri,
     );
-    if (out) {
-      exit(0);
-    } else {
-      exit(2);
-    }
   }
 }

lib/src/sourcemap/node/sourcemap_node.dart

@@ -1,5 +1,3 @@
-import 'dart:io';
-
 import 'package:raygun_cli/src/sourcemap/sourcemap_base.dart';
 
 /// Uploads a Node app sourcemap file to Raygun.
@@ -8,16 +6,17 @@ class SourcemapNode extends SourcemapBase {
   SourcemapNode({
     required super.command,
     required super.verbose,
+    required super.api,
   });
 
   @override
-  Future<void> upload() {
+  Future<bool> upload() async {
     // final baseUri = command.option('base-uri')!;
     // final src = command.option('src')!;
 
     // TODO: search for all map files in source folder and upload them.
 
-    print('Comming soon!');
-    exit(2);
+    print('Coming soon!');
+    return false;
   }
 }

lib/src/sourcemap/sourcemap_api.dart

@@ -1,50 +1,44 @@
 import 'dart:io';
 import 'package:http/http.dart' as http;
+import 'package:raygun_cli/src/core/raygun_api.dart';
 
-/// Uploads a sourcemap file to Raygun.
-/// returns true if the upload was successful.
-Future<bool> uploadSourcemap({
-  required String appId,
-  required String token,
-  required String path,
-  required String uri,
-}) async {
-  final file = File(path);
-  if (!file.existsSync()) {
-    print('$path: file not found!');
-    return false;
-  }
-  print('Uploading: $path');
+class SourcemapApi {
+  const SourcemapApi({
+    required this.httpClient,
+  });
 
-  // curl -X PUT 'https://api.raygun.com/v3/applications/{your-application-identifier}/source-maps' \
-  // -H 'Authorization: Bearer YOUR_PERSONAL_ACCESS_TOKEN' \
-  // -H 'Content-Type: multipart/form-data' \
-  // -F 'file=@path_to_your_source_map_file.map' \
-  // -F 'uri=your_source_map_uri'
+  /// Creates a new instance of [SourcemapApi] with the provided [httpClient].
+  SourcemapApi.create() : httpClient = http.Client();
 
-  final url = 'https://api.raygun.com/v3/applications/$appId/source-maps';
-  final request = http.MultipartRequest('PUT', Uri.parse(url));
-  request.headers.addAll({
-    'Authorization': 'Bearer $token',
-    'Content-Type': 'multipart/form-data',
-  });
-  request.files.add(
-    http.MultipartFile(
-      'file',
-      file.readAsBytes().asStream(),
-      file.lengthSync(),
-      filename: path.split("/").last,
-    ),
-  );
-  request.fields.addAll({
-    'uri': uri,
-  });
-  final res = await request.send();
-  if (res.statusCode == 200) {
-    print('File uploaded succesfully!');
-    return true;
-  } else {
-    print('Error uploading file. Response code: ${res.statusCode}');
-    return false;
+  final http.Client httpClient;
+
+  /// Uploads a sourcemap file to Raygun.
+  /// returns true if the upload was successful.
+  Future<bool> uploadSourcemap({
+    required String appId,
+    required String token,
+    required String path,
+    required String uri,
+  }) async {
+    final file = File(path);
+    if (!file.existsSync()) {
+      print('$path: file not found!');
+      return false;
+    }
+    print('Uploading: $path');
+
+    final request = RaygunMultipartRequestBuilder(
+      'https://api.raygun.com/v3/applications/$appId/source-maps',
+      'PUT',
+    ).addBearerToken(token).addField('uri', uri).addFile('file', path).build();
+
+    final response = await httpClient.send(request);
+    if (response.statusCode == 200) {
+      print('File uploaded successfully!');
+      return true;
+    } else {
+      print('Error uploading file. Response code: ${response.statusCode}');
+      return false;
+    }
   }
 }

lib/src/sourcemap/sourcemap_base.dart

@@ -1,10 +1,12 @@
 import 'package:args/args.dart';
+import 'package:raygun_cli/src/sourcemap/sourcemap_api.dart';
 
 /// Base class for JavaScript sourcemap uploaders
 abstract class SourcemapBase {
   SourcemapBase({
     required this.command,
     required this.verbose,
+    required this.api,
   });
 
   /// Command line arguments
@@ -13,6 +15,8 @@ abstract class SourcemapBase {
   /// Print verbose output
   final bool verbose;
 
+  final SourcemapApi api;
+
   /// Uploads the sourcemap
-  Future<void> upload();
+  Future<bool> upload();
 }