test_saveall(): Another small simplification; plus s/l/L/g.

tim-one · tim-one · commit d92ae840e940 · 2002-08-11T04:15:09.000Z
test_del(), test_del_newclass():  No need to use apply() in these.
diff --git a/Lib/test/test_gc.py b/Lib/test/test_gc.py
@@ -174,21 +174,19 @@ def test_saveall():
     gc.collect()
     vereq(gc.garbage, []) # if this fails, someone else created immortal trash
 
-    l = []
-    l.append(l)
-    id_l = id(l)
+    L = []
+    L.append(L)
+    id_L = id(L)
 
     debug = gc.get_debug()
     gc.set_debug(debug | gc.DEBUG_SAVEALL)
-    del l
+    del L
     gc.collect()
     gc.set_debug(debug)
 
     vereq(len(gc.garbage), 1)
-    if id(gc.garbage[0]) == id_l:
-        del gc.garbage[0]
-    else:
-        raise TestFailed, "didn't find obj in garbage (saveall)"
+    obj = gc.garbage.pop()
+    vereq(id(obj), id_L)
 
 def test_del():
     # __del__ methods can trigger collection, make this to happen
@@ -203,7 +201,7 @@ def __del__(self):
     del a
 
     gc.disable()
-    apply(gc.set_threshold, thresholds)
+    gc.set_threshold(*thresholds)
 
 def test_del_newclass():
     # __del__ methods can trigger collection, make this to happen
@@ -218,7 +216,7 @@ def __del__(self):
     del a
 
     gc.disable()
-    apply(gc.set_threshold, thresholds)
+    gc.set_threshold(*thresholds)
 
 class Ouch:
     n = 0
