From 62efd7e12e779113732df5c9856521e22ee814e0 Mon Sep 17 00:00:00 2001 From: davidkim1 Date: Sat, 14 Oct 2023 12:55:50 -0400 Subject: [PATCH 1/2] In test builders, added an elif case for publist that runs kwargs with pages argument --- tests/test_builders.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/test_builders.py b/tests/test_builders.py index 26741210f..b91bf8bc6 100644 --- a/tests/test_builders.py +++ b/tests/test_builders.py @@ -108,6 +108,9 @@ def test_builder(bm, db_src, make_db, make_mongodb, monkeypatch): elif bm == "grantreport": main(["build", bm, "--no-pdf", "--grant", "SymPy-1.1", "--from", "2017-04-01", "--to", "2018-03-31"]) + elif bm == "publist": + subprocess.run(["regolith", "build", bm, "--no-pdf", + "--kwargs", "pages-e103"], check=True, cwd=repo) else: subprocess.run(["regolith", "build", bm, "--no-pdf"], check=True, cwd=repo) os.chdir(os.path.join(repo, "_build", bm)) From 7064f592ee0ec7379a4e83d81bc8c44a8f971c26 Mon Sep 17 00:00:00 2001 From: davidkim1 Date: Sat, 14 Oct 2023 13:12:28 -0400 Subject: [PATCH 2/2] change the syntax in the new test to a key:value, and add a test case without kwargs command --- tests/test_builders.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/test_builders.py b/tests/test_builders.py index b91bf8bc6..cafb3767b 100644 --- a/tests/test_builders.py +++ b/tests/test_builders.py @@ -109,8 +109,9 @@ def test_builder(bm, db_src, make_db, make_mongodb, monkeypatch): main(["build", bm, "--no-pdf", "--grant", "SymPy-1.1", "--from", "2017-04-01", "--to", "2018-03-31"]) elif bm == "publist": + subprocess.run(["regolith", "build", bm, "--no-pdf"], check=True, cwd=repo) subprocess.run(["regolith", "build", bm, "--no-pdf", - "--kwargs", "pages-e103"], check=True, cwd=repo) + "--kwargs", "pages:e103"], check=True, cwd=repo) else: subprocess.run(["regolith", "build", bm, "--no-pdf"], check=True, cwd=repo) os.chdir(os.path.join(repo, "_build", bm))