Merge pull request #30 from Eyevinn/frontend/update-epas-spec-compliance

fix: use correct TWarningEventPayload type for spec compliance

Author: sinewave440hz

spec/PlayerAnalyticsConnector.spec.ts

@@ -371,6 +371,23 @@ describe("PlayerAnalyticsConnector", () => {
       expect(body.event).toBe("warning");
       expect(body.payload.category).toBe("player");
     });
+
+    it("should send warning event with all TWarningEventPayload fields", () => {
+      connector.reportWarning({
+        category: "decoder",
+        code: "CODEC_FALLBACK",
+        message: "Fell back to software decoding",
+        data: { codec: "hevc", fallback: "avc1" },
+      });
+
+      expect(mockFetch).toHaveBeenCalled();
+      const body = JSON.parse(mockFetch.calls.argsFor(0)[1].body);
+      expect(body.event).toBe("warning");
+      expect(body.payload.category).toBe("decoder");
+      expect(body.payload.code).toBe("CODEC_FALLBACK");
+      expect(body.payload.message).toBe("Fell back to software decoding");
+      expect(body.payload.data).toEqual({ codec: "hevc", fallback: "avc1" });
+    });
   });
 
   describe("deinit()", () => {

src/PlayerAnalyticsConnector.ts

@@ -10,6 +10,7 @@ import {
   TErrorEventPayload,
   TEventType,
   TMetadataEventPayload,
+  TWarningEventPayload,
   UUID,
 } from "@eyevinn/player-analytics-specification";
 
@@ -187,7 +188,7 @@ export class PlayerAnalyticsConnector {
     });
   }
 
-  public reportWarning(payload: TErrorEventPayload) {
+  public reportWarning(payload: TWarningEventPayload) {
     if (!this.analyticsInitiated) {
       console.warn("[PlayerAnalyticsConnector] Analytics not initiated");
       return;