Merge pull request #773 from OneSignal/fix/ci

Fix CI errors

Author: shepherd-l

.github/workflows/ci.yml

@@ -17,7 +17,7 @@ jobs:
       - name: Setup Flutter
         uses: subosito/flutter-action@v2
         with:
-          flutter-version: '2.10.4'
+          flutter-version: '3.3.2'
           channel: 'stable'
 
       - name: Checkout OneSignal-Flutter-SDK
@@ -27,7 +27,7 @@ jobs:
         run: flutter pub get
 
       - name: Static Analysis
-        run: flutter analyze --no-pub --no-fatal-infos
+        run: flutter analyze
 
       - name: Ensure the Dart code is formatted correctly
         run: flutter format -o none --set-exit-if-changed .

example/lib/main.dart

@@ -231,7 +231,7 @@ class _MyAppState extends State<MyApp> {
     // Toggle pausing (displaying or not) of IAMs
     OneSignal.InAppMessages.paused(true);
     var arePaused = await OneSignal.InAppMessages.arePaused();
-    print('Notifications paused ${arePaused}');
+    print('Notifications paused $arePaused');
   }
 
   oneSignalOutcomeExamples() async {

lib/onesignal_flutter.dart

@@ -23,12 +23,19 @@ class OneSignal {
   /// Note that the iOS and Android native libraries are static,
   /// so if you create multiple instances of OneSignal, they will
   /// mostly share the same state.
+  // ignore: non_constant_identifier_names
   static OneSignalDebug Debug = new OneSignalDebug();
+  // ignore: non_constant_identifier_names
   static OneSignalUser User = new OneSignalUser();
+  // ignore: non_constant_identifier_names
   static OneSignalNotifications Notifications = new OneSignalNotifications();
+  // ignore: non_constant_identifier_names
   static OneSignalSession Session = new OneSignalSession();
+  // ignore: non_constant_identifier_names
   static OneSignalLocation Location = new OneSignalLocation();
+  // ignore: non_constant_identifier_names
   static OneSignalInAppMessages InAppMessages = new OneSignalInAppMessages();
+  // ignore: non_constant_identifier_names
   static OneSignalLiveActivities LiveActivities = new OneSignalLiveActivities();
 
   // private channels used to bridge to ObjC/Java

lib/src/inappmessage.dart

@@ -1,4 +1,3 @@
-import 'package:onesignal_flutter/onesignal_flutter.dart';
 import 'package:onesignal_flutter/src/utils.dart';
 
 /// When a click action is defined on an In App Message form the dashboard,

lib/src/notification.dart

@@ -362,9 +362,7 @@ class OSNotificationWillDisplayEvent extends JSONStringRepresentable {
   }
 
   void preventDefault() {
-    if (notification != null) {
-      OneSignal.Notifications.preventDefault(notification.notificationId);
-    }
+    OneSignal.Notifications.preventDefault(notification.notificationId);
   }
 
   String jsonRepresentation() {
@@ -383,9 +381,7 @@ class OSNotificationClickEvent extends JSONStringRepresentable {
   }
 
   void preventDefault() {
-    if (notification != null) {
-      OneSignal.Notifications.preventDefault(notification.notificationId);
-    }
+    OneSignal.Notifications.preventDefault(notification.notificationId);
   }
 
   String jsonRepresentation() {

lib/src/notifications.dart

@@ -3,7 +3,6 @@ import 'dart:io';
 import 'package:flutter/services.dart';
 import 'package:onesignal_flutter/src/defines.dart';
 import 'package:onesignal_flutter/src/notification.dart';
-import 'package:onesignal_flutter/src/permission.dart';
 
 typedef void OnNotificationPermissionChangeObserver(bool permission);
 

test/mock_channel.dart

@@ -10,13 +10,15 @@ import 'package:flutter_test/flutter_test.dart';
 
 class OneSignalMockChannelController {
   MethodChannel _channel = const MethodChannel('OneSignal');
+  MethodChannel _debugChannel = const MethodChannel('OneSignal#debug');
   MethodChannel _tagsChannel = const MethodChannel('OneSignal#tags');
 
   late OneSignalState state;
 
   OneSignalMockChannelController() {
     this._channel.setMockMethodCallHandler(_handleMethod);
     this._tagsChannel.setMockMethodCallHandler(_handleMethod);
+    this._debugChannel.setMockMethodCallHandler(_handleMethod);
   }
 
   void resetState() {
@@ -117,7 +119,7 @@ class OneSignalState {
   }
 
   void setLogLevel(Map<dynamic, dynamic> params) {
-    int? level = params['console'] as int?;
+    int? level = params['logLevel'] as int?;
     int? visual = params['visual'] as int?;
 
     if (level != null) this.logLevel = OSLogLevel.values[level];

test/onesignalflutter_test.dart

@@ -7,13 +7,12 @@ void main() {
 
   OneSignalMockChannelController channelController =
       OneSignalMockChannelController();
-  OneSignal onesignal = OneSignal();
 
   setUp(() {
     channelController.resetState();
   });
 
-  test('set set log level', () {
+  test('set log level', () {
     OneSignal.Debug.setLogLevel(
       OSLogLevel.info,
     ).then(expectAsync1((v) {