Merge pull request #52 from mcollina/fix/del_parse

Del should accept strings

Author: jmatsushita

index.js

@@ -215,6 +215,17 @@ function levelgraphJSONLD(db, jsonldOpts) {
   graphdb.jsonld.del = function(obj, options, callback) {
     var blanks = {};
 
+    if (typeof obj === 'string') {
+      try {
+        obj = JSON.parse(obj);
+      } catch (e) {
+        // Handle case where we're trying to delete by passing an IRI
+        if (!N3Util.isIRI(obj)) {
+          throw e
+        }
+      }
+    }
+
     if (typeof options === 'function') {
       callback = options;
       options = {};

test/del_spec.js

@@ -125,4 +125,20 @@ describe('jsonld.del', function() {
       });
     });
   });
+
+  it('should del obj passed as stringified JSON', function(done) {
+    var jld = {"@context": { "@vocab": "https://schema.org/"}, "name": "BigBlueHat"};
+
+    db.jsonld.put(JSON.stringify(jld), {preserve:true}, function() {
+      db.jsonld.del(JSON.stringify(jld), {preserve:true}, function(err) {
+        expect(err).to.not.exist;
+        db.get({}, function(err, triples) {
+          // getting the full db
+          expect(triples).to.have.length(1);
+          done();
+        });
+      });
+    });
+  });
+
 });