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

Improvement: warning notification #124

Open
domi84 opened this issue Nov 11, 2018 · 2 comments
Open

Improvement: warning notification #124

domi84 opened this issue Nov 11, 2018 · 2 comments

Comments

@domi84
Copy link

domi84 commented Nov 11, 2018

Hi, thanks for this plugin.

Description of your problem

Would be useful have notification of the warning generated by the test (pytest in my case)

What steps will reproduce the problem?

When I run pytest in the terminal I got 2 warnings:

lab@domenico-HP-ENVY-Notebook:~/Dropbox/Python_Script/Papillon_dev$ pytest
================================================================ test session starts =================================================================
platform linux -- Python 3.6.4, pytest-3.2.1, py-1.4.34, pluggy-0.4.0
rootdir: /home/lab/Dropbox/Python_Script/Papillon_dev, inifile:
collected 21 items                                                                                                                                    

test/test_papillon.py .....................

================================================================== warnings summary ==================================================================
test/test_papillon.py::papillon_Test::test_plots
  /home/lab/anaconda3/lib/python3.6/site-packages/seaborn/categorical.py:3666: UserWarning: The `factorplot` function has been renamed to `catplot`. The original name will be removed in a future release. Please update your code. Note that the default `kind` in `factorplot` (`'point'`) has changed `'strip'` in `catplot`.
    warnings.warn(msg)
  /home/lab/anaconda3/lib/python3.6/site-packages/seaborn/categorical.py:3672: UserWarning: The `size` paramter has been renamed to `height`; please update your code.
    warnings.warn(msg, UserWarning)

-- Docs: http://doc.pytest.org/en/latest/warnings.html
======================================================= 21 passed, 2 warnings in 15.90 seconds =======================================================
lab@domenico-HP-ENVY-Notebook:~/Dropbox/Python_Script/Papillon_dev$

But if I run spyder-unittest it says 0 tests failed, 21 passed

if I click on "show output" the warning are there:

============================= test session starts ==============================
platform linux -- Python 3.6.4, pytest-3.2.1, py-1.4.34, pluggy-0.4.0
rootdir: /home/lab/Dropbox/Python_Script/Papillon_dev, inifile:
collected 21 items

test/test_papillon.py .....................

=============================== warnings summary ===============================
test/test_papillon.py::papillon_Test::test_plots
  /home/lab/anaconda3/lib/python3.6/site-packages/seaborn/categorical.py:3666: UserWarning: The `factorplot` function has been renamed to `catplot`. The original name will be removed in a future release. Please update your code. Note that the default `kind` in `factorplot` (`'point'`) has changed `'strip'` in `catplot`.
    warnings.warn(msg)
  /home/lab/anaconda3/lib/python3.6/site-packages/seaborn/categorical.py:3672: UserWarning: The `size` paramter has been renamed to `height`; please update your code.
    warnings.warn(msg, UserWarning)

-- Docs: http://doc.pytest.org/en/latest/warnings.html
==================== 21 passed, 2 warnings in 12.63 seconds ====================

What is the expected output? What do you see instead?

I would expect a yellow bar saying
0 tests failed, 21 passed, 2 warnings

I see:
a green bar saying
0 tests failed, 21 passed
that let me think everything is ok, while is not...

Key versions and other information:

  • Spyder version: 3.3.0
  • Version of spyder-unittest plugin: 0.3.1
  • Installation method for Spyder and the unittest plugin: pip3
  • Python version: 3.6.4
  • Testing framework used: py.test
  • Testing framework version: 3.2.1
  • Operating system: Ubuntu 16.04.4
@goanpeca
Copy link
Member

goanpeca commented May 4, 2020

Hi @jitseniesen is this still the case?

@jitseniesen
Copy link
Member

is this still the case?

Probably, I don't think there is any logic for reporting warnings in the summary box.

@jitseniesen jitseniesen added this to the v0.5.1 milestone Jan 6, 2022
@jitseniesen jitseniesen removed the v0.5.0 label Jan 6, 2022
@jitseniesen jitseniesen modified the milestones: v0.5.1, v0.5.2 Aug 17, 2022
@jitseniesen jitseniesen removed their assignment Aug 17, 2022
@jitseniesen jitseniesen modified the milestones: v0.5.2, v0.6.0 Mar 24, 2023
@jitseniesen jitseniesen modified the milestones: v0.6.0, v0.7.0 Jun 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants