Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

docs(guide): improve Protractor test for bindings #9330

Closed
wants to merge 1 commit into from

Conversation

elgalu
Copy link
Contributor

@elgalu elgalu commented Sep 29, 2014

Main reason is this test should be using by.binding locator instead of manually span[ng-bind].
This needs Protractor >= 1.3.0 to work.

@btford
Copy link
Contributor

btford commented Sep 30, 2014

@juliemr is there a way to use by.binding and keep the expectation DRY?

@btford btford added this to the Backlog milestone Sep 30, 2014
@juliemr
Copy link
Member

juliemr commented Sep 30, 2014

Maybe we'd prefer:

nameBindings.each(function(elem) {
  expect(elem.getText()).toEqual('Max Karl Ernst Ludwig Planck (April 23, 1858 – October 4, 1947)');
});

@btford
Copy link
Contributor

btford commented Sep 30, 2014

Thanks @juliemr!

@elgalu can you update your PR accordingly?

This needs Protractor >= 1.3.0 to work.
@elgalu
Copy link
Contributor Author

elgalu commented Sep 30, 2014

Fixed!

@jeffbcross jeffbcross force-pushed the master branch 4 times, most recently from e8dc429 to e83fab9 Compare October 10, 2014 17:38
@Narretz Narretz self-assigned this Jan 26, 2016
Narretz pushed a commit to Narretz/angular.js that referenced this pull request Jan 26, 2016
@Narretz Narretz closed this in adcfa74 Jan 26, 2016
Narretz pushed a commit that referenced this pull request Jan 26, 2016
This needs Protractor >= 1.3.0 to work.

Closes #9330
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants