Deprecate twisted.python.runtime.Platform.isWinNT

[Jean-Paul Calderone]

Twisted only supports Windows NT-derived platforms at this point (Windows XP, Windows 7). See <​http://en.wikipedia.org/wiki/Windows_NT>.

This function can only ever return true in a supported environment.

[Thijs Triemstra]

It was only used once, in python.win32.getProgramsMenuPath which should be replaced once isWinNT is removed.

[Thijs Triemstra]

(In [36891]) deprecate isWinNT, refs #5925

[Tom Prince]

1. If this only returns True on win32, is there any reason to do anything more complicated than return True there? (In which case we can test that fact)
2. test_isWinNT shouldn't emit a deprecation warning.
3. python.win32.getProgramsMenuPath should be fixed to not use isWinNT *before* before the deprecation of isWinNT (Since it currently lacks tests, this should perhaps be a separate ticket).
4. Consider using SynchronousTestCase.callDeprecated for testing the deprecation.
5. Consider leaving a comment that t.p.deprecate isn't used, to avoid a dependency on that module.

Please resubmit for review after addressing the above issues.

[Tom Prince]

Also, assertIn is preferable to assertTrue(... in ...).

[Jean-Paul Calderone]

Consider using SynchronousTestCase.callDeprecated for testing the deprecation

Development documentation currently recommends flushWarnings, not callDeprecated.

[Thijs Triemstra]

There are also some "nt" references in twisted/python/dist.py but that should probably be fixed in separate ticket.

[Thijs Triemstra]

(In [37117]) address review comments. refs #5925

[Thijs Triemstra]

Replying to tom.prince:

1. If this only returns True on win32, is there any reason to do anything more complicated than return True there? (In which case we can test that fact)

Replaced the logic with a simple isWindows() check.

1. test_isWinNT shouldn't emit a deprecation warning.

Fixed.

1. python.win32.getProgramsMenuPath should be fixed to not use isWinNT *before* before the deprecation of isWinNT (Since it currently lacks tests, this should perhaps be a separate ticket).

Fixed. Also added tests.

Additionally added a test for Platform.isKnown and added some missing docstrings.

Forced a ​build.

[Tom Prince]

1. It looks like there is one relevant twistedchecker error

   ************* Module twisted.python.test.test_win32
   W9015: 68,0: Too many blank lines, expected 2
   

2. There are a bunch of \rs in that file.
3. (optional) I think twisted.python.win32.getProgramsMenuPath can probably be allowed to fail on non-windows. i.e. just remove the conditional there. I guess that is an incompatible change though.
4. assertNotNull doesn't exists. The tests probably want to at least assert that the return value is a C{str}, since that is what we document.

Please resubmit after addressing the above issues.

[Thijs Triemstra]

(In [37120]) address review comments. refs #5925

[Thijs Triemstra]

Replying to tom.prince:

1. It looks like there is one relevant twistedchecker error

   ************* Module twisted.python.test.test_win32
   W9015: 68,0: Too many blank lines, expected 2
   

Removed. Tried forcing a new build but the process just hung.. Need to get twistedchecker installed locally.

1. There are a bunch of \rs in that file.

Removed those. Seems these type of changes don't show up in the trac browser.

1. (optional) I think twisted.python.win32.getProgramsMenuPath can probably be allowed to fail on non-windows. i.e. just remove the conditional there. I guess that is an incompatible change though.

I'd rather handle that in a different ticket (or deprecate these altogether).

1. assertNotNull doesn't exists. The tests probably want to at least assert that the return value is a C{str}, since that is what we document.

Used assertIsInstance instead.

[Tom Prince]

Looks good. Please merge after the windows builders complete. (There are some unrelated failure on those builders, so check that the tests of the code modified here pass).

[Thijs Triemstra]

(In [37185]) Merge deprecate-iswinnt-5925: Platform.isWinNT is deprecated in favor of isWindows.

Author: thijs Reviewer: tom.prince Fixes: #5925

Twisted only supports Windows NT-derived platforms at this point (Windows XP, Windows 7).