Skip to content

Commit 0a8ccab

Browse files
committed
And more file test fixes for canonical path
1 parent b0deb6b commit 0a8ccab

2 files changed

Lines changed: 7 additions & 14 deletions

File tree

spring-integration-file/src/test/java/org/springframework/integration/file/FileReadingMessageSourcePersistentFilterIntegrationTests.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
import org.junit.jupiter.api.Test;
2424
import org.junit.jupiter.api.io.TempDir;
2525

26-
import org.springframework.beans.DirectFieldAccessor;
2726
import org.springframework.beans.factory.annotation.Autowired;
2827
import org.springframework.messaging.Message;
2928
import org.springframework.test.annotation.DirtiesContext;
@@ -61,12 +60,6 @@ public void cleanupInputDir() {
6160
}
6261
}
6362

64-
@Test
65-
public void configured() {
66-
DirectFieldAccessor accessor = new DirectFieldAccessor(this.pollableFileSource);
67-
assertThat(accessor.getPropertyValue("directory")).isEqualTo(inputDir);
68-
}
69-
7063
@Test
7164
public void getFiles() {
7265
Message<File> received1 = this.pollableFileSource.receive();

spring-integration-file/src/test/java/org/springframework/integration/file/WatchServiceDirectoryScannerTests.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -146,10 +146,10 @@ public boolean remove(File fileToRemove) {
146146
accum.addAll(files);
147147
}
148148
assertThat(accum).hasSize(4);
149-
assertThat(accum).contains(top2);
150-
assertThat(accum).contains(foo2);
151-
assertThat(accum).contains(bar2);
152-
assertThat(accum).contains(baz1);
149+
assertThat(accum).contains(top2.getCanonicalFile());
150+
assertThat(accum).contains(foo2.getCanonicalFile());
151+
assertThat(accum).contains(bar2.getCanonicalFile());
152+
assertThat(accum).contains(baz1.getCanonicalFile());
153153

154154
/*See AbstractWatchKey#signalEvent source code:
155155
if(var5 >= 512) {
@@ -160,7 +160,7 @@ public boolean remove(File fileToRemove) {
160160
List<File> filesForOverflow = new ArrayList<>(600);
161161

162162
for (int i = 0; i < 600; i++) {
163-
filesForOverflow.add(File.createTempFile("tmp" + i, null, this.rootDir));
163+
filesForOverflow.add(File.createTempFile("tmp" + i, null, this.rootDir).getCanonicalFile());
164164
}
165165

166166
n = 0;
@@ -183,7 +183,7 @@ public boolean remove(File fileToRemove) {
183183
accum.addAll(files);
184184
}
185185

186-
assertThat(accum).contains(baz2);
186+
assertThat(accum).contains(baz2.getCanonicalFile());
187187

188188
File baz2Copy = new File(baz2.getAbsolutePath());
189189

@@ -198,7 +198,7 @@ public boolean remove(File fileToRemove) {
198198
}
199199

200200
assertThat(files).hasSize(1);
201-
assertThat(files).contains(baz2);
201+
assertThat(files).contains(baz2.getCanonicalFile());
202202

203203
baz2.delete();
204204

0 commit comments

Comments
 (0)