Dude! Where are the tests?

[yan-foto]

No description provided.

[yan-foto]

• tests with jest/mocha
• lint with jslint/eslint

[mackignacio]

Can I do this test? I can do Jest unit testing.

[yan-foto]

Hey @mackignacio ! Thanks for the help, I really appreciate it. Please feel free to write some tests. The reason that I didn't write any test is not knowing the best approach to write integration tests for electron. Starting with unit tests, nonetheless, would be a good start!

[mackignacio]

@yan-foto Thanks!!!

[mackignacio]

@yan-foto I noticed that there are two functions to be tested and both of them are not testable. createHardresetHandler() only returns a function and the returned function return a void. Then the default exported function also returns a void. Can we make the function at least return something so we can make it testable.

[yan-foto]

Not so fast 😄 !

Maybe the structure is not suitable for unit tests, but it sure is for integration tests. We should mock electron interfaces and see if electron-reload responses correctly to file changes.

Does that make sense?

[mackignacio]

Yeah that's also what I wanted to do.