Skip to content

[WIP] Add tests #141

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

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

[WIP] Add tests #141

wants to merge 3 commits into from

Conversation

simison
Copy link
Contributor

@simison simison commented Nov 15, 2016

Work in progress:

  • Add Karma test runner
  • Refactor injected interceptor to a separate service (requestService) for easier testing
  • Test that injecting said service works
  • Test that authService can be initialised
  • Test that httpBuffer can be initialised
  • Test that invoking loginConfirmed and loginCancelled broadcasts events
  • Test that invoking loginConfirmed and loginCancelled broadcasts events with data
  • Test that httpBuffer's methods retryAll, rejectAll, and append work
  • Test that everything works via $http using (mock using $httpBackend)

Not sure when I'll have time next time with this, so if anyone feels like continuing from here, do it! :-)

@simison
Copy link
Contributor Author

simison commented Nov 15, 2016

Ah, damn, noticed only now #69 — will look into combining these.

@simison
Copy link
Contributor Author

simison commented Nov 15, 2016

Included event tests by @nils-wisiol from #69

# 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.

1 participant