✅ Release buffer in unit test of ProtocolizedBuiltPacketImpl

YvanMazy · YvanMazy · commit 699fc2647e25 · 2025-05-12T14:51:01.000+02:00
diff --git a/core/src/test/java/net/transferproxy/network/packet/built/ProtocolizedBuiltPacketImplTest.java b/core/src/test/java/net/transferproxy/network/packet/built/ProtocolizedBuiltPacketImplTest.java
@@ -87,7 +87,7 @@ void testBufferIsCorrectlyCached() {
 
         for (int i = 0; i < 20; i++) {
             final int protocol = i;
-            assertDoesNotThrow(() -> builtPacket.get(allocator, protocol));
+            assertDoesNotThrow(() -> builtPacket.get(allocator, protocol)).release();
         }
 
         verify(builtPacket, times(3)).compute(anyInt(), anyInt());

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ void testBufferIsCorrectlyCached() {`
`87`	`87`
`88`	`88`	`for (int i = 0; i < 20; i++) {`
`89`	`89`	`final int protocol = i;`
`90`		`- assertDoesNotThrow(() -> builtPacket.get(allocator, protocol));`
	`90`	`+ assertDoesNotThrow(() -> builtPacket.get(allocator, protocol)).release();`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`verify(builtPacket, times(3)).compute(anyInt(), anyInt());`