update tests for Illegal constructor

Author: mohayonao

test/AnalyserNode.js

@@ -12,10 +12,9 @@ describe("AnalyserNode", function() {
 
       assert(node instanceof global.AnalyserNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.AnalyserNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.AnalyserNode(); }, TypeError);
     });
   });
 

test/AudioBuffer.js

@@ -23,10 +23,9 @@ describe("AudioBuffer", function() {
       assert.throws(function() {
         audioContext.createBuffer(2, 128, 44100.5);
       }, TypeError);
-
-      assert.throws(function() {
-        return new global.AudioBuffer();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.AudioBuffer(); }, TypeError);
     });
   });
 

test/AudioBufferSourceNode.js

@@ -12,10 +12,9 @@ describe("AudioBufferSourceNode", function() {
 
       assert(node instanceof global.AudioBufferSourceNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.AudioBufferSourceNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.AudioBufferSourceNode(); }, TypeError);
     });
   });
 

test/AudioDestinationNode.js

@@ -12,10 +12,9 @@ describe("AudioDestinationNode", function() {
 
       assert(node instanceof global.AudioDestinationNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.AudioDestinationNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.AudioDestinationNode(); }, TypeError);
     });
   });
 

test/AudioListener.js

@@ -11,10 +11,9 @@ describe("AudioListener", function() {
       var listener = audioContext.listener;
 
       assert(listener instanceof global.AudioListener);
-
-      assert.throws(function() {
-        return new global.AudioListener();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.AudioListener(); }, TypeError);
     });
   });
 

test/AudioNode.js

@@ -23,10 +23,6 @@ describe("AudioNode", function() {
       assert(node instanceof global.AudioNode);
       assert(node instanceof global.EventTarget);
 
-      assert.throws(function() {
-        return new global.AudioNode();
-      }, TypeError);
-
       return audioContext.close().then(function() {
         return immigration.apply(function(admission) {
           return new WebAudioTestAPI.AudioNode(admission, { context: audioContext });
@@ -37,6 +33,9 @@ describe("AudioNode", function() {
         assert(e instanceof TypeError && /audioContext has been closed/i.test(e.message));
       });
     });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.AudioNode(); }, TypeError);
+    });
   });
 
   describe("#context: AudioContext", function() {

test/AudioParam.js

@@ -19,13 +19,12 @@ describe("AudioParam", function() {
 
       assert(param instanceof global.AudioParam);
 
-      assert.throws(function() {
-        return new global.AudioParam();
-      }, TypeError);
-
       // test api
       assert(param.$node === node);
     });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.AudioParam(); }, TypeError);
+    });
   });
 
   describe("#name: string", function() {

test/AudioProcessingEvent.js

@@ -20,13 +20,12 @@ describe("AudioProcessingEvent", function() {
       assert(event instanceof global.AudioProcessingEvent);
       assert(event instanceof global.Event);
 
-      assert.throws(function() {
-        return new global.AudioProcessingEvent();
-      }, TypeError);
-
       // test api
       assert(event.$name === "AudioProcessingEvent");
       assert(event.$node === node);
     });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.AudioProcessingEvent(); }, TypeError);
+    });
   });
 });

test/BiquadFilterNode.js

@@ -12,10 +12,9 @@ describe("BiquadFilterNode", function() {
 
       assert(node instanceof global.BiquadFilterNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.BiquadFilterNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.BiquadFilterNode(); }, TypeError);
     });
   });
 

test/ChannelMergerNode.js

@@ -16,10 +16,9 @@ describe("ChannelMergerNode", function() {
       assert.throws(function() {
         audioContext.createChannelMerger(5.1);
       }, TypeError);
-
-      assert.throws(function() {
-        return new global.ChannelMergerNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.ChannelMergerNode(); }, TypeError);
     });
   });
 

test/ChannelSplitterNode.js

@@ -16,10 +16,9 @@ describe("ChannelSplitterNode", function() {
       assert.throws(function() {
         audioContext.createChannelSplitter(5.1);
       }, TypeError);
-
-      assert.throws(function() {
-        return new global.ChannelSplitterNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.ChannelSplitterNode(); }, TypeError);
     });
   });
 

test/ConvolverNode.js

@@ -12,10 +12,9 @@ describe("ConvolverNode", function() {
 
       assert(node instanceof global.ConvolverNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.ConvolverNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.ConvolverNode(); }, TypeError);
     });
   });
 

test/DelayNode.js

@@ -24,10 +24,9 @@ describe("DelayNode", function() {
       assert.throws(function() {
         audioContext.createDelay(undefined);
       }, TypeError);
-
-      assert.throws(function() {
-        return new global.DelayNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.DelayNode(); }, TypeError);
     });
   });
 

test/DynamicsCompressorNode.js

@@ -12,10 +12,9 @@ describe("DynamicsCompressorNode", function() {
 
       assert(node instanceof global.DynamicsCompressorNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.DynamicsCompressorNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.DynamicsCompressorNode(); }, TypeError);
     });
   });
 

test/GainNode.js

@@ -12,10 +12,9 @@ describe("GainNode", function() {
 
       assert(node instanceof global.GainNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.GainNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.GainNode(); }, TypeError);
     });
   });
 

test/MediaElementAudioSourceNode.js

@@ -17,10 +17,9 @@ describe("MediaElementAudioSourceNode", function() {
       assert.throws(function() {
         audioContext.createMediaElementSource("INVALID");
       }, TypeError);
-
-      assert.throws(function() {
-        return new global.MediaElementAudioSourceNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.MediaElementAudioSourceNode(); }, TypeError);
     });
   });
 

test/MediaStreamAudioDestinationNode.js

@@ -12,10 +12,9 @@ describe("MediaStreamAudioDestinationNode", function() {
 
       assert(node instanceof global.MediaStreamAudioDestinationNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.MediaStreamAudioDestinationNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.MediaStreamAudioDestinationNode(); }, TypeError);
     });
   });
 

test/MediaStreamAudioSourceNode.js

@@ -17,10 +17,9 @@ describe("MediaStreamAudioSourceNode", function() {
       assert.throws(function() {
         audioContext.createMediaStreamSource("INVALID");
       }, TypeError);
-
-      assert.throws(function() {
-        return new global.MediaStreamAudioSourceNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.MediaStreamAudioSourceNode(); }, TypeError);
     });
   });
 

test/OfflineAudioCompletionEvent.js

@@ -20,13 +20,12 @@ describe("OfflineAudioCompletionEvent", function() {
       assert(event instanceof global.OfflineAudioCompletionEvent);
       assert(event instanceof global.Event);
 
-      assert.throws(function() {
-        return new global.OfflineAudioCompletionEvent();
-      }, TypeError);
-
       // test api
       assert(event.$name === "OfflineAudioCompletionEvent");
       assert(event.$node === node);
     });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.OfflineAudioCompletionEvent(); }, TypeError);
+    });
   });
 });

test/OscillatorNode.js

@@ -12,10 +12,9 @@ describe("OscillatorNode", function() {
 
       assert(node instanceof global.OscillatorNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.OscillatorNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.OscillatorNode(); }, TypeError);
     });
   });
 

test/PannerNode.js

@@ -12,10 +12,9 @@ describe("PannerNode", function() {
 
       assert(node instanceof global.PannerNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.PannerNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.PannerNode(); }, TypeError);
     });
   });
 

test/PeriodicWave.js

@@ -36,10 +36,9 @@ describe("PeriodicWave", function() {
       assert.throws(function() {
         audioContext.createPeriodicWave(f128, f8192);
       }, TypeError);
-
-      assert.throws(function() {
-        return new global.PeriodicWave();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.PeriodicWave(); }, TypeError);
     });
   });
 

test/ScriptProcessorNode.js

@@ -36,10 +36,9 @@ describe("ScriptProcessorNode", function() {
       assert.throws(function() {
         audioContext.createScriptProcessor(1024, 1, undefined);
       }, TypeError);
-
-      assert.throws(function() {
-        return new global.ScriptProcessorNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.ScriptProcessorNode(256, 1, 1); }, TypeError);
     });
   });
 

test/StereoPannerNode.js

@@ -20,10 +20,9 @@ describe("StereoPannerNode", function() {
 
       assert(node instanceof global.StereoPannerNode);
       assert(node instanceof global.AudioNode);
-
-      assert.throws(function() {
-        return new global.StereoPannerNode();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.StereoPannerNode(); }, TypeError);
     });
   });
 

test/dom/Element.js

@@ -6,12 +6,9 @@ describe("Element", function() {
       var element = new WebAudioTestAPI.Element();
 
       assert(element instanceof global.window.Element);
-
-      assert.throws(function() {
-        return new global.Element();
-      }, function(e) {
-        return e instanceof TypeError;
-      });
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.Element(); }, TypeError);
     });
   });
 });

test/dom/EventTarget.js

@@ -6,10 +6,9 @@ describe("EventTarget", function() {
       var target = new WebAudioTestAPI.EventTarget();
 
       assert(target instanceof global.window.EventTarget);
-
-      assert.throws(function() {
-        return new global.EventTarget();
-      }, TypeError);
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.EventTarget(); }, TypeError);
     });
   });
 

test/dom/HTMLElement.js

@@ -7,12 +7,9 @@ describe("HTMLElement", function() {
 
       assert(element instanceof global.window.HTMLElement);
       assert(element instanceof global.window.Element);
-
-      assert.throws(function() {
-        return new global.HTMLElement();
-      }, function(e) {
-        return e instanceof TypeError;
-      });
+    });
+    it("not work when 'new' directly", function() {
+      assert.throws(function() { new global.HTMLElement(); }, TypeError);
     });
   });
 });