Update CI (#793)

rrousselGit · Dec 14, 2022 · 63cf7a3 · 63cf7a3
1 parent 495a4ca
commit 63cf7a3
Show file tree

Hide file tree

Showing 27 changed files with 41 additions and 35 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -1,25 +1,50 @@
 name: Build
 
 on:
-  push:
   pull_request:
+    paths-ignore:
+      - "**.md"
+  push:
+    branches:
+      - master
+    paths-ignore:
+      - "**.md"
   schedule:
     # runs the CI everyday at 10AM
     - cron: "0 10 * * *"
 
 jobs:
   flutter:
     runs-on: ubuntu-latest
-    container: cirrusci/flutter:${{matrix.channel}}
 
     strategy:
       matrix:
         channel:
+          - master
           - dev
           - beta
 
     steps:
       - uses: actions/checkout@v2
+      - uses: actions/checkout@v3.1.0
+        with:
+          fetch-depth: 2
+
+      - uses: subosito/flutter-action@v2.7.1
+        with:
+          channel: ${{ matrix.channel }}
+
+      - name: Add pub cache bin to PATH
+        run: echo "$HOME/.pub-cache/bin" >> $GITHUB_PATH
+      - name: Add pub cache to PATH
+        run: echo "PUB_CACHE="$HOME/.pub-cache"" >> $GITHUB_ENV
+
+      - name: Install dependencies
+        run: flutter pub get
+
+      - run: flutter format lib test --set-exit-if-changed
+      - run: flutter analyze --no-current-package
 
-      - name: Execute test script
-        run: curl -s https://raw.githubusercontent.com/rrousselGit/ci/master/scripts/ci.sh | bash -s nnbd
+      - run: flutter test --no-pub --coverage test/null_safe
+      - run: flutter test --no-pub --coverage --no-sound-null-safety test/mixed_mode
+        if: matrix.package_path != 'master'
diff --git a/example/test_driver/app_test.dart b/example/test_driver/app_test.dart
@@ -1,4 +1,3 @@
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:flutter_driver/flutter_driver.dart';
 import 'package:test/test.dart';
 

diff --git a/scripts/flutter_test.sh b/scripts/flutter_test.sh
diff --git a/test/common_legacy.dart → test/mixed_mode/common_legacy.dart b/test/common_legacy.dart → test/mixed_mode/common_legacy.dart
diff --git a/test/inherited_provider_legacy_test.dart → ..._mode/inherited_provider_legacy_test.dart b/test/inherited_provider_legacy_test.dart → ..._mode/inherited_provider_legacy_test.dart
@@ -4,7 +4,7 @@ import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:provider/provider.dart';
 
-import 'common.dart';
+import '../null_safe/common.dart';
 
 BuildContext get context => find.byType(Context).evaluate().single;
 

diff --git a/test/provider_legacy_mixing_impl.dart → ...xed_mode/provider_legacy_mixing_impl.dart b/test/provider_legacy_mixing_impl.dart → ...xed_mode/provider_legacy_mixing_impl.dart
diff --git a/test/provider_legacy_mixing_test.dart → ...xed_mode/provider_legacy_mixing_test.dart b/test/provider_legacy_mixing_test.dart → ...xed_mode/provider_legacy_mixing_test.dart
diff --git a/test/provider_legacy_test.dart → test/mixed_mode/provider_legacy_test.dart b/test/provider_legacy_test.dart → test/mixed_mode/provider_legacy_test.dart
@@ -4,7 +4,7 @@ import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:provider/provider.dart';
 
-import 'common.dart';
+import '../null_safe/common.dart';
 
 void main() {
   // See `provider_test.dart` for corresponding sound mode test.

diff --git a/test/builder_test.dart → test/null_safe/builder_test.dart b/test/builder_test.dart → test/null_safe/builder_test.dart
diff --git a/test/change_notifier_provider_test.dart → ...l_safe/change_notifier_provider_test.dart b/test/change_notifier_provider_test.dart → ...l_safe/change_notifier_provider_test.dart
diff --git a/test/common.dart → test/null_safe/common.dart b/test/common.dart → test/null_safe/common.dart
@@ -3,7 +3,6 @@ import 'dart:async';
 import 'package:flutter/foundation.dart';
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 

diff --git a/test/consumer_test.dart → test/null_safe/consumer_test.dart b/test/consumer_test.dart → test/null_safe/consumer_test.dart
@@ -1,6 +1,5 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 

diff --git a/test/context_test.dart → test/null_safe/context_test.dart b/test/context_test.dart → test/null_safe/context_test.dart
@@ -1,6 +1,5 @@
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 

diff --git a/test/devtool_test.dart → test/null_safe/devtool_test.dart b/test/devtool_test.dart → test/null_safe/devtool_test.dart
diff --git a/test/future_provider_test.dart → test/null_safe/future_provider_test.dart b/test/future_provider_test.dart → test/null_safe/future_provider_test.dart
@@ -2,7 +2,6 @@ import 'dart:async';
 
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 

diff --git a/test/inherited_provider_test.dart → test/null_safe/inherited_provider_test.dart b/test/inherited_provider_test.dart → test/null_safe/inherited_provider_test.dart
@@ -1,7 +1,6 @@
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
 
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 import 'package:provider/single_child_widget.dart';
@@ -554,7 +553,8 @@ The context used was: Context
         ),
       );
 
-      final rootElement = tester.element(find.bySubtype<InheritedProvider>());
+      final rootElement =
+          tester.element(find.byWidgetPredicate((w) => w is InheritedProvider));
 
       expect(
         rootElement.toString(),
@@ -583,7 +583,8 @@ The context used was: Context
         ),
       );
 
-      final rootElement = tester.element(find.bySubtype<InheritedProvider>());
+      final rootElement =
+          tester.element(find.byWidgetPredicate((w) => w is InheritedProvider));
 
       expect(
         rootElement.toString(),
@@ -607,7 +608,8 @@ The context used was: Context
         ),
       );
 
-      final rootElement = tester.element(find.bySubtype<InheritedProvider>());
+      final rootElement =
+          tester.element(find.byWidgetPredicate((w) => w is InheritedProvider));
 
       expect(
         rootElement.toString(),
@@ -634,8 +636,8 @@ The context used was: Context
         ),
       );
 
-      final rootElement =
-          tester.element(find.bySubtype<DeferredInheritedProvider>());
+      final rootElement = tester.element(
+          find.byWidgetPredicate((w) => w is DeferredInheritedProvider));
 
       expect(
         rootElement.toString(),
@@ -666,7 +668,8 @@ DeferredInheritedProvider<int, int>(controller: 42, value: 24)'''),
         ),
       );
 
-      final rootElement = tester.element(find.bySubtype<InheritedProvider>());
+      final rootElement =
+          tester.element(find.byWidgetPredicate((w) => w is InheritedProvider));
 
       expect(
         rootElement.toString(),

diff --git a/test/listenable_provider_test.dart → test/null_safe/listenable_provider_test.dart b/test/listenable_provider_test.dart → test/null_safe/listenable_provider_test.dart
@@ -1,7 +1,6 @@
 // ignore_for_file: invalid_use_of_protected_member
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 

diff --git a/test/listenable_proxy_provider_test.dart → ..._safe/listenable_proxy_provider_test.dart b/test/listenable_proxy_provider_test.dart → ..._safe/listenable_proxy_provider_test.dart
@@ -1,7 +1,6 @@
 // ignore_for_file: invalid_use_of_protected_member
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 

diff --git a/test/matchers.dart → test/null_safe/matchers.dart b/test/matchers.dart → test/null_safe/matchers.dart
diff --git a/test/multi_provider_test.dart → test/null_safe/multi_provider_test.dart b/test/multi_provider_test.dart → test/null_safe/multi_provider_test.dart
diff --git a/test/provider_test.dart → test/null_safe/provider_test.dart b/test/provider_test.dart → test/null_safe/provider_test.dart
diff --git a/test/proxy_provider_test.dart → test/null_safe/proxy_provider_test.dart b/test/proxy_provider_test.dart → test/null_safe/proxy_provider_test.dart
@@ -1,6 +1,5 @@
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 

diff --git a/test/reassemble_test.dart → test/null_safe/reassemble_test.dart b/test/reassemble_test.dart → test/null_safe/reassemble_test.dart
diff --git a/test/selector_test.dart → test/null_safe/selector_test.dart b/test/selector_test.dart → test/null_safe/selector_test.dart
@@ -1,9 +1,7 @@
 import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart' as mockito show when;
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 import 'package:provider/single_child_widget.dart';

diff --git a/test/stateful_provider_test.dart → test/null_safe/stateful_provider_test.dart b/test/stateful_provider_test.dart → test/null_safe/stateful_provider_test.dart
@@ -1,6 +1,5 @@
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 import 'package:provider/src/provider.dart';

diff --git a/test/stream_provider_test.dart → test/null_safe/stream_provider_test.dart b/test/stream_provider_test.dart → test/null_safe/stream_provider_test.dart
@@ -2,7 +2,6 @@ import 'dart:async';
 
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';
 

diff --git a/test/value_listenable_test.dart → test/null_safe/value_listenable_test.dart b/test/value_listenable_test.dart → test/null_safe/value_listenable_test.dart
@@ -1,7 +1,6 @@
 import 'package:flutter/foundation.dart';
 import 'package:flutter/widgets.dart';
 import 'package:flutter_test/flutter_test.dart';
-// ignore: import_of_legacy_library_into_null_safe
 import 'package:mockito/mockito.dart';
 import 'package:provider/provider.dart';