add testhelper for local update

Summary: these helper functions make these tests way easier to read. I did this when writing some tests today and figured we should keep it.

Reviewed By: connernilsen

Differential Revision: D50747692

fbshipit-source-id: 93b6afe8e5928badd841c7f7e811739ead44da68

source/code_navigation_server/test/basicTest.ml

@@ -312,17 +312,10 @@ let test_file_and_local_update context =
         open_file ~path:(PyrePath.absolute test_path) ~client_id;
         open_file ~path:(PyrePath.absolute test2_path) ~client_id;
         assert_type_error_count_for_path ~path:(PyrePath.absolute test_path) ~client_id ~expected:2;
-        ScratchProject.ClientConnection.assert_response
-          ~request:
-            Request.(
-              Command
-                (Command.LocalUpdate
-                   {
-                     path = PyrePath.absolute test_path;
-                     content = Some "from test2 import x";
-                     client_id;
-                   }))
-          ~expected:Response.Ok;
+        local_update
+          ~path:(PyrePath.absolute test_path)
+          ~content:(Some "from test2 import x")
+          ~client_id;
         assert_type_error_count_for_path ~path:(PyrePath.absolute test_path) ~client_id ~expected:1;
         (fun _ ->
           File.create test2_path ~content:"x: int = 42" |> File.write;

source/code_navigation_server/test/testHelper.ml

@@ -39,6 +39,12 @@ let close_file ~client_id ~path =
     ~expected:Response.Ok
 
 
+let local_update ~client_id ~path ~content =
+  ScratchProject.ClientConnection.assert_response
+    ~request:Request.(Command (Command.LocalUpdate { path; client_id; content }))
+    ~expected:Response.Ok
+
+
 let assert_type_error_count ~client_id ~path ~expected client =
   let%lwt raw_response =
     ScratchProject.ClientConnection.send_request