Thanks to visit codestin.com
Credit goes to github.com

Skip to content

remove print from test #3666

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
Merged
merged 1 commit into from
Oct 18, 2014
Merged

remove print from test #3666

merged 1 commit into from
Oct 18, 2014

Conversation

jenshnielsen
Copy link
Member

This removes a print in test_mathtext_exceptions. This shows up in Travis output because of -s to nose even when the tests pass. Looks like a leftover debug statement which should be removed.

The exception which this is inside is the expected outcome of the test.

@jenshnielsen
remove print from test
8ba9ac0 
This removes a print in test_mathtext_exceptions. This shows up in travis output because of -s to nose even when the tests pass. Looks like a leftover debug statment.
@jenshnielsen
Copy link
Member Author

This removes the following from a passing travis test log


\hspace{}
       ^
Expected \hspace{n} (at char 7), (line:1, col:8)

\hspace{foo}
       ^
Expected \hspace{n} (at char 7), (line:1, col:8)

\frac
     ^
Expected \frac{num}{den} (at char 5), (line:1, col:6)

\frac{}{}
     ^
Expected \frac{num}{den} (at char 5), (line:1, col:6)

\stackrel
         ^
Expected \stackrel{num}{den} (at char 9), (line:1, col:10)

\stackrel{}{}
         ^
Expected \stackrel{num}{den} (at char 9), (line:1, col:10)

\binom
      ^
Expected \binom{num}{den} (at char 6), (line:1, col:7)

\binom{}{}
      ^
Expected \binom{num}{den} (at char 6), (line:1, col:7)

\genfrac
        ^
Expected \genfrac{ldelim}{rdelim}{rulesize}{style}{num}{den} (at char 8), (line:1, col:9)

\genfrac{}{}{}{}{}{}
        ^
Expected \genfrac{ldelim}{rdelim}{rulesize}{style}{num}{den} (at char 8), (line:1, col:9)

\sqrt
     ^
Expected \sqrt{value} (at char 5), (line:1, col:6)

\sqrt f
      ^
Expected \sqrt{value} (at char 6), (line:1, col:7)

\overline
         ^
Expected \overline{value} (at char 9), (line:1, col:10)

\overline{}
         ^
Expected \overline{value} (at char 9), (line:1, col:10)

\leftF
     ^
Expected a delimiter (at char 5), (line:1, col:6)

\rightF
^
Unknown symbol: \rightF (at char 0), (line:1, col:1)

\left(\right
            ^
Expected a delimiter (at char 12), (line:1, col:13)

\left(
      ^
Expected "\right" (at char 6), (line:1, col:7)

tacaswell added a commit that referenced this pull request Oct 18, 2014
@tacaswell
Merge pull request #3666 from jenshnielsen/test_mathtext_print
641b37a 
TST : remove print from test
@tacaswell tacaswell merged commit 641b37a into matplotlib:master Oct 18, 2014
@tacaswell
Copy link
Member

huzzah! that bit of output has always bothered me....

Not sure why travis isn't picking this up, going to just merge it as-is.

@jenshnielsen jenshnielsen deleted the test_mathtext_print branch October 18, 2014 06:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jenshnielsen @tacaswell