fix(log): Fix log not getting serialized properly (#1180)

box · Mar 12, 2020 · e9b0c8e · e9b0c8e
1 parent 31f65e5
commit e9b0c8e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/src/lib/Preview.js b/src/lib/Preview.js
@@ -204,10 +204,7 @@ class Preview extends EventEmitter {
 
                 this.emitLogEvent(PREVIEW_METRIC, {
                     event_name: PREVIEW_END_EVENT,
-                    value: {
-                        duration: previewDuration,
-                        viewer_status: this.viewer.getLoadStatus(),
-                    },
+                    value: previewDuration,
                 });
             }
 

diff --git a/src/lib/__tests__/Preview-test.js b/src/lib/__tests__/Preview-test.js
@@ -105,7 +105,6 @@ describe('lib/Preview', () => {
             preview.file = {
                 id: 1,
             };
-            stubs.viewer.getLoadStatus.returns('loaded');
             sandbox.stub(preview, 'emitLogEvent');
             sandbox.stub(Timer, 'createTag').returns('duration_tag');
             sandbox.stub(Timer, 'get').returns({ elapsed: 7 });
@@ -116,13 +115,9 @@ describe('lib/Preview', () => {
             preview.destroy();
             expect(Timer.createTag).to.be.called;
             expect(Timer.stop).to.be.calledWith('duration_tag');
-            expect(stubs.viewer.getLoadStatus).to.be.called;
             expect(preview.emitLogEvent).to.be.calledWith(PREVIEW_METRIC, {
                 event_name: 'preview_end',
-                value: {
-                    duration: 7,
-                    viewer_status: 'loaded',
-                },
+                value: 7,
             });
         });