Merge pull request #51 from ipld/string_fix

fix: support all forms of json
ipld · Apr 4, 2017 · 6b14373 · 6b14373
2 parents c6f7c18 + f179108
commit 6b14373
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/src/util.js b/src/util.js
@@ -34,7 +34,13 @@ const decoder = new cbor.Decoder({
 })
 
 function replaceCIDbyTAG (dagNode) {
-  if (isCircular(dagNode)) {
+  let circular
+  try {
+    circular = isCircular(dagNode)
+  } catch (e) {
+    circular = false
+  }
+  if (circular) {
     throw new Error('The object passed has circular references')
   }
 

diff --git a/test/util.spec.js b/test/util.spec.js
@@ -62,6 +62,16 @@ describe('util', () => {
     })
   })
 
+  it('strings', (done) => {
+    dagCBOR.util.cid('some test string', (err, cid) => {
+      expect(err).to.not.exist()
+      expect(cid.version).to.equal(1)
+      expect(cid.codec).to.equal('dag-cbor')
+      expect(cid.multihash).to.exist()
+      done()
+    })
+  })
+
   it.skip('serialize and deserialize - garbage', (done) => {
     const inputs = []