- r semi inline uses lambda

Co-Authored-By: jmasonlee <262853+jmasonlee@users.noreply.github.com>
Co-Authored-By: Gregor Riegler <rieglerg85@gmail.com>
Co-Authored-By: blade290 <43077216+blade290@users.noreply.github.com>
Co-Authored-By: Nitsan Avni <nitsanav@gmail.com>
Co-Authored-By: bhargavgundu <7643639+bhargavgundu@users.noreply.github.com>

Author: 6 people

approvaltests/inline/inline_options.py

@@ -28,7 +28,7 @@ class SemiAutomaticInlineOptions(InlineOptions):
             def apply(self, options: "Options") -> "Options":
                 return options.with_reporter(
                     InlinePythonReporter(
-                        ReporterThatAutomaticallyApproves(), add_approval_line=True
+                        ReporterThatAutomaticallyApproves(), add_approval_line=False, previous_result=lambda __: DELETE_ME_TO_APPROVE_
                     )
                 )
 
@@ -61,18 +61,18 @@ def previous_capture():
         from approvaltests.namer.inline_python_reporter import InlinePythonReporter
         from approvaltests.reporters import ReporterThatAutomaticallyApproves
 
-        def applesauce(approved_path):
+        def create_previous_capture_suffix(approved_path):
             approved_text = Path(approved_path).read_text()
             approved_text = approved_text.rsplit("\n", 1)[0]
             approved_text = approved_text.rsplit(PREVIOUS_RESULT_, 1)[-1]
             previous_result_stuff = lambda: "\n" + PREVIOUS_RESULT_ + approved_text
             return DELETE_ME_TO_APPROVE_ + previous_result_stuff()
-        class SemiAutomaticInlineOptions(InlineOptions):
+        class PreviousCaptureInlineOptions(InlineOptions):
             def apply(self, options: "Options") -> "Options":
                 return options.with_reporter(
                     InlinePythonReporter(
-                        ReporterThatAutomaticallyApproves(), add_approval_line=False, previous_result=applesauce
+                        ReporterThatAutomaticallyApproves(), add_approval_line=False, previous_result=create_previous_capture_suffix
                     )
                 )
 
-        return SemiAutomaticInlineOptions()
+        return PreviousCaptureInlineOptions()

tests/approved_files/test_inline_approvals.test_inline_with_semi_automatic_inline.approved.txt

@@ -0,0 +1,2 @@
+42
+***** DELETE ME TO APPROVE *****

tests/test_inline_approvals.py

@@ -202,3 +202,15 @@ def test_inline_with_preserved_approved_text():
     except ApprovalException:
         pass
     verify(get_approved_via_doc_string())
+    
+def test_inline_with_semi_automatic_inline():
+    """
+    42
+    ***** DELETE ME TO APPROVE *****
+    """
+    options = Options().inline(InlineOptions.semi_automatic())
+    try:
+        verify("42", options=options)
+    except ApprovalException:
+        pass
+    verify(get_approved_via_doc_string())