fix(problems): handle no data

Author: andreuSignes

packages/cima_repository/lib/src/cima_repository.dart

@@ -131,22 +131,25 @@ class CimaRepository {
         params: params,
       );
       log('statusCode: ${response.statusCode}');
-      if (response.statusCode != 200) {
-        return Left(ServerFailure());
-      }
-
-      try {
-        final cimaPaginado = CimaPaginado.fromJson(jsonDecode(response.body));
-        if (cimaPaginado.resultados?.isEmpty ?? true) {
-          return Right([]);
+      if (response.statusCode == 200) {
+        try {
+          final cimaPaginado =
+              CimaPaginado.fromJson(jsonDecode(response.body));
+          if (cimaPaginado.resultados?.isEmpty ?? true) {
+            return Right([]);
+          }
+          final problemasSuministro = cimaPaginado.resultados!
+              .map((e) => SupplyProblems.fromJson(e))
+              .toList();
+          return Right(problemasSuministro);
+        } catch (e) {
+          log('error: $e');
+          return Left(FormatFailure());
         }
-        final problemasSuministro = cimaPaginado.resultados!
-            .map((e) => SupplyProblems.fromJson(e))
-            .toList();
-        return Right(problemasSuministro);
-      } catch (e) {
-        log('error: $e');
-        return Left(FormatFailure());
+      } else if (response.statusCode == 204) {
+        return Left(NoDataFailure());
+      } else {
+        return Left(ServerFailure());
       }
     } catch (e) {
       return Left(ServerFailure());

packages/cima_repository/test/src/cima_repository_test.dart

@@ -150,6 +150,15 @@ void main() {
         expect(result.fold((_) => null, (r) => r), problemasSuministro);
       });
 
+      test('find problemas suministro without data', () async {
+        when(() => apiClient.getProblemasSuministro(params: {}))
+            .thenAnswer((_) async => Response('', 204));
+
+        final repository = CimaRepository(remoteDataSource: apiClient);
+        final result = await repository.findProblemasSuministro(params: {});
+        expect(result, Left(NoDataFailure()));
+      });
+
       test('find with params', () async {
         final params = {'nombre': 'aspirina'};
         final problemasSuministroRaw =