fix: add more tests

Author: almeidx

src/node-opus.cc

@@ -66,7 +66,7 @@ NodeOpusEncoder::NodeOpusEncoder(const CallbackInfo& args): ObjectWrap<NodeOpusE
 	}
 
 	this->application = OPUS_APPLICATION_AUDIO;
-	this->outPcm = new opus_int16[channels * MAX_FRAME_SIZE];
+	this->outPcm = new opus_int16[this->channels * MAX_FRAME_SIZE];
 }
 
 NodeOpusEncoder::~NodeOpusEncoder() {

tests/test.js

@@ -14,12 +14,21 @@ const { OpusEncoder } = require('../lib/index.js');
   const reEncoded = opus.encode(decoded);
 
   assert(decoded.length === 640, 'Decoded frame length is not 640');
-  assert(reEncoded.length === 45, 're encoded frame length is not 45');
+  assert(reEncoded.length === 45, 're-encoded frame length is not 45');
 }
 
 // Default values work
 {
   new OpusEncoder();
+  new OpusEncoder(undefined, undefined);
+}
+
+// Invalid values throw
+{
+  assert.throws(() => new OpusEncoder("16000", 1), /Expected rate to be a number/);
+  assert.throws(() => new OpusEncoder(null, "1"), /Expected rate to be a number/);
+  assert.throws(() => new OpusEncoder(16000, "1"), /Expected channels to be a number/);
+  assert.throws(() => new OpusEncoder(16000, null), /Expected channels to be a number/);
 }
 
 console.log('Passed');