Fix CI after merge

* don't collect test result for example app, there are no test atm
* remove usage of `unawaited` - pedantic is deprecated and the new `unawait` from the Dart SDK required Dart 2.14 which would raise the min SDK for the library
* try to prevent test not completing with Dart 2.12 because of a a wrong expectation on a `Future` - use `expectLater` instead
* re-add example test which got lost in the merge

Author: kuhnroyal

.github/PULL_REQUEST_TEMPLATE.md

@@ -2,7 +2,7 @@
 
 - [ ] I have read the [Documentation](https://pub.dartlang.org/packages/dio)
 - [ ] I have searched for a similar pull request in the [project](https://github.com/flutterchina/dio/pulls) and found none
-- [ ] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)
+- [ ] I have updated this branch with the latest `develop` to avoid conflicts (via merge from master or rebase)
 - [ ] I have added the required tests to prove the fix/feature I am adding
 - [ ] I have updated the documentation (if necessary)
 - [ ] I have run the tests and they pass

.github/workflows/test-report.yml

@@ -1,11 +1,10 @@
-name: Test Report
+name: Publish Test Report
 
 on:
   workflow_run:
     workflows:
       - Integration
       - Example
-      - Example App
     types:
       - completed
 

dio/lib/src/adapters/browser_adapter.dart

@@ -1,11 +1,12 @@
 import 'dart:async';
 import 'dart:convert';
+import 'dart:html';
 import 'dart:typed_data';
-import '../dio_error.dart';
-import '../options.dart';
+
 import '../adapter.dart';
-import 'dart:html';
+import '../dio_error.dart';
 import '../headers.dart';
+import '../options.dart';
 
 HttpClientAdapter createAdapter() => BrowserHttpClientAdapter();
 
@@ -47,7 +48,7 @@ class BrowserHttpClientAdapter implements HttpClientAdapter {
 
     var completer = Completer<ResponseBody>();
 
-    unawaited(xhr.onLoad.first.then((_) {
+    xhr.onLoad.first.then((_) {
       Uint8List body = (xhr.response as ByteBuffer).asUint8List();
       completer.complete(
         ResponseBody.fromBytes(
@@ -58,7 +59,7 @@ class BrowserHttpClientAdapter implements HttpClientAdapter {
           isRedirect: xhr.status == 302 || xhr.status == 301,
         ),
       );
-    }));
+    });
 
     bool haveSent = false;
 
@@ -134,7 +135,7 @@ class BrowserHttpClientAdapter implements HttpClientAdapter {
       }
     });
 
-    unawaited(xhr.onError.first.then((_) {
+    xhr.onError.first.then((_) {
       // Unfortunately, the underlying XMLHttpRequest API doesn't expose any
       // specific information about the error itself.
       completer.completeError(
@@ -145,17 +146,17 @@ class BrowserHttpClientAdapter implements HttpClientAdapter {
         ),
         StackTrace.current,
       );
-    }));
+    });
 
-    unawaited(cancelFuture?.then((_) {
+    cancelFuture?.then((_) {
       if (xhr.readyState < 4 && xhr.readyState > 0) {
         try {
           xhr.abort();
         } catch (e) {
           // ignore
         }
       }
-    }));
+    });
 
     if (requestStream != null) {
       var _completer = Completer<Uint8List>();

dio/test/basic_test.dart

@@ -4,6 +4,7 @@
 @TestOn('vm')
 import 'dart:async';
 import 'dart:io';
+
 import 'package:dio/dio.dart';
 import 'package:test/test.dart';
 
@@ -68,7 +69,8 @@ void main() {
 
   test('#status error', () async {
     var dio = Dio()..options.baseUrl = 'http://httpbin.org/status/';
-    expect(
+
+    await expectLater(
       dio.get('401').catchError((e) => throw e.response.statusCode as Object),
       throwsA(401),
     );

dio/test/readtimeout_test.dart

@@ -59,7 +59,7 @@ void main() {
 
     dio.options
       ..baseUrl = serverUrl.toString()
-      ..receiveTimeout= SLEEP_DURATION_AFTER_CONNECTION_ESTABLISHED - 1000;
+      ..receiveTimeout = SLEEP_DURATION_AFTER_CONNECTION_ESTABLISHED - 1000;
 
     DioError error;
 

example/test/example_test.dart

@@ -0,0 +1,31 @@
+// ignore_for_file: unused_import
+
+import 'package:dio_example/adapter.dart';
+import 'package:dio_example/cancel_request.dart';
+import 'package:dio_example/cookie_mgr.dart';
+import 'package:dio_example/custom_cache_interceptor.dart';
+import 'package:dio_example/dio.dart';
+import 'package:dio_example/download.dart';
+import 'package:dio_example/download_with_trunks.dart';
+import 'package:dio_example/extend_dio.dart';
+import 'package:dio_example/formdata.dart';
+import 'package:dio_example/generic.dart';
+import 'package:dio_example/http2_adapter.dart';
+import 'package:dio_example/options.dart';
+import 'package:dio_example/post_stream_and_bytes.dart';
+import 'package:dio_example/proxy.dart';
+import 'package:dio_example/queue_interceptors.dart';
+import 'package:dio_example/queued_interceptor_crsftoken.dart';
+import 'package:dio_example/request_interceptors.dart';
+import 'package:dio_example/response_interceptor.dart';
+import 'package:dio_example/test.dart';
+import 'package:dio_example/transformer.dart';
+import 'package:test/test.dart';
+
+/// This test ensures that all examples are included in the compilation unit
+/// and have no compile errors during integration builds.
+///
+/// Otherwise this does nothing.
+void main() {
+  test('dummy', () {});
+}