Windows unit tests no longer use the Windows SDK

flutter_local_notifications_windows/lib/src/ffi/mock.dart

@@ -0,0 +1,55 @@
+// Just a mock, doesn't need real types or safety.
+// ignore_for_file: type_annotate_public_apis, always_specify_types
+
+import 'dart:ffi';
+
+import 'package:ffi/ffi.dart';
+
+import 'bindings.dart';
+
+/// Mocked FFI bindings.
+class MockBindings implements NotificationsPluginBindings {
+  @override
+  void cancelAll(_) {}
+
+  @override
+  void cancelNotification(_, int id) {}
+
+  @override
+  Pointer<NativePlugin> createPlugin() => malloc<Int>().cast();
+
+  @override
+  void disposePlugin(_) {}
+
+  @override
+  void freeLaunchDetails(_) {}
+
+  @override
+  void freeDetailsArray(ptr) => malloc.free(ptr);
+
+  @override
+  Pointer<NativeNotificationDetails> getActiveNotifications(_, __) =>
+      malloc<NativeNotificationDetails>();
+
+  @override
+  Pointer<NativeNotificationDetails> getPendingNotifications(_, __) =>
+      malloc<NativeNotificationDetails>();
+
+  @override
+  bool hasPackageIdentity() => false;
+
+  @override
+  bool init(_, __, ___, ____, _____, ______) => true;
+
+  @override
+  bool isValidXml(ptr) => true;
+
+  @override
+  bool scheduleNotification(a, b, c, d) => true;
+
+  @override
+  bool showNotification(a, b, c, d) => true;
+
+  @override
+  NativeUpdateResult updateNotification(a, b, c) => NativeUpdateResult.success;
+}

flutter_local_notifications_windows/lib/src/ffi/utils.dart

@@ -6,6 +6,17 @@ import 'package:flutter_local_notifications_platform_interface/flutter_local_not
 import '../details.dart';
 import '../plugin/base.dart';
 import 'bindings.dart';
+import 'mock.dart';
+
+/// Whether unit tests are running and to not use the real DLL files.
+bool isUnitTest = false;
+
+/// Gets the correct DLL bindings based on [isUnitTest].
+NotificationsPluginBindings getBindings() => isUnitTest
+    ? MockBindings()
+    : NotificationsPluginBindings(
+        DynamicLibrary.open('flutter_local_notifications_windows.dll'),
+      );
 
 /// Helpful methods on native string maps.
 extension NativeStringMapUtils on NativeStringMap {

flutter_local_notifications_windows/lib/src/plugin/ffi.dart

@@ -1,6 +1,7 @@
 import 'dart:ffi';
 
 import 'package:ffi/ffi.dart';
+import 'package:xml/xml.dart';
 
 import '../details.dart';
 import '../details/notification_to_xml.dart';
@@ -22,6 +23,16 @@ extension on String {
       this[23] == '-';
 }
 
+/// Does a basic syntax check on XML.
+bool checkXml(String xml) {
+  try {
+    XmlDocument.parse(xml);
+    return true;
+  } on XmlFormatException {
+    return false;
+  }
+}
+
 /// The Windows implementation of `package:flutter_local_notifications`.
 class FlutterLocalNotificationsWindows extends WindowsNotificationsBase {
   /// Creates an instance of the native plugin.
@@ -37,11 +48,7 @@ class FlutterLocalNotificationsWindows extends WindowsNotificationsBase {
   static FlutterLocalNotificationsWindows? instance;
 
   /// The FFI generated bindings to the native code.
-  late final NotificationsPluginBindings _bindings =
-      NotificationsPluginBindings(_library);
-
-  final DynamicLibrary _library =
-      DynamicLibrary.open('flutter_local_notifications_windows.dll');
+  late final NotificationsPluginBindings _bindings = getBindings();
 
   /// A pointer to the C++ handler class.
   late final Pointer<NativePlugin> _plugin;
@@ -281,6 +288,9 @@ class FlutterLocalNotificationsWindows extends WindowsNotificationsBase {
 
   @override
   bool isValidXml(String xml) => using((Arena arena) {
+        if (!checkXml(xml)) {
+          return false;
+        }
         final Pointer<Utf8> nativeXml = xml.toNativeUtf8(allocator: arena);
         return _bindings.isValidXml(nativeXml);
       });