trigger error event if peaks file cannot be found (#124)

* use absolute path

* trigger error event if peaks file cannot be found

CHANGES.md

@@ -1,9 +1,15 @@
 # videojs-wavesurfer changelog
 
+## 3.4.0 - unreleased
+
+- Trigger error event if peaks file cannot be found (#124)
+
+
 ## 3.3.1 - 2020/09/30
 
 - Doc fixes and updated dependencies
 
+
 ## 3.3.0 - 2020/08/10
 
 - Fixed webpack externals configuration: it's no longer needed to use additional webpack

karma.conf.js

@@ -165,7 +165,7 @@ module.exports = function(config) {
         colors: true,
         reporters: ['verbose', 'progress', 'coverage'],
         coverageReporter: {
-            dir: 'coverage',
+            dir: path.resolve(__dirname, 'coverage'),
             reporters: [
                 { type: 'html', subdir: 'html' },
                 { type: 'lcov', subdir: 'lcov' }

src/js/videojs.wavesurfer.js

@@ -324,11 +324,10 @@ class Wavesurfer extends Plugin {
                     this.log(err, 'error');
                 }
             });
-            request.on('error', e => {
-                this.log('Unable to retrieve peak data from ' + peaks +
-                    '. Status code: ' + request.response.status, 'warn');
-                this.log('Loading URL: ' + url);
-                this.surfer.load(url);
+            request.once('error', e => {
+                this.player.trigger(Event.ERROR,
+                    'Unable to retrieve peak data from ' + peaks +
+                    '. Status code: ' + request.response.status);
             });
         }
     }

test/videojs.wavesurfer.spec.js

@@ -335,9 +335,13 @@ describe('Wavesurfer', () => {
     });
 
     /** @test {Wavesurfer#load} */
-    it('ignores peaks if file cannot be found', (done) => {
+    it('throws error if peaks file cannot be found', (done) => {
 
-        player.one(Event.WAVE_READY, done);
+        player.one(Event.ERROR, (element, err) => {
+            expect(err).toStartWith('Unable to retrieve peak data from non_existing_file.json.');
+            expect(err).toEndWith('Status code: 404');
+            done();
+        });
 
         // try loading from non-existing peaks file
         player.src({