Làm cách nào để tôi có thể tạo tài nguyên có thể đóng tự động đúng cách?

Question

Tôi cố gắng để viết một bài kiểm tra đơn vị cho việc này:

try (final DatagramChannel channel = helper.createChannel()) { 

... 

} 

Trong thử nghiệm của tôi, tôi nhạo báng các helper (sử dụng Mockito), và nói với helper.createChannel() để trả về một kênh chế giễu.

thử nghiệm này không thành công với

java.lang.NullPointerException 
at java.nio.channels.spi.AbstractInterruptibleChannel.close(AbstractInterruptibleChannel.java:111) 

Tôi hiểu rằng các thiết bị thử-với-các nguồn lực trong Java gọi phương thức close() trong DatagramChannel lúc đi ra khỏi khối try, nhưng không nên đóng cửa() phương pháp trong DatagramChannel giả được gọi khi?

Trình gỡ lỗi cho tôi biết rằng khóa closeLock trong AbstractInterruptibleChannel là rỗng.

Tôi có nên phân lớp DatagramChannel, ghi đè lên phương thức close() trong đó và sau đó giả lập lớp con của tôi thay thế không? Hoặc, tôi đang làm điều gì đó sai trái một cách sâu sắc hơn (mô hình trợ giúp trả về một mô hình)?

Kính trọng, Fredrik Israelsson

mã kiểm tra, theo yêu cầu:

@Mock 
private InetAddress peerAddress; 
@Mock 
private UDPChannelHelper helper; 
@Mock 
private DatagramChannel channel; 

private UDPTransportImpl transport; 

@Before 
public void setUp() throws Exception { 
    MockitoAnnotations.initMocks(this); 
    when(helper.createChannel()).thenReturn(channel); 
    transport = new UDPTransportImpl(peerAddress, 0, helper); 
} 

@Test 
public void testNormalSubmit() throws Exception { 
    transport.submit("Hello"); 
} 

Như bạn thấy, tôi không chỉ định bất kỳ hành vi cho channel.close(). Tôi tin rằng tôi không cần phải, bởi vì close() trả về void.

Answer 1

Bạn đang chế nhạo một lớp thực tế DatagramChannel, mở rộng AbstractInterruptibleChannel. Tuy nhiên, AbstractInterruptibleChannel.close là cuối cùng và Mockito hiện không thể giả lập mã cuối cùng. Điều này giải thích tại sao bạn có NPE trong mã.

Tôi phải nhắc bạn rằng người ta thường chấp nhận rằng các loại chế nhạo bạn không sở hữu là thực hành không tốt. Tôi đã nhìn thấy mọi người làm điều đó và họ đã có những bất ngờ xấu sau này khi việc thực hiện thực sự đã thay đổi, nhưng hành vi giả lập không, vì vậy họ nghĩ rằng mọi thứ đều ổn khi họ cập nhật phiên bản của thư viện.

Vẫn còn nếu bạn muốn tiếp tục theo cách này bởi vì bạn có lý do hợp lệ cho điều đó (và có một số), bạn có thể quay trở lại thay vì mô hình giao diện, như Channel thực sự mở rộng Closeable. Hoặc bạn có thể sử dụng bất kỳ giao diện nào khác mà bạn cần tương tác với giao diện đã có trong DatagramChannel. Ngoài ra nếu bạn cần nhiều hơn một giao diện chỉ cần sử dụng mock(Channel.class, withSetting().extraInterfaces(...)).

Hy vọng rằng sẽ giúp Chúc mừng, Brice

Answer 2

Giữ dành cho dù bạn nên làm điều này hay không, một cách để bạn có thể làm việc xung quanh vấn đề này là do "sửa chữa" các AbstractInterruptibleChannel giả dụ (cho dù là một FileChannel, một DatagramChannel, vv) bằng cách cung cấp một đối tượng cho trường closeLock được sử dụng để đồng bộ hóa cuộc gọi gần.

private static void fixChannelMock(AbstractInterruptibleChannel mockFileChannel) throws Exception { 
    Field closeLockField = AbstractInterruptibleChannel.class.getDeclaredField("closeLock"); 
    closeLockField.setAccessible(true); 
    closeLockField.set(mockFileChannel, new Object()); 
} 

Hãy chuẩn bị để sửa mã trên trên bản phát hành Java nhỏ mặc dù việc triển khai nội bộ AbstractInterruptibleChannel có thể thay đổi.

Answer 3

Tôi đã có cùng một vấn đề và sử dụng gián điệp (..) thay vì giả (..) đã làm việc cho tôi.Tôi đã cố gắng để mô phỏng một lỗi trong khi cắt xén một tập tin và nếu hệ thống của tôi đã xử lý lỗi cho phù hợp.

FileChannel fileChannel = spy(FileChannel.class); 
mockStatic(FileChannel.class); 
when(FileChannel.open(eq(filePath), eq(StandardOpenOption.WRITE))).thenReturn(fileChannel); 
when(fileChannel.truncate(1000L)).thenThrow(new IOException("Unable to truncate file")); 

... 

// Snippet being tested! 
fileChannel = FileChannel.open(filePath, StandardOpenOption.WRITE); 
fileChannel.truncate(1000L); // Will throw the exception!