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.

Sign up for GitHub

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

Jump to bottom

Added some tests to new wrapping image #59

Closed
wants to merge 1 commit into from
Closed

Added some tests to new wrapping image #59

wants to merge 1 commit into from

Conversation

brunocasado
Copy link
Contributor

Tests #22

@gabrielbull
Copy link
Member

Tests should be made as Unit tests, so we can automatically test it and not break it with future updates.

@Progi1984 Progi1984 added this to the 0.7.2 milestone Feb 25, 2014
@Progi1984
Copy link
Member

@brunocasado Could you add some unit tests and push this pull request to the develop branch, please ?

@Progi1984 Progi1984 modified the milestones: 0.9.0, 0.9.1 Mar 24, 2014
@Progi1984 Progi1984 added the QA label Mar 26, 2014
@Progi1984 Progi1984 self-assigned this Mar 26, 2014
@Progi1984 Progi1984 removed their assignment Apr 11, 2014
@Progi1984
Copy link
Member

Someone could help to create unit test for that ?

@ivanlanin ivanlanin self-assigned this Apr 12, 2014
@ivanlanin
Copy link
Contributor

Will do

@ivanlanin
Copy link
Contributor

Merged with modification: 89f4288

@ivanlanin ivanlanin closed this Apr 13, 2014
@Progi1984
Copy link
Member

Thanks @ivanlanin ! Good job :)

@ivanlanin
Copy link
Contributor

You're welcome :) We will need to use the new DrawingML later, but I guess we can live with VML for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ivanlanin ivanlanin

Labels
Question Status: Responded
Milestone
0.10.0
Development

Successfully merging this pull request may close these issues.
4 participants
@brunocasado @gabrielbull @Progi1984 @ivanlanin