Skip to content

Commit 4fcddee

Browse files
Fruank4claude
andcommitted
fix: update StaticLongTermMemoryHookTest to expect memory at index 0
The test was asserting memory message at index 1 (end), which matched the old buggy behavior. Update to assert index 0 (beginning) to align with the fix in #1164. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent e989644 commit 4fcddee

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

agentscope-core/src/test/java/io/agentscope/core/memory/StaticLongTermMemoryHookTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -138,13 +138,13 @@ void testOnEventWithPreReasoningEventAndRetrievedMemories() {
138138
resultEvent -> {
139139
List<Msg> messages = resultEvent.getInputMessages();
140140
assertEquals(2, messages.size());
141-
assertEquals(MsgRole.SYSTEM, messages.get(1).getRole());
141+
assertEquals(MsgRole.SYSTEM, messages.get(0).getRole());
142142
assertTrue(
143-
messages.get(1)
143+
messages.get(0)
144144
.getTextContent()
145145
.contains("<long_term_memory>"));
146146
assertTrue(
147-
messages.get(1)
147+
messages.get(0)
148148
.getTextContent()
149149
.contains("User prefers dark mode"));
150150
})

0 commit comments

Comments
 (0)