fix: copywith missing fields (#103)

* fix: add missing fields in copyWith

* chore: cleanup

* test: updated tests

* ver: 2.0.0-beta.3

Author: vixez

CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 2.0.0-beta.3
+
+- 🐞  Fixed: `copyWith` was missing fields
+- 🚦  Tests: Updated tests.
+
 ## 2.0.0-beta.1
 
 - ❗️🛠  Changed: Renamed `Method` to use `HttpMethod` and refactored helper functions into extensions (`StringToMethod`, and `MethodToString`).

example/lib/main.dart

@@ -3,8 +3,6 @@ import 'dart:developer';
 import 'dart:io';
 
 import 'package:flutter/material.dart';
-import 'package:http/src/base_request.dart';
-import 'package:http/src/base_response.dart';
 import 'package:http_interceptor/http_interceptor.dart';
 import 'package:shared_preferences/shared_preferences.dart';
 

lib/extensions/multipart_request.dart

@@ -19,9 +19,9 @@ extension MultipartRequestCopyWith on MultipartRequest {
         method?.asString ?? this.method,
         url ?? this.url,
       )
-        ..headers.addAll(headers ?? {})
-        ..fields.addAll(fields ?? {})
-        ..files.addAll(files ?? [])
+        ..headers.addAll(headers ?? this.headers)
+        ..fields.addAll(fields ?? this.fields)
+        ..files.addAll(files ?? this.files)
         ..followRedirects = followRedirects ?? this.followRedirects
         ..maxRedirects = maxRedirects ?? this.maxRedirects
         ..persistentConnection =

lib/extensions/request.dart

@@ -21,7 +21,7 @@ extension RequestCopyWith on Request {
         method?.asString ?? this.method,
         url ?? this.url,
       )
-        ..headers.addAll(headers ?? {})
+        ..headers.addAll(headers ?? this.headers)
         ..encoding = encoding ?? this.encoding
         ..body = body ?? this.body
         ..followRedirects = followRedirects ?? this.followRedirects

lib/extensions/streamed_request.dart

@@ -18,7 +18,7 @@ extension StreamedRequestCopyWith on StreamedRequest {
       method?.asString ?? this.method,
       url ?? this.url,
     )
-      ..headers.addAll(headers ?? {})
+      ..headers.addAll(headers ?? this.headers)
       ..followRedirects = followRedirects ?? this.followRedirects
       ..maxRedirects = maxRedirects ?? this.maxRedirects
       ..persistentConnection =

lib/http/intercepted_http.dart

@@ -2,12 +2,8 @@ import 'dart:async';
 import 'dart:convert';
 import 'dart:typed_data';
 
-import 'package:http/http.dart';
 import 'package:http_interceptor/http_interceptor.dart';
 
-import 'intercepted_client.dart';
-import 'interceptor_contract.dart';
-
 /// Class to be used by the user as a replacement for 'http' with interceptor
 /// support.
 ///

pubspec.yaml

@@ -1,6 +1,6 @@
 name: http_interceptor
 description: A lightweight, simple plugin that allows you to intercept request and response objects and modify them if desired.
-version: 2.0.0-beta.1
+version: 2.0.0-beta.3
 homepage: https://github.com/CodingAleCR/http_interceptor
 issue_tracker: https://github.com/CodingAleCR/http_interceptor/issues
 repository: https://github.com/CodingAleCR/http_interceptor

test/extensions/base_reponse_test.dart

@@ -1,4 +1,3 @@
-import 'package:http/http.dart';
 import 'package:http_interceptor/http_interceptor.dart';
 import 'package:test/test.dart';
 

test/extensions/base_request_test.dart

@@ -1,6 +1,5 @@
 import 'dart:convert';
 
-import 'package:http/http.dart';
 import 'package:http_interceptor/http_interceptor.dart';
 import 'package:test/test.dart';
 

test/extensions/request_test.dart

@@ -1,6 +1,5 @@
 import 'dart:convert';
 
-import 'package:http/http.dart';
 import 'package:http_interceptor/http_interceptor.dart';
 import 'package:test/test.dart';
 
@@ -10,7 +9,8 @@ main() {
 
   setUpAll(() {
     baseRequest = Request("GET", Uri.https("www.google.com", "/helloworld"))
-      ..body = jsonEncode(<String, String>{'some_param': 'some value'});
+      ..body = jsonEncode(<String, String>{'some_param': 'some value'})
+      ..headers['some_header'] = 'header_value';
     request = baseRequest as Request;
   });
 
@@ -205,18 +205,23 @@ main() {
     test('Request is copied with different encoding', () {
       // Arrange
       final newEncoding = Encoding.getByName('latin1');
-      final changedHeaders = {'content-type': 'text/plain; charset=iso-8859-1'};
+      final changedHeaders = {'content-type': 'text/plain; charset=iso-8859-1'}
+        ..addAll(request.headers);
+
+      Request updatedHeadersRequest = request.copyWith(
+        headers: changedHeaders,
+      );
 
       // Act
-      Request copied = request.copyWith(
+      Request copied = updatedHeadersRequest.copyWith(
         encoding: newEncoding,
       );
 
       // Assert
       expect(copied.url, equals(request.url));
       expect(copied.method, equals(request.method));
       expect(copied.headers.length, equals(request.headers.length));
-      expect(copied.headers, equals(changedHeaders));
+      expect(updatedHeadersRequest.headers, equals(request.headers));
       expect(copied.body, equals(request.body));
       expect(
           copied.encoding,