Clean up some test assertions and make sure naming is consistent.

[Quinn-With-Two-Ns]

Clean up some test assertions and make sure naming is consistent. This is mostly just applying IDE recommendation for warnings.

---

Note

Normalize test assertions and class names, and update PlantUML generator references in tests and generated diagrams.

• Tests:
  • Replace assertEquals(null/true/false, ...) with assertNull/assertTrue/assertFalse, use assertNotEquals/assertInstanceOf, and fix assertion argument order across many tests.
  • Consolidate static imports (e.g., import static org.junit.Assert.*).
  • Rename test classes to *Test for consistency (e.g., UpdateTestTimeoutTest, WarnUnfinishedHandlersTest, WorkflowDescribeTest, various signal/update tests).
  • Minor assertion message/order tweaks and consistency fixes in state machine, worker, client, and workflow tests.
• Diagrams/Generators:
  • Update references from CommandsGeneratePlantUMLStateDiagrams to CommandsGeneratePlantUMLStateDiagramsTest in tests and generated .puml headers.

^{Written by Cursor Bugbot for commit 75f60a9. This will update automatically on new commits. Configure here.}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[maciejdudko]

A few minor suggestions but otherwise looks fine. LGTM!

[maciejdudko]

Suggested change

	assertNotSame(
	"Failure should not be benign", af.getCategory(), ApplicationErrorCategory.BENIGN);
	assertNotEquals(
	"Failure should not be benign", af.getCategory(), ApplicationErrorCategory.BENIGN);