Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

chore(test): error tests fixed #5069

Merged
merged 2 commits into from
Dec 1, 2018
Merged

Conversation

cnishina
Copy link
Member

  • Fix a missing await on createNextTaskRunner where the recursive call
    should be awaited and if there are no new tasks to still resolve the
    promise. Things were passing previously probably because we were
    running tasks out of sync.
  • Add in errorTest portion of the test suite
  • Turn on the angular2 and unit tests.

- Fix a missing await on createNextTaskRunner where the recursive call
should be awaited and if there are no new tasks to still resolve the
promise. Things were passing previously probably because we were
running tasks out of sync.
- Add in errorTest portion of the test suite
- Turn on the angular2 and unit tests.
- Turn on browserstack test in Travis.
@cnishina cnishina merged commit 6c3c621 into angular:selenium4 Dec 1, 2018
cnishina added a commit to cnishina/protractor that referenced this pull request Dec 19, 2018
- Fix a missing await on createNextTaskRunner where the recursive call
should be awaited and if there are no new tasks to still resolve the
promise. Things were passing previously probably because we were
running tasks out of sync.
- Add in errorTest portion of the test suite.
- Turn on the angular2 and unit tests.
- Turn on browserstack test and remove ciNg2Conf test from Travis.
cnishina added a commit to cnishina/protractor that referenced this pull request Mar 23, 2019
- Fix a missing await on createNextTaskRunner where the recursive call
should be awaited and if there are no new tasks to still resolve the
promise. Things were passing previously probably because we were
running tasks out of sync.
- Add in errorTest portion of the test suite.
- Turn on the angular2 and unit tests.
- Turn on browserstack test and remove ciNg2Conf test from Travis.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants