Update step_03.ngdoc

[fmagnan]

Hello,

when I run protactor, I got this error message:

Failures:
1) PhoneCat App Phone list view should display the current filter value in the title bar
Message: ReferenceError: query is not defined
Stacktrace: ReferenceError: query is not defined

It's because query is not retrieved from model. So I added it.

Hope this helps

[caitp]

This test is literally supposed to fail, please read the sentence following this code snippet

[caitp]

@petebacondarwin I am pretty sure the reference error is not the reason the test is supposed to fail though, we probably should put the query declaration after browser.get() in beforeEach()

[petebacondarwin]

Protractor doesn't care about these declarations being inside beforeEach since they are executed lazily when you put them inside a matcher - which is why we can create PageObjects for protractor specs.

I suspect that people are not copying the declaration of the query variable into their own code. So if it makes people less likely to complain that it is broken we should move it.

Also we should make the fact that this test is expected to fail more obvious no? Perhaps put it in the sentence before the code.

[fmagnan]

Hello,
thank you very much for your reply. The test is supposed to fail because we put query in title and title tag is not in scope. So, I did what the documentation said and I moved ng-controller in html tag and protractor still failed.

I think it's because query is not defined in protactor test. If you look at the previous test ("should filter the phone list as user types into the search box") you can see that query is retrieved from model.

So, I did the same and protractor run without errors.

[mary-poppins]

I'm sorry, but I wasn't able to verify your Contributor License Agreement (CLA) signature. CLA signature is required for any code contributions to AngularJS.

Please sign our CLA and ensure that the CLA signature email address and the email address in this PR's commits match.

If you signed the CLA as a corporation, please let us know the company's name.

Thanks a bunch!

PS: If you signed the CLA in the past then most likely the email addresses don't match. Please sign the CLA again or update the email address in the commit of this PR.
PS2: If you are a Googler, please sign the CLA as well to simplify the CLA verification process.

[mary-poppins]

CLA signature verified! Thank you!

Someone from the team will now triage your PR and it will be processed based on the determined priority (doc updates and fixes with tests are prioritized over other changes).

[gkalpak]

Closed with c2033d7.