Skip to content
This repository has been archived by the owner on Jun 2, 2022. It is now read-only.

misc: fix tests #7

Closed
wants to merge 1 commit into from
Closed

misc: fix tests #7

wants to merge 1 commit into from

Conversation

kevmoo
Copy link
Contributor

@kevmoo kevmoo commented Jul 19, 2018

Fixes #6

@kevmoo kevmoo requested a review from nex3 July 19, 2018 23:49
# # Uncomment this line to run tests on the browser.
# - test: --platform firefox
- test: --platform vm
- test: --platform firefox
Copy link
Contributor

Choose a reason for hiding this comment

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

This isn't a large package, so these should be tested together to avoid wasting cycles on spinning up an unnecessary VM.

@@ -13,5 +13,5 @@ dependencies:
http: ^0.11.0

dev_dependencies:
fake_async: ^0.1.2
fake_async: ^1.0.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are you changing the lower bound?

@kevmoo kevmoo closed this May 6, 2019
@kevmoo kevmoo deleted the fix_tests branch May 6, 2019 00:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Tests are failing: _FakeTimer doesn't have a tick member
2 participants