Address review comments

Author: MarieRoald

Doc/library/turtle.rst

@@ -222,18 +222,18 @@ If you have Python 3.14 or later, you don't need to call :func:`begin_fill` and
 example::
 
    with fill():
-      for i in range(4):
-         forward(100)
-         right(90)
+       for i in range(4):
+           forward(100)
+           right(90)
 
    forward(200)
 
 The code above is equivalent to::
 
    begin_fill()
    for i in range(4):
-      forward(100)
-      right(90)
+       forward(100)
+       right(90)
    end_fill()
 
    forward(200)
@@ -377,6 +377,7 @@ Pen control
 
    Filling
       | :func:`filling`
+      | :func:`fill`
       | :func:`begin_fill`
       | :func:`end_fill`
 

Doc/whatsnew/3.14.rst

@@ -438,6 +438,15 @@ sys
   which only exists in specialized builds of Python, may now return objects
   from other interpreters than the one it's called in.
 
+turtle
+------
+
+* Add context managers for :func:`turtle.fill`, :func:`turtle.poly` and
+  :func:`turtle.no_animation` for
+  :func:`turtle.begin_fill`/:func:`turtle.end_fill`,
+  :func:`turtle.begin_poly`/:func:`turtle.end_poly`, and
+  :func:`turtle.tracer(0)`/:func:`turtle.tracer(1)`, respectively.
+  (Contributed by Marie Roald and Yngve Mardal Moe in :gh:`126350`.)
 
 unicodedata
 -----------

Lib/test/test_turtle.py

@@ -3,8 +3,8 @@
 import re
 import unittest
 import unittest.mock
-import tempfile
 import sys
+import tempfile
 from contextlib import contextmanager
 from test import support
 from test.support import import_helper
@@ -572,7 +572,6 @@ def test_no_animation_calls_update_at_exit(self):
 
 class TestTurtle(unittest.TestCase):
     def test_begin_end_fill(self):
-        """begin_fill and end_fill counter each other."""
         with patch_screen():
             t = turtle.Turtle()
 
@@ -583,7 +582,7 @@ def test_begin_end_fill(self):
         self.assertFalse(t.filling())
 
     def test_fill(self):
-        """The context manager behaves like begin_ and end_ fill."""
+        # The context manager behaves like begin_fill and end_fill.
         with patch_screen():
             t = turtle.Turtle()
 
@@ -593,7 +592,7 @@ def test_fill(self):
         self.assertFalse(t.filling())
 
     def test_fill_resets_after_exception(self):
-        """The context manager cleans up correctly after exceptions."""
+        # The context manager cleans up correctly after exceptions.
         with patch_screen():
             t = turtle.Turtle()
         try:
@@ -604,7 +603,7 @@ def test_fill_resets_after_exception(self):
             self.assertFalse(t.filling())
 
     def test_fill_context_when_filling(self):
-        """The context manager works even when the turtle is already filling."""
+        # The context manager works even when the turtle is already filling.
         with patch_screen():
             t = turtle.Turtle()
 
@@ -615,7 +614,6 @@ def test_fill_context_when_filling(self):
         self.assertFalse(t.filling())
 
     def test_begin_end_poly(self):
-        """begin_fill and end_poly counter each other."""
         with patch_screen():
             t = turtle.Turtle()
 
@@ -626,7 +624,7 @@ def test_begin_end_poly(self):
         self.assertFalse(t._creatingPoly)
 
     def test_poly(self):
-        """The context manager behaves like begin_ and end_ poly."""
+        # The context manager behaves like begin_poly and end_poly.
         with patch_screen():
             t = turtle.Turtle()
 
@@ -636,7 +634,7 @@ def test_poly(self):
         self.assertFalse(t._creatingPoly)
 
     def test_poly_resets_after_exception(self):
-        """The context manager cleans up correctly after exceptions."""
+        # The context manager cleans up correctly after exceptions.
         with patch_screen():
             t = turtle.Turtle()
         try:
@@ -647,8 +645,7 @@ def test_poly_resets_after_exception(self):
             self.assertFalse(t._creatingPoly)
 
     def test_poly_context_when_creating_poly(self):
-        """The context manager works when the turtle is already creating poly.
-        """
+        # The context manager works when the turtle is already creating poly.
         with patch_screen():
             t = turtle.Turtle()
 

Lib/turtle.py

@@ -107,11 +107,10 @@
 
 from os.path import isfile, split, join
 from pathlib import Path
+from contextlib import contextmanager
 from copy import deepcopy
 from tkinter import simpledialog
 
-from contextlib import contextmanager
-
 _tg_classes = ['ScrolledCanvas', 'TurtleScreen', 'Screen',
                'RawTurtle', 'Turtle', 'RawPen', 'Pen', 'Shape', 'Vec2D']
 _tg_screen_functions = ['addshape', 'bgcolor', 'bgpic', 'bye',