devinbost commented on pull request #10548:
URL: https://github.com/apache/pulsar/pull/10548#issuecomment-839598310
hmm this test passes:
```
@Test
public void testOpEntryAdd_toString_doesNotThrowNPE(){
ManagedLedger ml = mock(ManagedLedger.class);
LedgerHandle ledger = mock(LedgerHandle.class);
when(ml.getName()).thenReturn(null);
when(ledger.getId()).thenReturn(124L);
long entryId = 12L;
long startTime = 1245L;
int dataLength = 566;
String test = "OpAddEntry{" +
"mlName=" + ml != null ? ml.getName() : "null" +
"ledgerId=" + ledger != null ?
String.valueOf(ledger.getId()) : "null" +
", entryId=" + entryId +
", startTime=" + startTime +
", dataLength=" + dataLength +
'}';
}
```
@lhotari maybe you can spot something I'm not seeing
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
