[Python-Dev] Still looking for volunteer to run Windows buildbot (original) (raw)

Tim Peters tim.peters at gmail.com
Tue Mar 14 16:55:21 CET 2006

[Trent Mick]

I have a patch in the works that defaults to "yes, this machine does have a soundcard" if cscript.exe cannot be found on the PATH.

However, one wrinkle: testwinsound.py is made up of three test cases: BeepTest MessageBeepTest PlaySoundTest only the last need be skipped if there is not soundcard.

I'd say instead that they should never be skipped: the real difference on your box is the expected outcome in the third category.

After umpteen years we've got a universe of one machine where PlaySoundTest is known to fail, and now a little mound of VB code that presumably returns something different on that machine than on other machines. In reality, that's more code to test.

We seem to be assuming here that "the VB code says no sound device" means "PlaySoundTest will fail in a particular way", and have one box on which that's known to be true. So sure, skip the tests on that box, and the immediate buildbot failure on that box will go away. Other possiblities include that the test will also be skipped on boxes where it would actually work, because the VB code isn't actually a definitive test for some reason.

Since we can't be sure from a universe of one exception, better to test that assumption too, by reworking the tests to say "oh, but if the VB code thinks we don't have a sound card, then this test should raise RuntimeError instead". There's still a testable outcome here.

More information about the Python-Dev mailing list