Linting fixes.

Author: davidlehn

.eslintignore

@@ -0,0 +1 @@
+tests/setImmediate.js

lib/jsonld.js

@@ -87,6 +87,9 @@ const _browser = !_nodejs &&
 // attaches jsonld API to the given object
 const wrapper = function(jsonld) {
 
+/** Registered RDF dataset parsers hashed by content-type. */
+const _rdfParsers = {};
+
 /* Core API */
 
 /**
@@ -464,11 +467,10 @@ jsonld.frame = util.callbackify(async function(input, frame, options) {
 
   // if the unexpanded frame includes a key expanding to @graph, frame the
   // default graph, otherwise, the merged graph
-  let framed;
   // FIXME should look for aliases of @graph
   opts.merged = !('@graph' in frame);
   // do framing
-  framed = _frameMergedOrDefault(expanded, expandedFrame, opts);
+  const framed = _frameMergedOrDefault(expanded, expandedFrame, opts);
 
   // compact result (force @graph option to true, skip expansion,
   // check for linked embeds)
@@ -594,7 +596,8 @@ jsonld.fromRDF = util.callbackify(async function(dataset, options) {
     format: _isString(dataset) ? 'application/n-quads' : undefined
   });
 
-  let {format, rdfParser} = options;
+  const {format} = options;
+  let {rdfParser} = options;
 
   // handle special format
   if(format) {
@@ -969,9 +972,6 @@ jsonld.useDocumentLoader = function(type) {
     jsonld, Array.prototype.slice.call(arguments, 1));
 };
 
-/** Registered RDF dataset parsers hashed by content-type. */
-const _rdfParsers = {};
-
 /**
  * Registers an RDF dataset parser by content-type, for use with
  * jsonld.fromRDF. An RDF dataset parser will always be given two parameters,

lib/url.js

@@ -19,13 +19,15 @@ api.parsers = {
     keys: [
       'href', 'scheme', 'authority', 'path', 'query', 'fragment'
     ],
+    /* eslint-disable-next-line max-len */
     regex: /^(?:([^:\/?#]+):)?(?:\/\/([^\/?#]*))?([^?#]*)(?:\?([^#]*))?(?:#(.*))?/
   },
   full: {
     keys: [
       'href', 'protocol', 'scheme', 'authority', 'auth', 'user', 'password',
       'hostname', 'port', 'path', 'directory', 'file', 'query', 'fragment'
     ],
+    /* eslint-disable-next-line max-len */
     regex: /^(([^:\/?#]+):)?(?:\/\/((?:(([^:@]*)(?::([^:@]*))?)?@)?([^:\/?#]*)(?::(\d*))?))?(?:(((?:[^?#\/]*\/)*)([^?#]*))(?:\?([^#]*))?(?:#(.*))?)/
   }
 };

tests/earl-report.js

@@ -19,6 +19,7 @@ function EarlReport(options) {
       '0' + (today.getMonth() + 1) : today.getMonth() + 1) + '-' +
     (today.getDate() < 10 ? '0' + today.getDate() : today.getDate());
   this.id = options.id;
+  /* eslint-disable quote-props */
   this._report = {
     '@context': {
       'doap': 'http://usefulinc.com/ns/doap#',
@@ -66,6 +67,7 @@ function EarlReport(options) {
     },
     'subjectOf': []
   };
+  /* eslint-enable quote-props */
   this._report['@id'] += '#' + this.id;
   this._report['doap:name'] += ' ' + this.id;
   this._report['dc:title'] += ' ' + this.id;

tests/misc.js

@@ -89,6 +89,7 @@ describe('other toRDF tests', () => {
   it('should process with options and promise', done => {
     const p = jsonld.toRDF({}, {});
     assert(p instanceof Promise);
+    /* eslint-disable-next-line no-unused-vars */
     p.catch(e => {
       assert.fail();
     }).then(output => {
@@ -100,6 +101,7 @@ describe('other toRDF tests', () => {
   it('should process with no options and promise', done => {
     const p = jsonld.toRDF({});
     assert(p instanceof Promise);
+    /* eslint-disable-next-line no-unused-vars */
     p.catch(e => {
       assert.fail();
     }).then(output => {
@@ -109,6 +111,7 @@ describe('other toRDF tests', () => {
   });
 
   it('should fail with no args and callback', done => {
+    /* eslint-disable-next-line no-unused-vars */
     jsonld.toRDF((err, output) => {
       assert(err);
       done();
@@ -118,6 +121,7 @@ describe('other toRDF tests', () => {
   it('should fail with no args and promise', done => {
     const p = jsonld.toRDF();
     assert(p instanceof Promise);
+    /* eslint-disable-next-line no-unused-vars */
     p.then(output => {
       assert.fail();
     }).catch(e => {
@@ -127,6 +131,7 @@ describe('other toRDF tests', () => {
   });
 
   it('should fail for bad format and callback', done => {
+    /* eslint-disable-next-line no-unused-vars */
     jsonld.toRDF({}, {format: 'bogus'}, (err, output) => {
       assert(err);
       assert.equal(err.name, 'jsonld.UnknownFormat');
@@ -198,6 +203,7 @@ describe('other fromRDF tests', () => {
   it('should process with options and promise', done => {
     const p = jsonld.fromRDF(emptyNQuads, {});
     assert(p instanceof Promise);
+    /* eslint-disable-next-line no-unused-vars */
     p.catch(e => {
       assert.fail();
     }).then(output => {
@@ -209,6 +215,7 @@ describe('other fromRDF tests', () => {
   it('should process with no options and promise', done => {
     const p = jsonld.fromRDF(emptyNQuads);
     assert(p instanceof Promise);
+    /* eslint-disable-next-line no-unused-vars */
     p.catch(e => {
       assert.fail();
     }).then(output => {
@@ -218,6 +225,7 @@ describe('other fromRDF tests', () => {
   });
 
   it('should fail with no args and callback', done => {
+    /* eslint-disable-next-line no-unused-vars */
     jsonld.fromRDF((err, output) => {
       assert(err);
       done();
@@ -227,6 +235,7 @@ describe('other fromRDF tests', () => {
   it('should fail with no args and promise', done => {
     const p = jsonld.fromRDF();
     assert(p instanceof Promise);
+    /* eslint-disable-next-line no-unused-vars */
     p.then(output => {
       assert.fail();
     }).catch(e => {
@@ -236,6 +245,7 @@ describe('other fromRDF tests', () => {
   });
 
   it('should fail for bad format and callback', done => {
+    /* eslint-disable-next-line no-unused-vars */
     jsonld.fromRDF(emptyNQuads, {format: 'bogus'}, (err, output) => {
       assert(err);
       assert.equal(err.name, 'jsonld.UnknownFormat');

tests/node-document-loader-tests.js

@@ -3,6 +3,7 @@
  *
  * @author goofballLogic
  */
+/* eslint-disable quote-props */
 const jsonld = require('..');
 const assert = require('assert');
 
@@ -17,6 +18,7 @@ describe('For the node.js document loader', function() {
   describe('When built with no options specified', function() {
     it('loading should work', function(done) {
       jsonld.useDocumentLoader(documentLoaderType);
+      /* eslint-disable-next-line no-unused-vars */
       jsonld.expand('http://schema.org/', function(err, expanded) {
         assert.ifError(err);
         done();
@@ -55,7 +57,9 @@ describe('For the node.js document loader', function() {
       'Authorization': 'Bearer d783jkjaods9f87o83hj'
     };
 
-    it('loading should pass the headers through on the request', function(done) {
+    /* eslint-disable indent */
+    it('loading should pass the headers through on the request',
+      function(done) {
       jsonld.useDocumentLoader(documentLoaderType, options);
       requestMock.calls = [];
       const iri = 'http://some.thing.test.com/my-thing.jsonld';
@@ -75,9 +79,12 @@ describe('For the node.js document loader', function() {
         done();
       });
     });
+    /* eslint-enable indent */
   });
 
-  describe('When built using headers that already contain an Accept header', function() {
+  /* eslint-disable indent */
+  describe('When built using headers that already contain an Accept header',
+    function() {
     const options = {request: requestMock};
     options.headers = {
       'x-test-header-3': 'Third value',
@@ -99,5 +106,5 @@ describe('For the node.js document loader', function() {
       done();
     });
   });
-
+  /* eslint-enable indent */
 });

tests/test-common.js

@@ -836,7 +836,7 @@ function addTest(manifest, test, tests) {
               .on('error', err => {
                 reject(new Error(err));
               })
-              .on('complete', e => {
+              .on('complete', () => {
                 resolve();
               })
               .run({async: true});
@@ -1214,7 +1214,7 @@ function createDocumentLoader(test) {
     return p.then(readJson).then(json => {
       doc.document = json;
       return doc;
-    }).catch(err => {
+    }).catch(() => {
       throw {name: 'loading document failed', url};
     });
   }

tests/test-karma.js

@@ -22,6 +22,7 @@
  *
  * Copyright (c) 2011-2017 Digital Bazaar, Inc. All rights reserved.
  */
+/* global serverRequire */
 // FIXME: hack to ensure delay is set first
 mocha.setup({delay: true, ui: 'bdd'});
 
@@ -100,6 +101,7 @@ const options = {
   nodejs: false,
   assert,
   jsonld,
+  /* eslint-disable-next-line no-unused-vars */
   exit: code => {
     console.error('exit not implemented');
     throw new Error('exit not implemented');
@@ -126,6 +128,7 @@ const options = {
       return fs.outputFile(filename, data);
     });
   },
+  /* eslint-disable-next-line no-unused-vars */
   import: f => {
     console.error('import not implemented');
   }

tests/test-webidl.js

@@ -5,8 +5,12 @@
  *
  * Copyright (c) 2011-2017 Digital Bazaar, Inc. All rights reserved.
  */
+/* global IdlArray */
+/* global add_result_callback */
+/* global add_completion_callback */
+/* global done */
+/* eslint-disable no-var */
 const assert = require('chai').assert;
-const jsonld = require('..');
 
 require('./webidl/testharness.js');
 require('./webidl/WebIDLParser.js');
@@ -71,6 +75,7 @@ return new Promise((resolve, reject) => {
       done();
     };
   });
+  /* eslint-disable-next-line no-unused-vars */
   add_completion_callback(function(tests, status) {
     resolve(suite);
   });