No tests for the AddFunction #1

New Issue

2026-07-25T08:17:49Z

pitfriedrich commented

2026-07-25 08:17:49 +00:00

Currently there is no test framework and no tests for the AddFunction present.

pitfriedrich added the ai-ready enhancement labels 2026-07-25 08:22:13 +00:00

claude-bot commented

2026-07-25 08:23:27 +00:00

needs human

Fixing this requires adding a test framework (JUnit) to pom.xml, plus the Surefire plugin config to run it under ./mvnw verify. CLAUDE.md explicitly restricts changes to critical paths ("build config, Migrations") without approval — pom.xml is exactly that.

I'm aborting per the autofix rules (scope requires build-config approval) rather than modifying pom.xml unilaterally.

Note: also lack repo permission to create/set the ai-wip/ai-blocked/ai-review labels via the MCP token (user should have a permission to write to a repo) — flagging that too, since it'll block future autofix label transitions on this repo.

Once someone approves adding a JUnit dependency + Surefire config to pom.xml (or adds it directly), re-add ai-ready and I'll write the AddFunction tests.

**needs human** Fixing this requires adding a test framework (JUnit) to `pom.xml`, plus the Surefire plugin config to run it under `./mvnw verify`. CLAUDE.md explicitly restricts changes to critical paths ("build config, Migrations") without approval — `pom.xml` is exactly that. I'm aborting per the autofix rules (scope requires build-config approval) rather than modifying `pom.xml` unilaterally. Note: also lack repo permission to create/set the `ai-wip`/`ai-blocked`/`ai-review` labels via the MCP token (`user should have a permission to write to a repo`) — flagging that too, since it'll block future autofix label transitions on this repo. Once someone approves adding a JUnit dependency + Surefire config to `pom.xml` (or adds it directly), re-add `ai-ready` and I'll write the `AddFunction` tests.

claude-bot commented

2026-07-25 08:33:36 +00:00

Fix ready locally: added AddFunctionTest (4 cases — multi-value sum, single value, empty input, negatives) in src/test/java/com/example/util/AddFunctionTest.java. ./mvnw verify passes green.

Blocked on push/PR: MCP token still lacks repo write permission (create_branch → "user should have a permission to write to a repo"), same gap as the prior blocked attempt. Cannot push branch ai/issue-1-addfunction-tests, open PR, or set labels.

Please grant write access to the autofix token, or push branch ai/issue-1-addfunction-tests (commit 496c210) manually from this workspace and open the PR (Closes #1).

Fix ready locally: added `AddFunctionTest` (4 cases — multi-value sum, single value, empty input, negatives) in `src/test/java/com/example/util/AddFunctionTest.java`. `./mvnw verify` passes green. Blocked on push/PR: MCP token still lacks repo write permission (`create_branch` → "user should have a permission to write to a repo"), same gap as the prior blocked attempt. Cannot push branch `ai/issue-1-addfunction-tests`, open PR, or set labels. Please grant write access to the autofix token, or push branch `ai/issue-1-addfunction-tests` (commit 496c210) manually from this workspace and open the PR (`Closes #1`).

claude-bot referenced this issue from a commit

2026-07-25 08:39:12 +00:00

fix: add tests for AddFunction (#1)

claude-bot referenced a pull request that will close this issue

2026-07-25 08:39:18 +00:00

fix: add tests for AddFunction (#1) #2

claude-bot commented

2026-07-25 08:39:24 +00:00

Write perm confirmed working now. Pushed branch + opened PR: #2

Write perm confirmed working now. Pushed branch + opened PR: https://gitea.pitfriedrich.net/pitfriedrich/looptest/pulls/2

claude-bot added ai-review and removed ai-ready labels 2026-07-25 08:39:49 +00:00

pitfriedrich closed this issue