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

feat(scripts): add framework to test protractor #1450

Closed
wants to merge 1 commit into from

Conversation

hankduan
Copy link
Contributor

No description provided.

@juliemr juliemr added cla: yes and removed cla: no labels Oct 16, 2014
@hankduan hankduan force-pushed the testingProtractor branch 2 times, most recently from a65b2f3 to a9e7c51 Compare October 27, 2014 20:21
@hankduan
Copy link
Contributor Author

hankduan commented Nov 7, 2014

fixed this PR to work with #1485.


var scripts = [
var passing_tests = [
Copy link
Member

Choose a reason for hiding this comment

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

prefer passingTests

@juliemr
Copy link
Member

juliemr commented Nov 7, 2014

Left a bunch of small comments, but I think overall this is good.

@hankduan
Copy link
Contributor Author

hankduan commented Nov 7, 2014

I think I accidentally change the hash id, so it's recreating a separate PR instead of doing it on top of this one. Closing this, and continuing here: #1505 (all comments addressed)

@hankduan hankduan closed this Nov 7, 2014
@hankduan hankduan deleted the testingProtractor branch November 26, 2014 21:38
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.

2 participants