style: add ignore_for_file lint directives

- Added to avoid linting issues
- No functional code changes

Author: fulleni

lib/src/middleware/error_handler.dart

@@ -1,3 +1,6 @@
+//
+// ignore_for_file: avoid_catches_without_on_clauses
+
 import 'dart:io';
 
 import 'package:dart_frog/dart_frog.dart';

routes/api/v1/countries/[isoCode].dart

@@ -1,3 +1,6 @@
+//
+// ignore_for_file: avoid_catches_without_on_clauses
+
 import 'dart:async';
 import 'dart:io';
 

routes/api/v1/countries/index.dart

@@ -1,3 +1,6 @@
+//
+// ignore_for_file: avoid_catches_without_on_clauses
+
 import 'dart:async';
 import 'dart:io';
 

routes/api/v1/index.dart

@@ -6,10 +6,10 @@ import 'package:dart_frog/dart_frog.dart';
 ///
 /// Returns a simple welcome message indicating the API is running.
 Response onRequest(RequestContext context) {
-  // You could potentially add more information here 
+  // You could potentially add more information here
   // like links to documentation.
   return Response.json(
     statusCode: HttpStatus.ok, // 200
-    body: {'message': 'Welcome to the Headlines Toolkit API V1!'}, 
+    body: {'message': 'Welcome to the Headlines Toolkit API V1!'},
   );
 }

test/routes/api/v1/countries/[isoCode]_test.dart

@@ -1,4 +1,3 @@
-import 'dart:convert';
 import 'dart:io';
 
 import 'package:dart_frog/dart_frog.dart';
@@ -65,7 +64,7 @@ void main() {
     });
 
     test('lets CountryNotFound bubble up (handled globally)', () async {
-      final exception = CountryNotFound('Country $isoCode not found');
+      const exception = CountryNotFound('Country $isoCode not found');
       when(() => mockClient.fetchCountry(isoCode)).thenThrow(exception);
 
       expect(
@@ -76,7 +75,7 @@ void main() {
     });
 
     test('lets CountryFetchFailure bubble up (handled globally)', () async {
-      final exception = CountryFetchFailure('Network error');
+      const exception = CountryFetchFailure('Network error');
       when(() => mockClient.fetchCountry(isoCode)).thenThrow(exception);
 
       expect(
@@ -123,13 +122,15 @@ void main() {
     });
 
     test('returns 400 Bad Request for invalid JSON body', () async {
-      when(() => request.json()).thenThrow(FormatException('Bad JSON'));
+      when(() => request.json()).thenThrow(const FormatException('Bad JSON'));
 
       final response = await route.onRequest(context, isoCode);
 
       expect(response.statusCode, equals(HttpStatus.badRequest));
-      expect(await response.body(),
-          equals('Invalid JSON format in request body.'));
+      expect(
+        await response.body(),
+        equals('Invalid JSON format in request body.'),
+      );
       verifyNever(() => mockClient.updateCountry(any()));
     });
 
@@ -161,14 +162,14 @@ void main() {
         await response.json(),
         equals({
           'error': 'ISO code in request body ("XX") does not match ISO code '
-              'in URL path ("US").'
+              'in URL path ("US").',
         }),
       );
       verifyNever(() => mockClient.updateCountry(any()));
     });
 
     test('lets CountryNotFound bubble up (handled globally)', () async {
-      final exception = CountryNotFound('Cannot update non-existent country');
+      const exception = CountryNotFound('Cannot update non-existent country');
       // Ensure the specific exception is thrown by the mock
       when(() => mockClient.updateCountry(any())).thenThrow(exception);
 
@@ -180,7 +181,7 @@ void main() {
     });
 
     test('lets CountryUpdateFailure bubble up (handled globally)', () async {
-      final exception = CountryUpdateFailure('DB conflict during update');
+      const exception = CountryUpdateFailure('DB conflict during update');
       // Ensure the specific exception is thrown by the mock
       when(() => mockClient.updateCountry(any())).thenThrow(exception);
 
@@ -207,7 +208,7 @@ void main() {
     });
 
     test('lets CountryNotFound bubble up (handled globally)', () async {
-      final exception = CountryNotFound('Cannot delete non-existent country');
+      const exception = CountryNotFound('Cannot delete non-existent country');
       when(() => mockClient.deleteCountry(isoCode)).thenThrow(exception);
 
       expect(
@@ -218,7 +219,7 @@ void main() {
     });
 
     test('lets CountryDeleteFailure bubble up (handled globally)', () async {
-      final exception = CountryDeleteFailure('Permission error');
+      const exception = CountryDeleteFailure('Permission error');
       when(() => mockClient.deleteCountry(isoCode)).thenThrow(exception);
 
       expect(

test/routes/api/v1/countries/index.dart

@@ -1,4 +1,3 @@
-import 'dart:convert';
 import 'dart:io';
 
 import 'package:dart_frog/dart_frog.dart';
@@ -12,7 +11,8 @@ import '../../../../../routes/api/v1/countries/index.dart' as route;
 // --- Mocks ---
 class _MockRequestContext extends Mock implements RequestContext {}
 
-class _MockHttpRequest extends Mock implements Request {} // Use dart_frog Request
+class _MockHttpRequest extends Mock
+    implements Request {} // Use dart_frog Request
 
 class _MockHtCountriesClient extends Mock implements HtCountriesClient {}
 // --- End Mocks ---
@@ -98,7 +98,7 @@ void main() {
       ).called(1);
     });
 
-     test('uses max limit when limit parameter exceeds max', () async {
+    test('uses max limit when limit parameter exceeds max', () async {
       final countries = createSampleCountries(5);
       // Limit is 150, should be clamped to 100 (defined in route)
       uri = Uri.parse('http://localhost/api/v1/countries?limit=150');
@@ -148,20 +148,25 @@ void main() {
         equals('Invalid query parameter: "limit" must be an integer.'),
       );
       verifyNever(
-        () => mockClient.fetchCountries(limit: any(named: 'limit'), startAfterId: any(named: 'startAfterId')),
+        () => mockClient.fetchCountries(
+          limit: any(named: 'limit'),
+          startAfterId: any(named: 'startAfterId'),
+        ),
       );
     });
 
-    // Add test for client throwing CountryFetchFailure (handled by global handler)
+    // Add test for client throwing CountryFetchFailure
+    //(handled by global handler)
     test('lets CountryFetchFailure bubble up (handled globally)', () async {
       uri = Uri.parse('http://localhost/api/v1/countries?limit=10');
       when(() => request.uri).thenReturn(uri);
-      final exception = CountryFetchFailure('DB error');
+      const exception = CountryFetchFailure('DB error');
       when(
         () => mockClient.fetchCountries(limit: 10, startAfterId: null),
       ).thenThrow(exception);
 
-      // Expect the specific exception to be thrown, letting the global handler catch it
+      // Expect the specific exception to be thrown, letting the global handler
+      // catch it
       expect(
         () => route.onRequest(context),
         throwsA(isA<CountryFetchFailure>()),
@@ -183,7 +188,6 @@ void main() {
     // Remove 'id' as it's generated by client/constructor usually
     final requestJson = Map<String, dynamic>.from(newCountryJson)..remove('id');
 
-
     setUp(() {
       // Set request method for all POST tests in this group
       when(() => request.method).thenReturn(HttpMethod.post);
@@ -215,7 +219,7 @@ void main() {
     test('returns 400 Bad Request for invalid JSON body', () async {
       // Override request.json stub to throw format exception
       when(() => request.json()).thenThrow(
-        FormatException('Unexpected character'),
+        const FormatException('Unexpected character'),
       );
 
       final response = await route.onRequest(context);
@@ -228,10 +232,11 @@ void main() {
       verifyNever(() => mockClient.createCountry(any()));
     });
 
-     test('returns 400 Bad Request for invalid country data structure', () async {
-       // Provide JSON missing a required field
-       final invalidJson = {'iso_code': 'DE', 'flag_url': '...'}; // Missing name
-       when(() => request.json()).thenAnswer((_) async => invalidJson);
+    test('returns 400 Bad Request for invalid country data structure',
+        () async {
+      // Provide JSON missing a required field
+      final invalidJson = {'iso_code': 'DE', 'flag_url': '...'}; // Missing name
+      when(() => request.json()).thenAnswer((_) async => invalidJson);
 
       final response = await route.onRequest(context);
 
@@ -243,9 +248,10 @@ void main() {
       verifyNever(() => mockClient.createCountry(any()));
     });
 
-    // Add test for client throwing CountryCreateFailure (handled by global handler)
-     test('lets CountryCreateFailure bubble up (handled globally)', () async {
-      final exception = CountryCreateFailure('Duplicate entry');
+    // Add test for client throwing CountryCreateFailure
+    // (handled by global handler)
+    test('lets CountryCreateFailure bubble up (handled globally)', () async {
+      const exception = CountryCreateFailure('Duplicate entry');
       when(() => mockClient.createCountry(any())).thenThrow(exception);
 
       // Expect the specific exception to be thrown
@@ -258,7 +264,7 @@ void main() {
     });
   });
 
-   test('returns 405 Method Not Allowed for unsupported methods', () async {
+  test('returns 405 Method Not Allowed for unsupported methods', () async {
     // Test with PUT, DELETE etc.
     when(() => request.method).thenReturn(HttpMethod.put);
     final responsePut = await route.onRequest(context);
@@ -268,7 +274,12 @@ void main() {
     final responseDelete = await route.onRequest(context);
     expect(responseDelete.statusCode, equals(HttpStatus.methodNotAllowed));
 
-     verifyNever(() => mockClient.fetchCountries(limit: any(named: 'limit'), startAfterId: any(named: 'startAfterId')));
-     verifyNever(() => mockClient.createCountry(any()));
+    verifyNever(
+      () => mockClient.fetchCountries(
+        limit: any(named: 'limit'),
+        startAfterId: any(named: 'startAfterId'),
+      ),
+    );
+    verifyNever(() => mockClient.createCountry(any()));
   });
 }

test/src/middleware/error_handler_test.dart

@@ -10,6 +10,7 @@ import 'package:test/test.dart';
 class _MockRequestContext extends Mock implements RequestContext {}
 
 // Define a class that can be called like a Handler function
+// ignore: one_member_abstracts
 abstract class _CallableHandler {
   Future<Response> call(RequestContext context);
 }
@@ -42,7 +43,8 @@ void main() {
       () async {
     final expectedResponse = Response(body: 'Success');
     // Configure the mock handler's 'call' method
-    when(() => mockHandler.call(any())).thenAnswer((_) async => expectedResponse);
+    when(() => mockHandler.call(any()))
+        .thenAnswer((_) async => expectedResponse);
 
     // Execute the middleware chain, passing the mock handler's call method
     final response = await executeMiddleware(context, mockHandler.call);
@@ -155,4 +157,4 @@ void main() {
       verify(() => mockHandler.call(context)).called(1);
     });
   });
-}
+}