Skip to content

Commit e82018c

Browse files
committed
fix failing test
1 parent b294c2b commit e82018c

File tree

1 file changed

+17
-11
lines changed

1 file changed

+17
-11
lines changed

src/test/java/org/cryptomator/cryptofs/health/dirid/OrphanContentDirTest.java

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,8 @@
3232
import java.util.UUID;
3333
import java.util.concurrent.atomic.AtomicReference;
3434

35+
import static org.mockito.Mockito.mockStatic;
36+
3537
public class OrphanContentDirTest {
3638

3739
@TempDir
@@ -157,8 +159,8 @@ public void init() throws IOException {
157159
@Test
158160
@DisplayName("prepareStepParent() runs without error on not-existing stepparent")
159161
public void testPrepareStepParent() throws IOException {
160-
try (var uuidClass = Mockito.mockStatic(UUID.class); //
161-
var dirIdBackupClass = Mockito.mockStatic(DirectoryIdBackup.class)) {
162+
try (var uuidClass = mockStatic(UUID.class); //
163+
var dirIdBackupClass = mockStatic(DirectoryIdBackup.class)) {
162164
UUID uuid = Mockito.mock(UUID.class);
163165
uuidClass.when(UUID::randomUUID).thenReturn(uuid);
164166
Mockito.doReturn("aaaaaa").when(uuid).toString();
@@ -181,8 +183,8 @@ public void testPrepareStepParentExistingStepParentDir() throws IOException {
181183
Path cipherStepparent = dataDir.resolve("22/2222");
182184
Files.createDirectories(cipherStepparent);
183185

184-
try (var uuidClass = Mockito.mockStatic(UUID.class); //
185-
var dirIdBackupClass = Mockito.mockStatic(DirectoryIdBackup.class)) {
186+
try (var uuidClass = mockStatic(UUID.class); //
187+
var dirIdBackupClass = mockStatic(DirectoryIdBackup.class)) {
186188
UUID uuid = Mockito.mock(UUID.class);
187189
uuidClass.when(UUID::randomUUID).thenReturn(uuid);
188190
Mockito.doReturn("aaaaaa").when(uuid).toString();
@@ -205,8 +207,8 @@ public void testPrepareStepParentOrphanedStepParentDir() throws IOException {
205207
Path cipherStepparent = dataDir.resolve("22/2222");
206208
Files.createDirectories(cipherStepparent);
207209

208-
try (var uuidClass = Mockito.mockStatic(UUID.class); //
209-
var dirIdBackupClass = Mockito.mockStatic(DirectoryIdBackup.class)) {
210+
try (var uuidClass = mockStatic(UUID.class); //
211+
var dirIdBackupClass = mockStatic(DirectoryIdBackup.class)) {
210212
UUID uuid = Mockito.mock(UUID.class);
211213
uuidClass.when(UUID::randomUUID).thenReturn(uuid);
212214
Mockito.doReturn("aaaaaa").when(uuid).toString();
@@ -238,7 +240,7 @@ public void init() {
238240
@DisplayName("Successful reading dirId from backup file")
239241
public void success() {
240242
var dirId = new byte[]{'f', 'o', 'o'};
241-
try (var dirIdBackupMock = Mockito.mockStatic(DirectoryIdBackup.class)) {
243+
try (var dirIdBackupMock = mockStatic(DirectoryIdBackup.class)) {
242244
dirIdBackupMock.when(() -> DirectoryIdBackup.read(cryptor, cipherOrphan)).thenReturn(dirId);
243245
var result = resultSpy.retrieveDirId(cipherOrphan, cryptor);
244246
Assertions.assertTrue(result.isPresent());
@@ -250,7 +252,7 @@ public void success() {
250252
@DisplayName("retrieveDirId returns an empty optional on any exception")
251253
@FieldSource("expectedExceptions")
252254
public void testRetrieveDirIdIOExceptionReadingFile(Throwable t) throws IOException {
253-
try (var dirIdBackupMock = Mockito.mockStatic(DirectoryIdBackup.class)) {
255+
try (var dirIdBackupMock = mockStatic(DirectoryIdBackup.class)) {
254256
dirIdBackupMock.when(() -> DirectoryIdBackup.read(cryptor, cipherOrphan)).thenThrow(t);
255257
var notExistingResult = resultSpy.retrieveDirId(cipherOrphan, cryptor);
256258
Assertions.assertTrue(notExistingResult.isEmpty());
@@ -339,7 +341,7 @@ public void testAdoptOrphanedShortened() throws IOException {
339341
Files.createDirectories(stepParentDir.path());
340342

341343
Mockito.doReturn("adopted").when(fileNameCryptor).encryptFilename(Mockito.any(), Mockito.any(), Mockito.any());
342-
try (var baseEncodingClass = Mockito.mockStatic(BaseEncoding.class)) {
344+
try (var baseEncodingClass = mockStatic(BaseEncoding.class)) {
343345
MessageDigest sha1 = Mockito.mock(MessageDigest.class);
344346
Mockito.doReturn(new byte[]{}).when(sha1).digest(Mockito.any());
345347

@@ -367,7 +369,7 @@ public void testAdoptOrphanedShortenedMissingNameC9s() throws IOException {
367369
Files.createDirectories(stepParentDir.path());
368370

369371
Mockito.doReturn("adopted").when(fileNameCryptor).encryptFilename(Mockito.any(), Mockito.any(), Mockito.any());
370-
try (var baseEncodingClass = Mockito.mockStatic(BaseEncoding.class)) {
372+
try (var baseEncodingClass = mockStatic(BaseEncoding.class)) {
371373
MessageDigest sha1 = Mockito.mock(MessageDigest.class);
372374
Mockito.doReturn(new byte[]{}).when(sha1).digest(Mockito.any());
373375

@@ -410,7 +412,11 @@ public void testFixNoDirId() throws IOException {
410412
return null;
411413
}).when(resultSpy).adoptOrphanedResource(Mockito.any(), Mockito.any(), Mockito.anyBoolean(), Mockito.eq(stepParentDir), Mockito.eq(fileNameCryptor), Mockito.any());
412414

413-
resultSpy.fix(pathToVault, config, masterkey, cryptor);
415+
try ( var dirIdBackup = mockStatic(DirectoryIdBackup.class)) {
416+
dirIdBackup.when(() -> DirectoryIdBackup.read(cryptor, cipherOrphan)).thenThrow(IllegalStateException.class);
417+
resultSpy.fix(pathToVault, config, masterkey, cryptor);
418+
}
419+
414420

415421
Mockito.verify(resultSpy, Mockito.times(2)).adoptOrphanedResource(Mockito.any(), Mockito.any(), Mockito.anyBoolean(), Mockito.eq(stepParentDir), Mockito.eq(fileNameCryptor), Mockito.any());
416422
Assertions.assertTrue(Files.notExists(cipherOrphan));

0 commit comments

Comments
 (0)