Merge pull request #620 from spassvogel/main

fix initialization of extensions

Author: thejustinwalsh

src/components/Application.tsx

@@ -3,6 +3,7 @@ import {
     useContextBridge,
 } from 'its-fine';
 import {
+    type ExtensionFormat,
     type Application as PixiApplication,
     extensions as PixiExtensions,
     TextStyle,
@@ -45,7 +46,7 @@ const ApplicationImplementation = forwardRef<ApplicationRef, ApplicationProps>(f
 
     const applicationRef: RefObject<PixiApplication | null> = useRef(null);
     const canvasRef = useRef<HTMLCanvasElement>(null);
-    const extensionsRef = useRef<Set<any>>(new Set());
+    const extensionsRef = useRef<Set<ExtensionFormat>>(new Set());
 
     useImperativeHandle(forwardedRef, () => ({
         getApplication()
@@ -119,7 +120,7 @@ const ApplicationImplementation = forwardRef<ApplicationRef, ApplicationProps>(f
             }
 
             // Load any remaining extensions.
-            for (const extension in extensionsToHandle)
+            for (const extension of extensionsToHandle)
             {
                 PixiExtensions.add(extension);
                 extensionsState.add(extension);

test/e2e/components/Application.test.tsx

@@ -1,4 +1,4 @@
-import { Application as PixiApplication } from 'pixi.js';
+import { Application as PixiApplication, extensions as PixiExtensions, ExtensionType } from 'pixi.js';
 import {
     createContext,
     createRef,
@@ -195,4 +195,24 @@ describe('Application', () =>
             expect(roots.size).toEqual(0);
         });
     });
+
+    it('loads extensions provided in the extensions prop', async () =>
+    {
+        const customLoader = {
+            extension: {
+                type: ExtensionType.LoadParser,
+                name: 'custom-loader',
+                priority: 100,
+            },
+        };
+
+        const addSpy = vi.spyOn(PixiExtensions, 'add');
+
+        await act(async () => render((
+            <Application extensions={[customLoader]} />
+        )));
+
+        expect(addSpy).toHaveBeenCalledWith(customLoader);
+        addSpy.mockRestore();
+    });
 });