Merge pull request #1 from edburns/edburns/review-belaltaher8-pr-1864

belaltaher8 · web-flow · commit 9806296620fe · 2026-07-14T20:10:08.000-04:00
test: assert null metadata arg in generated tool definition
diff --git a/java/src/test/java/com/github/copilot/tool/CopilotToolProcessorTest.java b/java/src/test/java/com/github/copilot/tool/CopilotToolProcessorTest.java
@@ -261,6 +261,8 @@ public String doSomething(@CopilotToolParam("Input") String input) {
         String generated = result.getGeneratedSource("test.PlainTools$$CopilotToolMeta");
         assertFalse(generated.contains("Map.<String, Object>of("),
                 "Expected no metadata map for a tool without metadata, got:\n" + generated);
+        assertTrue(generated.contains("                null\n            )"),
+                "Expected metadata constructor argument to be null when metadata is absent, got:\n" + generated);
     }
 
     @Test

Original file line number	Diff line number	Diff line change
`@@ -261,6 +261,8 @@ public String doSomething(@CopilotToolParam("Input") String input) {`
`261`	`261`	`String generated = result.getGeneratedSource("test.PlainTools$$CopilotToolMeta");`
`262`	`262`	`assertFalse(generated.contains("Map.<String, Object>of("),`
`263`	`263`	`"Expected no metadata map for a tool without metadata, got:\n" + generated);`
	`264`	`+ assertTrue(generated.contains(" null\n )"),`
	`265`	`+ "Expected metadata constructor argument to be null when metadata is absent, got:\n" + generated);`
`264`	`266`	`}`
`265`	`267`
`266`	`268`	`@Test`