🎨 Updated exception based on feedback. Added test for void.

chopper/lib/src/chopper_http_exception.dart

@@ -0,0 +1,13 @@
+import 'package:chopper/src/response.dart';
+
+/// An exception thrown when a [Response] is unsuccessful < 200 or > 300.
+class ChopperHttpException implements Exception {
+  ChopperHttpException(this.response);
+
+  final Response response;
+
+  @override
+  String toString() {
+    return 'Could not fetch the response for ${response.base.request}. Status code: ${response.statusCode}, error: ${response.error}';
+  }
+}

chopper/lib/src/response.dart

@@ -1,6 +1,7 @@
 import 'dart:io';
 import 'dart:typed_data';
 
+import 'package:chopper/src/chopper_http_exception.dart';
 import 'package:equatable/equatable.dart' show EquatableMixin;
 import 'package:http/http.dart' as http;
 import 'package:meta/meta.dart';
@@ -87,7 +88,7 @@ base class Response<BodyType> with EquatableMixin {
       if (error is Exception) {
         throw error!;
       }
-      throw HttpException('Could not fetch response $statusCode: $error');
+      throw ChopperHttpException(this);
     }
   }
 

chopper/test/response_test.dart

@@ -1,8 +1,7 @@
-import 'dart:io';
-
+import 'package:chopper/src/chopper_http_exception.dart';
 import 'package:chopper/src/response.dart';
-import 'package:test/test.dart';
 import 'package:http/http.dart' as http;
+import 'package:test/test.dart';
 
 import 'fixtures/error_fixtures.dart';
 
@@ -81,25 +80,35 @@ void main() {
       final base = http.Response('Foobar', 400);
       final response = Response(base, '', error: 'Error occurred');
 
-      expect(() => response.bodyOrThrow, throwsA(isA<HttpException>()));
+      expect(() => response.bodyOrThrow, throwsA(isA<ChopperHttpException>()));
     });
 
     test(
-        'Response is unsuccessful and has no error, [bodyOrThrow throws HttpException]',
+        'Response is unsuccessful and has no error, [bodyOrThrow throws ChopperHttpException]',
         () {
       final base = http.Response('Foobar', 400);
       final response = Response(base, '');
 
-      expect(() => response.bodyOrThrow, throwsA(isA<HttpException>()));
+      expect(() => response.bodyOrThrow, throwsA(isA<ChopperHttpException>()));
     });
 
     test(
-        'Response is successful and has no body, [bodyOrThrow throws HttpException]',
+        'Response is successful and has no body, [bodyOrThrow throws ChopperHttpException]',
+        () {
+      final base = http.Response('Foobar', 200);
+      final Response<String> response = Response(base, null);
+
+      expect(() => response.bodyOrThrow, throwsA(isA<ChopperHttpException>()));
+    });
+
+    test('Response is successful and has void body, [bodyOrThrow returns void]',
         () {
       final base = http.Response('Foobar', 200);
-      final response = Response(base, null);
+      // Ignoring void checks for testing purposes
+      //ignore: void_checks
+      final Response<void> response = Response(base, Null);
 
-      expect(() => response.bodyOrThrow, throwsA(isA<HttpException>()));
+      expect(() => response.bodyOrThrow, returnsNormally);
     });
   });
 }

chopper/test/test_without_response_service.dart

@@ -80,7 +80,7 @@ abstract class HttpTestService extends ChopperService {
   Future putTest(@Path('id') String test, @Body() String data);
 
   @Delete(path: 'delete/{id}', headers: {'foo': 'bar'})
-  Future deleteTest(@Path() String id);
+  Future<void> deleteTest(@Path() String id);
 
   @Patch(path: 'patch/{id}')
   Future patchTest(@Path() String id, @Body() String data);

chopper_generator/test/test_without_response_service.dart

@@ -80,7 +80,7 @@ abstract class HttpTestService extends ChopperService {
   Future putTest(@Path('id') String test, @Body() String data);
 
   @Delete(path: 'delete/{id}', headers: {'foo': 'bar'})
-  Future deleteTest(@Path() String id);
+  Future<void> deleteTest(@Path() String id);
 
   @Patch(path: 'patch/{id}')
   Future patchTest(@Path() String id, @Body() String data);