test: cover k8s-client 404 detection against real ApiException shape (FAR-85)

farhoodlabs/paperclip-adapter-opencode-k8s

There was no test file for k8s-client.ts. Existing pvc.test.ts mocked
`getPvc` directly and never exercised the underlying isNotFound predicate,
so the v1.x ApiException `code` vs `statusCode` regression had nothing to
catch it.

Add k8s-client.test.ts that mocks @kubernetes/client-node, throws errors
shaped exactly like the real ApiException (status under `code`), and
verifies:

- getPvc returns null on code=404 (the FAR-85 case)
- getPvc still handles legacy statusCode=404 and response.statusCode=404
- getPvc re-throws non-404 errors (500, 403)
- deletePvc swallows 404, re-throws others
- createPvc forwards spec to the SDK

Confirmed the new tests fail when k8s-client.ts is reverted to the
pre-fix predicate (2 failures), and pass with the fix in place.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

This commit is contained in:

Chris Farhood

2026-04-25 22:05:48 +00:00

committed by

Hugh Commit [agent]

parent 3daf2dd676

commit c71e224b43

2 changed files with 146 additions and 1 deletions

									
										package.json
									
		+1
		-1
	
												View File
												
				@@ -1,6 +1,6 @@

				{

				  "name": "paperclip-adapter-opencode-k8s",

				  "version": "0.1.27",

				  "version": "0.1.28",

				  "description": "Paperclip adapter plugin that runs OpenCode agents as Kubernetes Jobs",

				  "license": "MIT",

				  "type": "module",