Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add an assertion on Element #35

Merged
merged 1 commit into from May 19, 2013
Merged

Add an assertion on Element #35

merged 1 commit into from May 19, 2013

Conversation

@fdubost
Copy link

@fdubost fdubost commented May 17, 2013

Add an assertion on Element to test if it has not empty content

@jubianchi

This comment has been minimized.

Indent here ?

@jubianchi
Copy link
Member

@jubianchi jubianchi commented May 17, 2013

Except a weird indent, it's OK for me ;)

I don't know why I didn't implement this along with the hasNoContent assertion... perhaps I was too lazy :)

Thank for the patch !

@stephpy
Copy link
Member

@stephpy stephpy commented May 17, 2013

+1, thanks for this work :)

FlorianLB added a commit that referenced this pull request May 19, 2013
Add an assertion on Element
@FlorianLB FlorianLB merged commit ba0b123 into atoum:master May 19, 2013
1 check passed
1 check passed
default The Travis CI build passed
Details
@FlorianLB
Copy link
Member

@FlorianLB FlorianLB commented May 19, 2013

Thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants