FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Updated QUnit test cases and frameworks by swapnilagarwwal · Pull Request #4 · browserstack/karma-browserstack-example · GitHub

/ karma-browserstack-example Public

Updated QUnit test cases and frameworks - #4

Open
swapnilagarwwal wants to merge 3 commits into
browserstack:masterfrom
swapnilagarwwal:master
ConversationCommits (3)ChecksFiles changed
Open

Updated QUnit test cases and frameworks#4
swapnilagarwwal wants to merge 3 commits into
browserstack:masterfrom
swapnilagarwwal:master

Conversation

swapnilagarwwal commented Mar 13, 2019

Copy link
Copy Markdown

No description provided.

Borewit commented Jan 28, 2020 •
edited
Loading

Copy link
Copy Markdown

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 mentioned this pull request Jan 29, 2020
Open

Borewit commented Jan 29, 2020

Copy link
Copy Markdown

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

Borewit commented Feb 5, 2020

Copy link
Copy Markdown

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

$HOME/.browserstack/BrowserStackLocal

this PR worked for me.

francisf reviewed Feb 6, 2020
View reviewed changes
Comment thread tests/test.js

QUnit.test("is 2 odd", function( assert ) {
assert.notOk(isOdd(2), 'two is an even number');
}); No newline at end of file

francisf Feb 6, 2020

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality
Suggested change
});
});

Borewit mentioned this pull request Mar 1, 2020
Open
ajlchiban approved these changes Jun 18, 2024
View reviewed changes
Comment thread README.md
@@ -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`

ajlchiban Jun 18, 2024

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

P

Comment thread README.md
- 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

ajlchiban Jun 18, 2024

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Sudo install

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

francisf francisf left review comments
+1 more reviewer
ajlchiban ajlchiban approved these changes
Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL