[issue19595] Silently skipped test in test_winsound
Roundup Robot added the comment: New changeset 30b3798782f1 by Zachary Ware in branch '2.7': Issue #19595: Re-enable a long-disabled test in test_winsound http://hg.python.org/cpython/rev/30b3798782f1 New changeset 63f3e8670fa6 by Zachary Ware in branch '3.3': Issue #19595: Re-enable a long-disabled test in test_winsound http://hg.python.org/cpython/rev/63f3e8670fa6 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19595 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19595] Silently skipped test in test_winsound
Zachary Ware added the comment: As long as the buildbots stay happy, this test is back to actually testing. -- resolution: - fixed stage: patch review - committed/rejected status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19595 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19595] Silently skipped test in test_winsound
Roundup Robot added the comment: New changeset 6f1c5d0b44ed by Zachary Ware in branch 'default': Issue #19595: Re-enable a long-disabled test in test_winsound http://hg.python.org/cpython/rev/6f1c5d0b44ed -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19595 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19595] Silently skipped test in test_winsound
Zachary Ware added the comment:
I don't know that it's expected, but it doesn't appear to be unexpected.
regrtest doesn't appear to know anything about expected failures or unexpected
successes, mostly because both concepts appear at the individual test level
while regrtest deals at the test module level. unittest docs don't actually
say what happens when an expected failure fails to fail.
As for this issue, it looks like the skipping of the test predates
_have_soundcard and has_sound. If there are no objections, I'll try the
attached patch on default; if it causes failures, I'll try a
'skipUnless(has_sound('SystemDefault')...', and if that work's I'll backport it
to the other two branches.
--
assignee: - zach.ware
keywords: +patch
stage: test needed - patch review
Added file: http://bugs.python.org/file32704/issue19595.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19595
___
___
Python-bugs-list mailing list
Unsubscribe:
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19595] Silently skipped test in test_winsound
Ezio Melotti added the comment: Maybe someone else can try it, or just enable it and see if the buildbots are happy (I think you can use custom builds as well for this kind of experiments). at least on 3.4 an unexpected success is silently ignored by regrtest is this expected? -- nosy: +ezio.melotti ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19595 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19595] Silently skipped test in test_winsound
New submission from Zachary Ware: In test_winsound, PlaySoundTest.test_alias_fallback has been commented out for quite some time but has been reported as 'ok' ever since. Does anyone know what the current status of the test is? Should it still be skipped (explicitly)? Should it be marked as an expected failure? Would it be dangerous to enable? My own testing shows it to be harmless on my machine, but my range of test configurations is sorely limited at present. My own suggestion is to mark it as an expected failure (or just enable it wholesale, and mark it if it does prove flaky) and see what the buildbots make of it, but I don't want to do that if it could cause anything worse than an exception. Thoughts? -- components: Tests, Windows messages: 202887 nosy: brian.curtin, serhiy.storchaka, tim.golden, zach.ware priority: normal severity: normal status: open title: Silently skipped test in test_winsound type: behavior versions: Python 2.7, Python 3.3, Python 3.4 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19595 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19595] Silently skipped test in test_winsound
Changes by Zachary Ware zachary.w...@gmail.com: -- stage: - test needed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19595 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19595] Silently skipped test in test_winsound
R. David Murray added the comment: I believe that expected failure will give an error report if the test succeeds. -- nosy: +r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19595 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue19595] Silently skipped test in test_winsound
Zachary Ware added the comment: It doesn't appear to; at least on 3.4 an unexpected success is silently ignored by regrtest in non-verbose mode and added to the report in verbose mode (e.g., OK (skipped=1, unexpected successes=1)) or when run alone. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue19595 ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
