fix linting errors

.eslintignore

@@ -3,5 +3,5 @@ dynamic/**/*
 generator/**/*
 utils/**/*
 coverage/**/*
-modules/dynamic/propertyPrefixMap.js
-modules/static/propertyPrefixMap.js
+modules/dynamic/dynamicData.js
+modules/static/staticData.js

.istanbul.yml

@@ -0,0 +1,2 @@
+instrumentation:
+  excludes: ['modules/generator/**']

modules/dynamic/index.js

@@ -11,6 +11,9 @@ import position from './plugins/position'
 import sizing from './plugins/sizing'
 import transition from './plugins/transition'
 
+import prefixAll from '../static'
+import { prefixMap } from './dynamicData'
+
 const plugins = [
   crossFade,
   cursor,
@@ -24,9 +27,6 @@ const plugins = [
   flex
 ]
 
-import prefixAll from '../static'
-import { prefixMap } from './dynamicData'
-
 const Prefixer = createPrefixer(
   {
     prefixMap,

modules/generator/generateDynamicPrefixMap.js

@@ -32,14 +32,6 @@ const flexPropsIE = [
   'flexBasis'
 ]
 
-function filterAndRemoveIfEmpty(map: Object, property: string, filter: Function): void {
-  map[property] = map[property].filter(filter)
-
-  if (map[property].length === 0) {
-    delete map[property]
-  }
-}
-
 export default function generateDynamicPrefixMap(browserList: Object): Object {
   const prefixMap = {}
 

modules/generator/generateStaticPrefixMap.js

@@ -19,8 +19,6 @@ const prefixBrowserMap = {
   ie_mob: 'ms'
 }
 
-const browsers = Object.keys(prefixBrowserMap)
-
 // remove flexprops from IE
 const flexPropsIE = [
   'alignContent',

test/dynamic/createPrefixer-test.js

@@ -55,20 +55,15 @@ describe('Dynamic Prefixer', () => {
     })
   })
   describe('Running on android < 4.4', () => {
-    it(
-      'should use the osversion if its the native browser to check for required props',
-      () => {
-        const andPrefixer = new Prefixer({ userAgent: Android4_4_4 })
-        console.log(andPrefixer._browserInfo)
-        expect(
-          andPrefixer._browserInfo.browserVersion
-        ).to.eql(andPrefixer._browserInfo.osVersion)
-        expect(andPrefixer._browserInfo.browserVersion).to.eql(4.4)
-        const transform = { transform: 'rotate(40deg)' }
-        const output = { WebkitTransform: 'rotate(40deg)' }
-        expect(new Prefixer({ userAgent: Android4_4_4 }).prefix(transform)).to.eql(output)
-      }
-    )
+    it('should use the osversion if its the native browser to check for required props', () => {
+      const andPrefixer = new Prefixer({ userAgent: Android4_4_4 })
+      console.log(andPrefixer._browserInfo)
+      expect(andPrefixer._browserInfo.browserVersion).to.eql(andPrefixer._browserInfo.osVersion)
+      expect(andPrefixer._browserInfo.browserVersion).to.eql(4.4)
+      const transform = { transform: 'rotate(40deg)' }
+      const output = { WebkitTransform: 'rotate(40deg)' }
+      expect(new Prefixer({ userAgent: Android4_4_4 }).prefix(transform)).to.eql(output)
+    })
     it('should use the chrome version if its chrome to check for required props', () => {
       const andPrefixer = new Prefixer({ userAgent: Android4_2_2Chrome47 })
       expect(andPrefixer._browserInfo.osVersion).to.eql(4.2)
@@ -84,9 +79,7 @@ describe('Dynamic Prefixer', () => {
     it('always force Safari prefixing as iOS forces to use Safari under the hood', () => {
       const iosPrefixer = new Prefixer({ userAgent: iOSChrome47 })
       console.log(iosPrefixer._browserInfo)
-      expect(
-        iosPrefixer._browserInfo.browserVersion
-      ).to.eql(iosPrefixer._browserInfo.osVersion)
+      expect(iosPrefixer._browserInfo.browserVersion).to.eql(iosPrefixer._browserInfo.osVersion)
       const input = { display: 'flex' }
       const output = { display: '-webkit-flex' }
       expect(iosPrefixer.prefix(input)).to.eql(output)
@@ -223,9 +216,7 @@ describe('Dynamic Prefixer', () => {
   })
   describe('Prefixing keyframes', () => {
     it('should return the correct keyframes string', () => {
-      expect(
-        new Prefixer({ userAgent: Chrome14 }).prefixedKeyframes
-      ).to.eql('-webkit-keyframes')
+      expect(new Prefixer({ userAgent: Chrome14 }).prefixedKeyframes).to.eql('-webkit-keyframes')
       expect(new Prefixer({ userAgent: Chrome49 }).prefixedKeyframes).to.eql('keyframes')
       expect(new Prefixer({ userAgent: Chromium }).prefixedKeyframes).to.eql('keyframes')
       expect(new Prefixer({ userAgent: PhantomJS }).prefixedKeyframes).to.eql('keyframes')
@@ -238,6 +229,7 @@ describe('Dynamic Prefixer', () => {
       expect(new Prefixer({ userAgent: Chrome22 }).prefix(input)).to.eql(prefixed)
     })
   })
+
   describe('Prefixing transitions', () => {
     it('should add prefixes to properties in value', () => {
       const input = { transition: 'appearance 200ms linear' }
@@ -379,17 +371,11 @@ describe('Dynamic Prefixer', () => {
   })
   describe('Prefixing display', () => {
     it('should not remove display property', () => {
-      expect(
-        new Prefixer({ userAgent: MSIE10 }).prefix({ display: 'block' })
-      ).to.eql({ display: 'block' })
+      expect(new Prefixer({ userAgent: MSIE10 }).prefix({ display: 'block' })).to.eql({ display: 'block' })
     })
     it('should not throw if display is null or undefined', () => {
-      expect(
-        new Prefixer({ userAgent: Chrome45 }).prefix({ display: null })
-      ).to.eql({ display: null })
-      expect(
-        new Prefixer({ userAgent: Chrome45 }).prefix({ display: undefined })
-      ).to.eql({ display: undefined })
+      expect(new Prefixer({ userAgent: Chrome45 }).prefix({ display: null })).to.eql({ display: null })
+      expect(new Prefixer({ userAgent: Chrome45 }).prefix({ display: undefined })).to.eql({ display: undefined })
     })
   })
   describe('Using Prefixer.prefixAll', () => {