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

docs(guide/Unit Testing): fixing the example for testing fiflter. #11410

Closed
wants to merge 1 commit into from

Conversation

gonengar
Copy link
Contributor

Hi there,
It seems that in the example which starts at line 256 there need to be an injection for $filter as in the previous example.

Hi there,
It seems that in the example which starts at line 256 there need to be an injection for $filter as in the previous example.
@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

it('returns 0 when given null', function() {
var length = $filter('length');
expect(length(null)).toEqual(0);
});

it('returns the correct value when given a string of chars', function() {
beforeEach(inject(function(_$filter_){
Copy link
Contributor

Choose a reason for hiding this comment

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

The second beforeEach should not be needed - beforeEachh is applied to every it inside it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The first one is fine then?

Copy link
Contributor

Choose a reason for hiding this comment

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

YEp!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Great. Happy to help.

@Narretz
Copy link
Contributor

Narretz commented Mar 25, 2015

One more thing before we can merge: Can you please sign the CLA?

@gonengar
Copy link
Contributor Author

Sure,
Where do I sign?

On Wed, Mar 25, 2015 at 8:02 PM, Martin Staffa notifications@github.com
wrote:

One more thing before we can merge: Can you please sign the CLA?


Reply to this email directly or view it on GitHub
#11410 (comment).

@googlebot
Copy link

CLAs look good, thanks!

@gonengar
Copy link
Contributor Author

I thing I have already signed it.
Can you see it?

On Wed, Mar 25, 2015 at 8:10 PM, gonen jerbi gonengar@gmail.com wrote:

Sure,
Where do I sign?

On Wed, Mar 25, 2015 at 8:02 PM, Martin Staffa notifications@github.com
wrote:

One more thing before we can merge: Can you please sign the CLA?


Reply to this email directly or view it on GitHub
#11410 (comment).

@gonengar
Copy link
Contributor Author

Sure.

CLAs look good, thanks!


Reply to this email directly or view it on GitHub
#11410 (comment).

@Narretz Narretz self-assigned this Apr 3, 2015
@Narretz Narretz closed this in c075126 Apr 27, 2015
Narretz pushed a commit that referenced this pull request Apr 27, 2015
Hi there,
It seems that in the example which starts at line 256 there needs to
be an injection for $filter as in the previous example.

Closes #11410
netman92 pushed a commit to netman92/angular.js that referenced this pull request Aug 8, 2015
Hi there,
It seems that in the example which starts at line 256 there needs to
be an injection for $filter as in the previous example.

Closes angular#11410
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.

3 participants