search

acrosman / Salesforce2Sql

Electron app to convert a Salesforce org's schema into a similarly shaped database.
MIT License
14 stars 8 forks source link

Extend Jest Tests #63

Open acrosman opened 3 years ago

acrosman commented 3 years ago

Summary With the merge of #49 which closed #48, we now have some test coverage. Bu there is only a little test coverage at the moment. Over time we'll work toward more.

This issue is for generally reminding of the need to include more tests. Additional sub-issues and related issues will be added for specific requests. If you'd like to add tests generically, you may want to look at the functions in src/sf_calls.js and src/preferences.js that do not callback to the main window directly (the project already includes rewire to make it easy to access non-exported features of the modules).

There are also no tests currently covering the render JS files, which are fairly extensive as well.

Specific Testing Issues

imrishabh18 commented 3 years ago

I have started to learn jest and wanted to contribute to some open source projects. What do you think would be best to start with here @acrosman ?

acrosman commented 3 years ago

Any one of:

All three have a pattern to review from other tests so you aren't starting totally from scratch. The last one is a bit more of a free space to run since it involves starting a whole new test file and the least figuring out how existing code works. The first two have clearer patterns to follow, but also require reading and understanding the code you're testing more deeply.