fix(encoder): correct return value on .write

Due to not returning the success value on `.write`, the encoding was
not finished.

Closes #27

Author: dignifiedquire

src/encoder.js

@@ -372,10 +372,10 @@ class Encoder {
    * Alias for `.pushAny`
    *
    * @param {*} obj
-   * @returns {undefind}
+   * @returns {boolean} true on success
    */
   write (obj) {
-    this.pushAny(obj)
+    return this.pushAny(obj)
   }
 
   /**
@@ -500,7 +500,10 @@ class Encoder {
    */
   static encode (o) {
     const enc = new Encoder()
-    enc.pushAny(o)
+    const ret = enc.pushAny(o)
+    if (!ret) {
+      throw new Error('Failed to encode input')
+    }
 
     return enc.finalize()
   }

test/encoder.spec.js

@@ -172,4 +172,49 @@ describe('encoder', () => {
       }
     })
   })
+
+  describe('nested classes', () => {
+    class Serializable {
+      serialize () {
+        const encoder = new cbor.Encoder()
+        const ret = this.encodeCBOR(encoder)
+        if (!ret) {
+          throw new Error('unable to serialize input')
+        }
+        return encoder.finalize()
+      }
+
+      static deserialize (serialized) {
+        return cbor.decodeAll(serialized)
+      }
+    }
+
+    class TestA extends Serializable {
+      encodeCBOR (gen) {
+        return gen.write(44)
+      }
+    }
+
+    class TestB extends Serializable {
+      encodeCBOR (gen) {
+        return gen.write([
+          new TestA(),
+          true
+        ])
+      }
+    }
+
+    it('flat', () => {
+      const a1 = new TestA()
+
+      expect(a1.serialize()).to.be.eql(Buffer.from([24, 44]))
+      expect(TestA.deserialize(a1.serialize())).to.be.eql([44])
+    })
+
+    it('nested', () => {
+      const b1 = new TestB()
+      const encoded = b1.serialize()
+      expect(TestB.deserialize(encoded)).to.be.eql([[44, true]])
+    })
+  })
 })