Skip to content

Suppress a warning in the test suite. #13210

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
Jan 17, 2019
Merged

Suppress a warning in the test suite. #13210

merged 1 commit into from
Jan 17, 2019

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Jan 17, 2019

Namely

lib/matplotlib/tests/test_axes.py::test_step_linestyle[png]
  <venv>/lib/python3.7/site-packages/_pytest/python.py:165: RuntimeWarning: Second argument 'y' is ambiguous: could be a color spec but is in data; using as data.  Either rename the entry in data or use three arguments to plot.
    testfunction(**testargs)
  <venv>/lib/python3.7/site-packages/_pytest/python.py:165: RuntimeWarning: Second argument 'y1' is ambiguous: could be a color spec but is in data; using as data.  Either rename the entry in data or use three arguments to plot.
    testfunction(**testargs)
  <venv>/lib/python3.7/site-packages/_pytest/python.py:165: RuntimeWarning: Second argument 'y2' is ambiguous: could be a color spec but is in data; using as data.  Either rename the entry in data or use three arguments to plot.
    testfunction(**testargs)

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

@anntzer
Suppress a warning in the test suite.
13f641c 
Namely

    lib/matplotlib/tests/test_axes.py::test_step_linestyle[png]
      <venv>/lib/python3.7/site-packages/_pytest/python.py:165: RuntimeWarning: Second argument 'y' is ambiguous: could be a color spec but is in data; using as data.  Either rename the entry in data or use three arguments to plot.
        testfunction(**testargs)
      <venv>/lib/python3.7/site-packages/_pytest/python.py:165: RuntimeWarning: Second argument 'y1' is ambiguous: could be a color spec but is in data; using as data.  Either rename the entry in data or use three arguments to plot.
        testfunction(**testargs)
      <venv>/lib/python3.7/site-packages/_pytest/python.py:165: RuntimeWarning: Second argument 'y2' is ambiguous: could be a color spec but is in data; using as data.  Either rename the entry in data or use three arguments to plot.
        testfunction(**testargs)
@anntzer anntzer mentioned this pull request Jan 17, 2019
Merged
6 tasks
dstansby
dstansby approved these changes Jan 17, 2019
View reviewed changes
Copy link
Member

@dstansby dstansby left a comment

Choose a reason for hiding this comment

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

👍 if tests pass, anyone feel free to merge

@jklymak jklymak merged commit 55e504b into matplotlib:master Jan 17, 2019
@anntzer anntzer deleted the test_warning branch January 17, 2019 22:55
@QuLogic QuLogic added this to the v3.1 milestone Jan 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jklymak jklymak jklymak approved these changes

@dstansby dstansby dstansby approved these changes

Assignees
No one assigned
Labels
topic: testing
Projects
None yet
Milestone
v3.1.0
Development

Successfully merging this pull request may close these issues.
4 participants
@anntzer @jklymak @dstansby @QuLogic