Add test for no-pid API update

kohler · Mar 4, 2025 · a5ec163 · a5ec163
1 parent 86eadc4
commit a5ec163
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/src/api/api_paper.php b/src/api/api_paper.php
@@ -1,6 +1,6 @@
 <?php
 // api_paper.php -- HotCRP paper API call
-// Copyright (c) 2008-2024 Eddie Kohler; see LICENSE.
+// Copyright (c) 2008-2025 Eddie Kohler; see LICENSE.
 
 class Paper_API extends MessageSet {
     /** @var Conf

diff --git a/test/t_paperapi.php b/test/t_paperapi.php
@@ -1,6 +1,6 @@
 <?php
 // t_paperapi.php -- HotCRP tests
-// Copyright (c) 2024 Eddie Kohler; see LICENSE.
+// Copyright (c) 2024-2025 Eddie Kohler; see LICENSE.
 
 class PaperAPI_Tester {
     /** @var Conf
@@ -286,6 +286,15 @@ function test_match() {
         xassert_eqq($jr->ok, false);
     }
 
+    function test_no_pid() {
+        $qreq = $this->make_post_json_qreq(["calories" => 9], ["p" => "1"]);
+        $jr = call_api("=paper", $this->u_chair, $qreq);
+        xassert_eqq($jr->ok, true);
+        xassert_eqq($jr->change_list, ["calories"]);
+        xassert_eqq($jr->paper->pid, 1);
+        xassert_eqq($jr->paper->calories, 9);
+    }
+
     function test_new_paper_after_deadline() {
         $this->conf->save_setting("sub_update", Conf::$now - 10);
         $this->conf->save_setting("sub_sub", Conf::$now - 10);