Skip to content

Commit 2f2c374

Browse files
committed
style(eslint): Enforce linting on all code
1 parent 8c8c6b4 commit 2f2c374

File tree

8 files changed

+9
-13
lines changed

8 files changed

+9
-13
lines changed

.eslintignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,5 +2,4 @@ blueprints/**/files/**
22
coverage/**
33
node_modules/**
44
dist/**
5-
*.spec.js
65
src/index.html

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
"scripts": {
1111
"clean": "rimraf dist",
1212
"compile": "better-npm-run compile",
13-
"lint": "eslint src tests server",
13+
"lint": "eslint bin build config server src tests",
1414
"lint:fix": "npm run lint -- --fix",
1515
"start": "better-npm-run start",
1616
"dev": "better-npm-run dev",

tests/components/Counter/Counter.spec.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ describe('(Component) Counter', () => {
5454

5555
_spies.dispatch.should.have.been.called
5656
_spies.increment.should.have.been.called
57-
});
57+
})
5858
})
5959

6060
describe('A Double (Async) button...', () => {
@@ -75,6 +75,6 @@ describe('(Component) Counter', () => {
7575

7676
_spies.dispatch.should.have.been.called
7777
_spies.doubleAsync.should.have.been.called
78-
});
78+
})
7979
})
8080
})

tests/components/Header/Header.spec.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ describe('(Component) Header', () => {
88
let _wrapper
99

1010
beforeEach(() => {
11-
_wrapper = shallow(<Header/>)
11+
_wrapper = shallow(<Header />)
1212
})
1313

1414
it('Renders a welcome message', () => {
@@ -18,7 +18,6 @@ describe('(Component) Header', () => {
1818
})
1919

2020
describe('Navigation links...', () => {
21-
2221
it('Should render a Link to Home route', () => {
2322
expect(_wrapper.contains(
2423
<IndexLink activeClassName={classes.activeRoute} to='/'>

tests/framework.spec.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,16 +23,16 @@ describe('(Framework) Karma Plugins', function () {
2323
})
2424

2525
it('Should have chai-as-promised helpers.', function () {
26-
const pass = new Promise(res => res('test'))
27-
const fail = new Promise((res, rej) => rej())
26+
const pass = new Promise(resolve => resolve('test'))
27+
const fail = new Promise((resolve, reject) => reject())
2828

2929
return Promise.all([
3030
expect(pass).to.be.fulfilled,
3131
expect(fail).to.not.be.fulfilled
3232
])
3333
})
3434

35-
it('should have chai-enzyme working', function() {
35+
it('should have chai-enzyme working', function () {
3636
let wrapper = shallow(<Fixture />)
3737
expect(wrapper.find('#checked')).to.be.checked()
3838

tests/routes/Counter/index.spec.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,10 @@ describe('(Route) Counter', () => {
88
})
99

1010
it('Should return a route configuration object', () => {
11-
expect(typeof(_route)).to.equal('object')
11+
expect(typeof _route).to.equal('object')
1212
})
1313

1414
it('Configuration should contain path `counter`', () => {
1515
expect(_route.path).to.equal('counter')
1616
})
17-
1817
})

tests/routes/Home/components/HomeView.spec.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,5 +20,4 @@ describe('(View) Home', () => {
2020
expect(duck).to.exist
2121
expect(duck.attr('alt')).to.match(/This is a duck, because Redux!/)
2222
})
23-
2423
})

tests/routes/Home/index.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ describe('(Route) Home', () => {
88
})
99

1010
it('Should return a route configuration object', () => {
11-
expect(typeof(HomeRoute)).to.equal('object')
11+
expect(typeof HomeRoute).to.equal('object')
1212
})
1313

1414
it('Should define a route component', () => {

0 commit comments

Comments
 (0)