added eslint

.editorconfig

@@ -0,0 +1,7 @@
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+indent_style = space
+indent_size = 4

.eslintignore

@@ -0,0 +1,2 @@
+node_modules/**
+test/transpiled/**

.eslintrc.json

@@ -0,0 +1,22 @@
+{
+    "extends": "airbnb-base",
+    "root": true,
+    "rules": {
+        "no-console": 0,
+        "indent": [ "error", 4 ],
+        "no-underscore-dangle": "off"
+    },
+    "plugins": [
+        "jasmine",
+        "sinon",
+        "chai"
+    ],
+    "env": {
+        "jasmine": true,
+        "browser": true
+    },
+    "globals": {
+        "testHandler": true,
+        "sinon": true
+    }
+}

package.json

@@ -9,15 +9,23 @@
     "test": "tests"
   },
   "scripts": {
-    "test": "npm run build-tests && wct --plugin sauce",
-    "test-local": "npm run build-tests && wct -l chrome -p --skip-selenium-install",
+    "test": "npm run lint && npm run build-tests && wct --plugin sauce",
+    "test-local": "npm run lint && npm run build-tests && npm run wct",
     "build": "rollup -c",
-    "build-tests": "webpack"
+    "build-tests": "webpack",
+    "lint": "eslint src/**/*.js test/**/*.js",
+    "wct": "wct -l chrome -p --skip-selenium-install"
   },
   "devDependencies": {
     "babel-loader": "^7.1.2",
     "babel-plugin-external-helpers": "^6.22.0",
     "babel-preset-es2015": "^6.24.1",
+    "eslint": "^4.6.1",
+    "eslint-config-airbnb-base": "^12.0.0",
+    "eslint-plugin-chai": "0.0.1",
+    "eslint-plugin-import": "^2.7.0",
+    "eslint-plugin-jasmine": "^2.8.4",
+    "eslint-plugin-sinon": "^0.2.0",
     "rollup": "^0.49.2",
     "rollup-plugin-babel": "^3.0.2",
     "rollup-plugin-node-resolve": "^3.0.0",

src/elements/ags-view.js

@@ -1,9 +1,9 @@
-import {html} from 'lit-html';
-import {render} from 'lit-html/lib/lit-extended';
-import {PropertyAccessors} from '@polymer/polymer/lib/mixins/property-accessors';
-import {ViewTemplates} from '../template-registry';
+import { html } from 'lit-html';
+import { render } from 'lit-html/lib/lit-extended';
+import { PropertyAccessors } from '@polymer/polymer/lib/mixins/property-accessors';
+import { ViewTemplates } from '../template-registry';
 
-const defaultWrapper = (view) => html`
+const defaultWrapper = view => html`
 <style>
 :host {
     display: block;
@@ -14,9 +14,8 @@ ${view}`;
 
 const notFoundTemplate = html`<div>Template not found</div>`;
 
-export class AgsView extends PropertyAccessors(HTMLElement) {
-
-    constructor(){
+export default class AgsView extends PropertyAccessors(HTMLElement) {
+    constructor() {
         super();
 
         this.predicate = null;
@@ -31,7 +30,7 @@ export class AgsView extends PropertyAccessors(HTMLElement) {
             'object',
             'predicate',
             'templateScope',
-            'ignoreMissing'
+            'ignoreMissing',
         ];
     }
 
@@ -46,9 +45,9 @@ export class AgsView extends PropertyAccessors(HTMLElement) {
     _render() {
         let template;
 
-        if(this.object) {
+        if (this.object) {
             if (!this.shadowRoot) {
-                this.attachShadow({mode: 'open'});
+                this.attachShadow({ mode: 'open' });
             }
 
             template = ViewTemplates.getTemplate(this.object, this.predicate, this.templateScope);
@@ -71,8 +70,8 @@ export class AgsView extends PropertyAccessors(HTMLElement) {
 
         this.dispatchEvent(new CustomEvent('render', {
             detail: {
-                template
-            }
+                template,
+            },
         }));
     }
 }

src/index.js

@@ -1,7 +1,7 @@
 import './elements/ags-view';
-import {ViewTemplates, FormTemplates} from "./template-registry";
+import { ViewTemplates, FormTemplates } from './template-registry';
 
 export {
     ViewTemplates,
-    FormTemplates
-}
+    FormTemplates,
+};

src/template-registry/TemplateSelector.js

@@ -1,5 +1,5 @@
 export default class TemplateSelector {
-    constructor(){
+    constructor() {
         this.name = '';
         this._matchers = [];
     }

src/template-registry/TemplateSelectorBuilder.js

@@ -1,32 +1,25 @@
-import TemplateSelector from "./TemplateSelector";
-
-export class TemplateSelectorBuilder {
+import TemplateSelector from './TemplateSelector';
 
+export default class TemplateSelectorBuilder {
     constructor(registry) {
         this._registry = registry;
         this._selector = new TemplateSelector();
     }
 
     value(valueMatcher) {
-        this._selector._matchers.push((v, p, o) => {
-            return valueMatcher(v);
-        });
+        this._selector._matchers.push(v => valueMatcher(v));
 
         return this;
     }
 
     property(propertyMatcher) {
-        this._selector._matchers.push((v, p, s) => {
-            return propertyMatcher(p);
-        });
+        this._selector._matchers.push((v, p) => propertyMatcher(p));
 
         return this;
     }
 
     scope(scopeMatcher) {
-        this._selector._matchers.push((v, p, s) => {
-            return scopeMatcher(s);
-        });
+        this._selector._matchers.push((v, p, s) => scopeMatcher(s));
 
         return this;
     }

src/template-registry/index.js

@@ -1,4 +1,4 @@
-import {TemplateRegistry} from './template-registry';
+import TemplateRegistry from './template-registry';
 
 export const ViewTemplates = new TemplateRegistry();
 export const FormTemplates = new TemplateRegistry();

src/template-registry/template-registry.js

@@ -1,8 +1,6 @@
-import {TemplateResult} from 'lit-html';
-import {TemplateSelectorBuilder} from "./TemplateSelectorBuilder";
-
-export class TemplateRegistry {
+import TemplateSelectorBuilder from './TemplateSelectorBuilder';
 
+export default class TemplateRegistry {
     constructor() {
         this._templates = [];
     }
@@ -20,28 +18,26 @@ export class TemplateRegistry {
     }
 
     getTemplate(value, predicate, scope) {
-        const selectedTemplate = this._templates.find(template => {
-            return template.selector.matches(value, predicate, scope);
-        });
+        const selectedTemplate = this._templates.find(template =>
+            template.selector.matches(value, predicate, scope));
 
         return {
-          render: selectedTemplate.templateFunc,
-          name: selectedTemplate.name || null
+            render: selectedTemplate.templateFunc,
+            name: selectedTemplate.name || null,
         };
     }
 
     push(selector, templateFuncOrResult, name) {
         let templateFunc = templateFuncOrResult;
 
-        if(typeof templateFunc !== 'function') {
+        if (typeof templateFunc !== 'function') {
             templateFunc = () => templateFuncOrResult;
         }
 
         this._templates.push({
             selector,
             templateFunc,
-            name
+            name,
         });
     }
 }
-

test/.eslintrc.json

@@ -0,0 +1,5 @@
+{
+    "rules": {
+        "no-unused-expressions": "off"
+    }
+}

test/_TestHelpers.js

@@ -1,26 +1,28 @@
+/* eslint-disable */
+
 function setInputValue(input, text) {
     input.value = text;
     input.dispatchEvent(new Event('input'));
 }
 
 (function (Mocha) {
-  function extendInterfaceWithFixture (interfaceName) {
-    var originalInterface = Mocha.interfaces[interfaceName];
+    function extendInterfaceWithFixture(interfaceName) {
+        var originalInterface = Mocha.interfaces[interfaceName];
 
-    Mocha.interfaces[interfaceName] = function (suite) {
-      originalInterface.apply(this, arguments);
+        Mocha.interfaces[interfaceName] = function (suite) {
+            originalInterface.apply(this, arguments);
 
-      suite.on('pre-require', function (context, file, mocha) {
-        context.testHandler = function (target, event, handler) {
+            suite.on('pre-require', function (context, file, mocha) {
+                context.testHandler = function (target, event, handler) {
 
-          target.addEventListener(event, handler);
-          target.addEventListener(event, function() {
-            target.removeEventListener(event, handler);
-          });
+                    target.addEventListener(event, handler);
+                    target.addEventListener(event, function () {
+                        target.removeEventListener(event, handler);
+                    });
+                };
+            });
         };
-      });
-    };
-  }
+    }
 
-  Object.keys(Mocha.interfaces).forEach(extendInterfaceWithFixture);
-})(this.Mocha);
+    Object.keys(Mocha.interfaces).forEach(extendInterfaceWithFixture);
+})(this.Mocha);

test/elements/ags-view.js

@@ -1,9 +1,8 @@
+import { html } from 'lit-html';
 import '../../src/elements/ags-view';
-import {ViewTemplates} from '../../src/template-registry';
-import {html} from "lit-html";
+import { ViewTemplates } from '../../src/template-registry';
 
 describe('ags-view', () => {
-
     let agsView;
     let getTemplate;
 
@@ -27,28 +26,25 @@ describe('ags-view', () => {
 
     it('should render found template', (done) => {
         getTemplate.returns({
-            render: (object) => html`<span>${object.value}</span>`
+            render: object => html`<span>${object.value}</span>`,
         });
 
         testHandler(agsView, 'render', () => {
             const span = agsView.shadowRoot.querySelector('span');
 
-            assert.equal(span.textContent, 'test');
+            expect(span.textContent).to.equal('test');
             done();
         });
 
         agsView.object = {
             '@id': 'test',
-            value: 'test'
+            value: 'test',
         };
     });
 
     describe('rendering nested templates', () => {
-
         it('should render', () => {
 
         });
-
     });
-
 });

test/templates/template-registry.js

@@ -1,28 +1,24 @@
-import {TemplateRegistry} from '../../src/template-registry/template-registry'
-import {html, render} from 'lit-html';
+import { html, render } from 'lit-html';
+import TemplateRegistry from '../../src/template-registry/template-registry';
 
 describe('Template Registry', () => {
-
     let registry;
 
     const matchAllSelector = {
-        matches: () => true
+        matches: () => true,
     };
 
     beforeEach(() => {
         registry = new TemplateRegistry();
     });
 
     describe('initially', () => {
-
         it('should be empty', () => {
             expect(registry.count).to.be.equal(0);
         });
-
     });
 
     describe('when adding selectors', () => {
-
         it('should count them', () => {
             // given
             registry.when.value(() => true).renders(html``);
@@ -32,11 +28,9 @@ describe('Template Registry', () => {
             // then
             expect(registry.count).to.be.equal(3);
         });
-
     });
 
     describe('when selecting template', () => {
-
         const templateFunc = () => html``;
 
         it('should return name and template func', () => {
@@ -78,5 +72,4 @@ describe('Template Registry', () => {
             expect(renderTarget.textContent).to.equal('test');
         });
     });
-
 });