fix(tracker): fix custom script factory provider

fixes #56

Author: EmmanuelRoux

projects/tracker/src/lib/matomo-initializer.service.spec.ts

@@ -12,6 +12,7 @@ import { ALREADY_INITIALIZED_ERROR, ALREADY_INJECTED_ERROR } from './errors';
 import { MatomoHolder } from './holder';
 import { MatomoInitializerService } from './matomo-initializer.service';
 import { MatomoTracker, NoopMatomoTracker } from './matomo-tracker.service';
+import { NgxMatomoTrackerModule } from './ngx-matomo-tracker.module';
 import {
   createDefaultMatomoScriptElement,
   MATOMO_SCRIPT_FACTORY,
@@ -452,6 +453,39 @@ describe('MatomoInitializerService', () => {
     expect(injectedScript?.dataset.cookieconsent).toEqual('statistics');
   });
 
+  it('should create custom script tag with forRoot factory', () => {
+    // Given
+    let injectedScript: HTMLScriptElement | undefined;
+
+    setUpScriptInjection(script => (injectedScript = script));
+
+    TestBed.configureTestingModule({
+      imports: [
+        NgxMatomoTrackerModule.forRoot(
+          {
+            siteId: 1,
+            trackerUrl: '',
+            scriptUrl: '/fake/script/url',
+          } as MatomoConfiguration,
+          (scriptUrl, document) => {
+            const script = createDefaultMatomoScriptElement(scriptUrl, document);
+
+            script.setAttribute('data-cookieconsent', 'statistics');
+
+            return script;
+          }
+        ),
+      ],
+    });
+
+    // Inject service to trigger initialization on module init
+    TestBed.inject(MatomoInitializerService);
+
+    // Then
+    expect(injectedScript?.src).toMatch('^(.+://[^/]+)?/fake/script/url$');
+    expect(injectedScript?.dataset.cookieconsent).toEqual('statistics');
+  });
+
   it('should defer script injection until tracker configuration is provided', () => {
     // Given
     let injectedScript: HTMLScriptElement | undefined;

projects/tracker/src/lib/ngx-matomo-tracker.module.ts

@@ -28,7 +28,7 @@ export class NgxMatomoTrackerModule {
     const providers: Provider[] = [{ provide: MATOMO_CONFIGURATION, useValue: config }];
 
     if (scriptFactory) {
-      providers.push({ provide: MATOMO_SCRIPT_FACTORY, useFactory: scriptFactory });
+      providers.push({ provide: MATOMO_SCRIPT_FACTORY, useValue: scriptFactory });
     }
 
     return {