From: Brett Cannon <bcannon@gmail.com>
Date: Thu, 19 Aug 2004 03:48:24 +0000 (+0000)
Subject: Rewrite test_order so as to be more "proper".  Originally relied on an
X-Git-Tag: v2.4a3~198
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=caa9798410887317226c4bf89f38d318b8436612;p=python

Rewrite test_order so as to be more "proper".  Originally relied on an
error based on decorating with staticmethod too soon for the code to execute.
This meant that if the test didn't pass it just errored out.  Now if the test
doesn't pass it leads to a failure instead.
---

diff --git a/Lib/test/test_decorators.py b/Lib/test/test_decorators.py
index c4eb8beb02..56aa5e1323 100644
--- a/Lib/test/test_decorators.py
+++ b/Lib/test/test_decorators.py
@@ -191,13 +191,19 @@ class TestDecorators(unittest.TestCase):
         self.assertEqual(C.foo.booh, 42)
 
     def test_order(self):
-        class C(object):
-            @staticmethod
-            @funcattrs(abc=1)
-            def foo(): return 42
-        # This wouldn't work if staticmethod was called first
-        self.assertEqual(C.foo(), 42)
-        self.assertEqual(C().foo(), 42)
+        # Test that decorators are applied in the proper order to the function
+        # they are decorating.
+        def callnum(num):
+            """Decorator factory that returns a decorator that replaces the
+            passed-in function with one that returns the value of 'num'"""
+            def deco(func):
+                return lambda: num
+            return deco
+        @callnum(2)
+        @callnum(1)
+        def foo(): return 42
+        self.assertEqual(foo(), 2,
+                            "Application order of decorators is incorrect")
 
     def test_eval_order(self):
         # Evaluating a decorated function involves four steps for each