Polishing

Author: EmilianoSanchez

splitio_web/lib/splitio_web.dart

@@ -33,15 +33,15 @@ class SplitioWeb extends SplitioPlatform {
     required String matchingKey,
     required String? bucketingKey,
     SplitConfiguration? sdkConfiguration,
-  }) async {
+  }) {
     if (_initFuture == null) {
       _initFuture = this._init(
           apiKey: apiKey,
           matchingKey: matchingKey,
           bucketingKey: bucketingKey,
           sdkConfiguration: sdkConfiguration);
     }
-    return _initFuture;
+    return _initFuture!;
   }
 
   Future<void> _init({
@@ -92,7 +92,8 @@ class SplitioWeb extends SplitioPlatform {
     // Create and inject script tag
     final script = document.createElement('script') as HTMLScriptElement;
     script.type = 'text/javascript';
-    script.src = 'assets/packages/splitio_web/web/split-browser-1.6.0.full.min.js';
+    script.src =
+        'assets/packages/splitio_web/web/split-browser-1.6.0.full.min.js';
 
     // Wait for script to load
     final completer = Completer<void>();

splitio_web/lib/src/js_interop.dart

@@ -316,18 +316,6 @@ JSAny buildJsKey(String matchingKey, String? bucketingKey) {
   return matchingKey.toJS;
 }
 
-({String matchingKey, String? bucketingKey}) buildDartKey(JSAny splitKey) {
-  return splitKey is JSString
-      ? (matchingKey: splitKey.toDart, bucketingKey: null)
-      : (
-          matchingKey:
-              (reflectGet(splitKey as JSObject, 'matchingKey'.toJS) as JSString)
-                  .toDart,
-          bucketingKey:
-              (reflectGet(splitKey, 'bucketingKey'.toJS) as JSString).toDart,
-        );
-}
-
 String buildKeyString(String matchingKey, String? bucketingKey) {
   return bucketingKey == null ? matchingKey : '${matchingKey}_$bucketingKey';
 }

splitio_web/test/splitio_web_test.dart

@@ -701,8 +701,7 @@ void main() {
               eventsEndpoint: 'https://events.domain/api',
               authServiceEndpoint: 'https://auth.domain/api/v2',
               streamingServiceEndpoint: 'https://streaming.domain/sse',
-              telemetryServiceEndpoint:
-                  'https://telemetry.domain/api/v1',
+              telemetryServiceEndpoint: 'https://telemetry.domain/api/v1',
               syncConfig: SyncConfig(
                   names: ['flag_1', 'flag_2'], prefixes: ['prefix_1']),
               impressionsMode: ImpressionsMode.none,
@@ -1062,7 +1061,8 @@ void main() {
       await Future<void>.delayed(Duration.zero); // onResume is async
       mockClient.emit(mockClient.Event.SDK_UPDATE);
 
-      await Future<void>.delayed(Duration.zero); // let last event deliver before cancel
+      await Future<void>.delayed(
+          Duration.zero); // let last event deliver before cancel
       await subscription.cancel();
     });
   });

splitio_web/test/utils/js_interop_test_utils.dart

@@ -7,6 +7,19 @@ external JSPromise<Null> _promiseResolve();
 @JS('Object.assign')
 external JSObject _objectAssign(JSObject target, JSObject source);
 
+// Not WASM-compatible. Currently used only in tests
+({String matchingKey, String? bucketingKey}) buildDartKey(JSAny splitKey) {
+  return splitKey is JSString
+      ? (matchingKey: splitKey.toDart, bucketingKey: null)
+      : (
+          matchingKey:
+              (reflectGet(splitKey as JSObject, 'matchingKey'.toJS) as JSString)
+                  .toDart,
+          bucketingKey:
+              (reflectGet(splitKey, 'bucketingKey'.toJS) as JSString).toDart,
+        );
+}
+
 class SplitioMock {
   // JS Browser SDK API mock
   final JS_BrowserSDKPackage splitio = JSObject() as JS_BrowserSDKPackage;