search

envistaInteractive / eslint-config-enspire

ESLint configuration used at Enspire Commerce
http://www.enspirecommerce.com/
2 stars 0 forks source link

Add tests #1

Open onel0p3z opened 7 years ago

onel0p3z commented 7 years ago

There should be tests matching all rules define here

stivenson commented 7 years ago

hi all @onel0p3z @cperez08 @cristianPerez @zetogk , A question please. For test implied-eval rule, I couldn't do the test. Apparently this mechanism doesn't exist in nodejs 6. Because it's only applicable by execScript(), setTimeout() and setInterval() functions in a browser.. Do I try to simulate it? Adjunt video about this: video in google driver, about implied-eval problem

onel0p3z commented 7 years ago

@stivenson @zetogk @cperez08 @cristianPerez let's get together and talk about these tests tomorrow first thing in the morning.