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

Updated QUnit test cases and frameworks #4

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

Conversation

swapnilagarwwal
Copy link

No description provided.

@Borewit
Copy link

Borewit commented Jan 28, 2020

Your PR did not work for me, I am getting:

28 01 2020 21:48:06.681:INFO [karma-server]: Karma v4.4.1 server started at http://0.0.0.0:9876/
28 01 2020 21:48:06.686:INFO [launcher]: Launching browsers bs_chrome_windows, bs_iphoneX with concurrency unlimited
28 01 2020 21:48:06.728:INFO [launcher]: Starting browser chrome 72.0 (Windows 10) on BrowserStack
28 01 2020 21:48:06.729:INFO [launcher]: Starting browser iPhone X (ios 11.0) on BrowserStack
28 01 2020 21:48:06.733:ERROR [karma-server]: UncaughtException

The changes in karma.conf.js are fixing #7

@Borewit
Copy link

Borewit commented Jan 29, 2020

Blocked by: browserstack/browserstack-local-nodejs#96

@Borewit
Copy link

Borewit commented Feb 5, 2020

After removing (as suggest by BrowserStack Support) the executable:

$HOME/.browserstack/BrowserStackLocal

this PR worked for me.


QUnit.test("is 2 odd", function( assert ) {
assert.notOk(isOdd(2), 'two is an even number');
});
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
});
});

@@ -32,11 +31,3 @@ You can further customize configuration in karma.conf.js. For detailed reference
Execute the following command to run the karma tests:

`npm test`

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P

- Use Karmav0.12 or below and add `transports: ['websocket','jsonp-polling']` to the karma.conf.js file
- Checkout [karma issue] for more details.

[dashboard]:https://www.browserstack.com/automate

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sudo install

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.

4 participants