Ticket #4086: twisted4086.patch

twisted/scripts/test/test_tap2rpm.py

@@ -5 +5 @@
 Tests for L{twisted.scripts.tap2rpm}.
 """
 import os
+import warnings
+
 from twisted.trial.unittest import TestCase, SkipTest
 from twisted.python import procutils
+from twisted.python import versions
 from twisted.python.failure import Failure
 from twisted.internet import utils
 from twisted.scripts import tap2rpm
@@ -378 +381 @@
 
         # Try and make an RPM from that tapfile.
         _makeRPMs(tapfile=tapfile)
+
+
+    def test_unsignedFlagDeprecationWarning(self):
+        """
+        The 'unsigned' flag in tap2rpm should be deprecated, and its use
+        should raise a warning as such.
+        """
+        config = tap2rpm.MyOptions()
+        config.parseOptions(['--unsigned'])
+        warnings = self.flushWarnings()
+        self.assertEqual(DeprecationWarning, warnings[0]['category'])
+        self.assertEqual(
+            "The 'unsigned' flag is deprecated, as it's not currently used.",
+            warnings[0]['message'])
+        self.assertEqual(1, len(warnings))

twisted/scripts/tap2rpm.py

@@ -7 +7 @@
 import tempfile
 import tarfile
 from StringIO import StringIO
+import warnings
 
 from twisted.python import usage, log
 
@@ -138 +139 @@
 
 ###############################
 class MyOptions(usage.Options):
-    optFlags = [["unsigned", "u"], ['quiet', 'q']]
+    optFlags = [['unsigned', 'u'], ['quiet', 'q']]
     optParameters = [
                      ["tapfile", "t", "twistd.tap"],
                      ["maintainer", "m", "tap2rpm"],
@@ -176 +177 @@
         self['twistd_option'] = type_dict[self['type']]
         self['release-name'] = '%s-%s' % (self['rpmfile'], self['set-version'])
 
+        # Deprecated 'unsigned' flag
+        msg = "The 'unsigned' flag is deprecated, as it's not currently used."
+        warnings.warn(msg, category=DeprecationWarning, stacklevel=2)
 
 
 type_dict = {