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

feat(): native component testing with spectron 🚀 #544

Open
wants to merge 983 commits into
base: master
Choose a base branch
from

Conversation

MarkGeeRomano
Copy link
Contributor

No description provided.

@vipyne
Copy link
Contributor

vipyne commented Apr 22, 2019


> [email protected] test /Users/vipyne/Documents/littlstar/ara/ara-file-manager
> ava test/renderer

/Users/vipyne/Documents/littlstar/ara/ara-file-manager/test/renderer/native-components/fileSelector/main.js:7
app.on('ready', () => {
    ^

TypeError: Cannot read property 'on' of undefined
    at Object.on (/Users/vipyne/Documents/littlstar/ara/ara-file-manager/test/renderer/native-components/fileSelector/main.js:5:5)
    at Module._compile (internal/modules/cjs/loader.js:799:30)
    at extensions.(anonymous function) (/Users/vipyne/Documents/littlstar/ara/ara-file-manager/node_modules/require-precompiled/index.js:13:11)
    at Object.require.extensions.(anonymous function) [as .js] (/Users/vipyne/Documents/littlstar/ara/ara-file-manager/node_modules/ava/lib/process-adapter.js:100:4)
    at Module.load (internal/modules/cjs/loader.js:666:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:606:12)
    at Function.Module._load (internal/modules/cjs/loader.js:598:3)
    at Module.require (internal/modules/cjs/loader.js:705:19)
    at require (internal/modules/cjs/helpers.js:14:16)
    at Object.<anonymous> (/Users/vipyne/Documents/littlstar/ara/ara-file-manager/node_modules/ava/lib/test-worker.js:62:1)
/Users/vipyne/Documents/littlstar/ara/ara-file-manager/test/renderer/native-components/afsFileTable/afsFileRow/main.js:7
app.on('ready', () => {
    ^

TypeError: Cannot read property 'on' of undefined
    at Object.on (/Users/vipyne/Documents/littlstar/ara/ara-file-manager/test/renderer/native-components/afsFileTable/afsFileRow/main.js:5:5)
    at Module._compile (internal/modules/cjs/loader.js:799:30)
    at extensions.(anonymous function) (/Users/vipyne/Documents/littlstar/ara/ara-file-manager/node_modules/require-precompiled/index.js:13:11)
    at Object.require.extensions.(anonymous function) [as .js] (/Users/vipyne/Documents/littlstar/ara/ara-file-manager/node_modules/ava/lib/process-adapter.js:100:4)
    at Module.load (internal/modules/cjs/loader.js:666:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:606:12)
    at Function.Module._load (internal/modules/cjs/loader.js:598:3)
    at Module.require (internal/modules/cjs/loader.js:705:19)
    at require (internal/modules/cjs/helpers.js:14:16)
    at Object.<anonymous> (/Users/vipyne/Documents/littlstar/ara/ara-file-manager/node_modules/ava/lib/test-worker.js:62:1)

  58 passed
  2 exceptions

  ✖ test/renderer/native-components/fileSelector/main.js exited with a non-zero exit code: 1

  ✖ test/renderer/native-components/afsFileTable/afsFileRow/main.js exited with a non-zero exit code: 1
npm ERR! Test failed.  See above for more details.```

@MarkGeeRomano
Copy link
Contributor Author

MarkGeeRomano commented Apr 22, 2019

@vipyne I need to configure ava so there is a single point of entry to all tests, like you tried to do. But for now, to get the new tests to run:

ava test/renderer/native-components/afsFileTable/afsFileRow/index.js
ava test/renderer/native-components/fileSelector/index.js

@vipyne
Copy link
Contributor

vipyne commented Apr 22, 2019

can we add that to the npm run script?

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.

7 participants