Skip to content

Commit f5e9662

Browse files
brunoborgesbrunoborges
committed
Refactor data access methods in tests to use new naming conventions
- Updated test assertions in MetadataApiTest, PermissionsTest, SessionEventHandlingTest, SessionEventParserTest, SessionEventsE2ETest, SkillsTest, and ToolsTest to reflect changes in data access methods from `getX()` to `x()`. - Ensured consistency across all test files by replacing deprecated method calls with the new accessors.
1 parent 2980897 commit f5e9662

53 files changed

Lines changed: 457 additions & 2320 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎src/main/java/com/github/copilot/sdk/CopilotClient.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848
* var session = client.createSession(new SessionConfig().setModel("gpt-5")).get();
4949
*
5050
* session.on(AssistantMessageEvent.class, msg -> {
51-
* System.out.println(msg.getData().getContent());
51+
* System.out.println(msg.getData().content());
5252
* });
5353
*
5454
* session.send(new MessageOptions().setPrompt("Hello!")).get();

‎src/main/java/com/github/copilot/sdk/CopilotSession.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@
6363
*
6464
* // Register type-safe event handlers
6565
* session.on(AssistantMessageEvent.class, msg -> {
66-
* System.out.println(msg.getData().getContent());
66+
* System.out.println(msg.getData().content());
6767
* });
6868
* session.on(SessionIdleEvent.class, idle -> {
6969
* System.out.println("Session is idle");
@@ -338,7 +338,7 @@ public CompletableFuture<AssistantMessageEvent> sendAndWait(MessageOptions optio
338338
} else if (evt instanceof SessionIdleEvent) {
339339
future.complete(lastAssistantMessage.get());
340340
} else if (evt instanceof SessionErrorEvent errorEvent) {
341-
String message = errorEvent.getData() != null ? errorEvent.getData().getMessage() : "session error";
341+
String message = errorEvent.getData() != null ? errorEvent.getData().message() : "session error";
342342
future.completeExceptionally(new RuntimeException("Session error: " + message));
343343
}
344344
};
@@ -453,7 +453,7 @@ public Closeable on(Consumer<AbstractSessionEvent> handler) {
453453
* <pre>{@code
454454
* // Handle assistant messages
455455
* session.on(AssistantMessageEvent.class, msg -> {
456-
* System.out.println(msg.getData().getContent());
456+
* System.out.println(msg.getData().content());
457457
* });
458458
*
459459
* // Handle session idle
@@ -463,7 +463,7 @@ public Closeable on(Consumer<AbstractSessionEvent> handler) {
463463
*
464464
* // Handle streaming deltas
465465
* session.on(AssistantMessageDeltaEvent.class, delta -> {
466-
* System.out.print(delta.getData().getDeltaContent());
466+
* System.out.print(delta.getData().deltaContent());
467467
* });
468468
* }</pre>
469469
*

‎src/main/java/com/github/copilot/sdk/events/AbortEvent.java‎

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -32,17 +32,6 @@ public void setData(AbortData data) {
3232
}
3333

3434
@JsonIgnoreProperties(ignoreUnknown = true)
35-
public static class AbortData {
36-
37-
@JsonProperty("reason")
38-
private String reason;
39-
40-
public String getReason() {
41-
return reason;
42-
}
43-
44-
public void setReason(String reason) {
45-
this.reason = reason;
46-
}
35+
public record AbortData(@JsonProperty("reason") String reason) {
4736
}
4837
}

‎src/main/java/com/github/copilot/sdk/events/AbstractSessionEvent.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
* <pre>{@code
3838
* session.on(event -> {
3939
* if (event instanceof AssistantMessageEvent msg) {
40-
* System.out.println("Assistant: " + msg.getData().getContent());
40+
* System.out.println("Assistant: " + msg.getData().content());
4141
* } else if (event instanceof SessionIdleEvent) {
4242
* System.out.println("Session is idle");
4343
* }

‎src/main/java/com/github/copilot/sdk/events/AssistantIntentEvent.java‎

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -32,17 +32,6 @@ public void setData(AssistantIntentData data) {
3232
}
3333

3434
@JsonIgnoreProperties(ignoreUnknown = true)
35-
public static class AssistantIntentData {
36-
37-
@JsonProperty("intent")
38-
private String intent;
39-
40-
public String getIntent() {
41-
return intent;
42-
}
43-
44-
public void setIntent(String intent) {
45-
this.intent = intent;
46-
}
35+
public record AssistantIntentData(@JsonProperty("intent") String intent) {
4736
}
4837
}

‎src/main/java/com/github/copilot/sdk/events/AssistantMessageDeltaEvent.java‎

Lines changed: 4 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -32,50 +32,9 @@ public void setData(AssistantMessageDeltaData data) {
3232
}
3333

3434
@JsonIgnoreProperties(ignoreUnknown = true)
35-
public static class AssistantMessageDeltaData {
36-
37-
@JsonProperty("messageId")
38-
private String messageId;
39-
40-
@JsonProperty("deltaContent")
41-
private String deltaContent;
42-
43-
@JsonProperty("totalResponseSizeBytes")
44-
private Double totalResponseSizeBytes;
45-
46-
@JsonProperty("parentToolCallId")
47-
private String parentToolCallId;
48-
49-
public String getMessageId() {
50-
return messageId;
51-
}
52-
53-
public void setMessageId(String messageId) {
54-
this.messageId = messageId;
55-
}
56-
57-
public String getDeltaContent() {
58-
return deltaContent;
59-
}
60-
61-
public void setDeltaContent(String deltaContent) {
62-
this.deltaContent = deltaContent;
63-
}
64-
65-
public Double getTotalResponseSizeBytes() {
66-
return totalResponseSizeBytes;
67-
}
68-
69-
public void setTotalResponseSizeBytes(Double totalResponseSizeBytes) {
70-
this.totalResponseSizeBytes = totalResponseSizeBytes;
71-
}
72-
73-
public String getParentToolCallId() {
74-
return parentToolCallId;
75-
}
76-
77-
public void setParentToolCallId(String parentToolCallId) {
78-
this.parentToolCallId = parentToolCallId;
79-
}
35+
public record AssistantMessageDeltaData(@JsonProperty("messageId") String messageId,
36+
@JsonProperty("deltaContent") String deltaContent,
37+
@JsonProperty("totalResponseSizeBytes") Double totalResponseSizeBytes,
38+
@JsonProperty("parentToolCallId") String parentToolCallId) {
8039
}
8140
}

0 commit comments

Comments
 (0)