Skip to content
New issue

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

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

Already on GitHub? # to your account

Use $timeout #9

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

Use $timeout #9

wants to merge 2 commits into from

Conversation

nate250
Copy link

@nate250 nate250 commented Dec 10, 2014

I tweaked the ng-websocket guts to use $interval and $timeout. This should allow for smoother unit testing - both for ng-websocket itself (as demonstrated) and for projects that use it.

Improve mocking and testing ability
I did x another test. I'm sure I could make that one and the pre-x'd test work with a little more time.
});
});

describe('Testing mock remote fixtures feature', function () {
xdescribe('Testing mock remote fixtures feature', function () {
Copy link
Owner

Choose a reason for hiding this comment

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

Why did you turn off this test?

@wilk
Copy link
Owner

wilk commented Dec 14, 2014

Hi @nate250 and thanks for contributing!
Let me ask you just a question: #9 (comment)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants