Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wip: test preload files #199

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

wip: test preload files #199

wants to merge 1 commit into from

Conversation

cloverich
Copy link
Owner

wip commit for testing preload files with electron-mocha. I think the test build is correct, but it wont run with regular mocha because the node version differs beween eletron and local node. I could rebuild for test vs dev but would be slow and maybe fragile constantly going back and forth. One alternative is to use electrons version of node; you can do this but need to then call mocha and do some electron setup; this is what elecgtron-mocha does. But as of this commit its not working -- it doesnt find the test files for one reason or another (outputs 0 tests ran).

Some other changes:

  • Re-name scripts in /scripts to match package.json (test -> pretest, start -> dev)
  • Refactor database setup, so it can be called in tests (in short, preload tests need a database)

Parking as WIP until I have a solid chunk of time to dig in further.

Closes #183

- wip commit for testing preload files with electron-mocha. I think the test build is correct, but it wont run with regular mocha because the node version differs beween eletron and local node. I could rebuild for test vs dev but would be slow and maybe fragile constantly going back and forth. One alternative is to use electrons version of node; you can do this but need to then call mocha and do some electron setup; this is what elecgtron-mocha does. But as of this commit its not working -- it doesnt find the test files for one reason or another (outputs 0 tests ran).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test setup for prebuild files
1 participant