From 50a5dad608eb4e582a30726c2214b6578d6e11db Mon Sep 17 00:00:00 2001 From: Zachary Ware Date: Wed, 27 Nov 2013 23:56:04 -0600 Subject: [PATCH] Issue #19595: Re-enable a long-disabled test in test_winsound --- Lib/test/test_winsound.py | 20 ++++++++------------ Misc/NEWS | 2 ++ 2 files changed, 10 insertions(+), 12 deletions(-) diff --git a/Lib/test/test_winsound.py b/Lib/test/test_winsound.py index 61d864a6485..d8a617e64fb 100644 --- a/Lib/test/test_winsound.py +++ b/Lib/test/test_winsound.py @@ -158,18 +158,14 @@ class PlaySoundTest(unittest.TestCase): ) def test_alias_fallback(self): - # This test can't be expected to work on all systems. The MS - # PlaySound() docs say: - # - # If it cannot find the specified sound, PlaySound uses the - # default system event sound entry instead. If the function - # can find neither the system default entry nor the default - # sound, it makes no sound and returns FALSE. - # - # It's known to return FALSE on some real systems. - - # winsound.PlaySound('!"$%&/(#+*', winsound.SND_ALIAS) - return + if _have_soundcard(): + winsound.PlaySound('!"$%&/(#+*', winsound.SND_ALIAS) + else: + self.assertRaises( + RuntimeError, + winsound.PlaySound, + '!"$%&/(#+*', winsound.SND_ALIAS + ) def test_alias_nofallback(self): if _have_soundcard(): diff --git a/Misc/NEWS b/Misc/NEWS index 4ed79503987..3c82e1ab218 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -31,6 +31,8 @@ Library Tests ----- +- Issue #19595: Re-enabled a long-disabled test in test_winsound. + - Issue #19588: Fixed tests in test_random that were silently skipped most of the time. Patch by Julian Gindi.