Skip to content

Commit b978577

Browse files
committed
extract duplicated code to Resources.open
1 parent ee4b46d commit b978577

File tree

5 files changed

+16
-27
lines changed

5 files changed

+16
-27
lines changed

e2e/src/test/java/org.dhatim.fastexcel.benchmarks/BenchmarkLauncher.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@
88
import org.openjdk.jmh.runner.options.Options;
99
import org.openjdk.jmh.runner.options.OptionsBuilder;
1010

11-
import java.util.concurrent.TimeUnit;
1211
import java.util.regex.Pattern;
1312

1413
@BenchmarkMode(Mode.SingleShotTime)

fastexcel-reader/src/test/java/org/dhatim/fastexcel/reader/FastExcelReaderTest.java

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717

1818
import static java.util.stream.Collectors.toList;
1919
import static org.assertj.core.api.Assertions.*;
20+
import static org.dhatim.fastexcel.reader.Resources.open;
2021

2122
import java.io.IOException;
2223
import java.io.InputStream;
@@ -215,14 +216,6 @@ private static String toODT(Date date) {
215216
return FORMAT.format(date);
216217
}
217218

218-
private static InputStream open(String name) {
219-
InputStream result = FastExcelReaderTest.class.getResourceAsStream(name);
220-
if (result == null) {
221-
throw new IllegalStateException("Cannot read resource " + name);
222-
}
223-
return result;
224-
}
225-
226219
private static String getRawValue(org.apache.poi.ss.usermodel.Cell cell) {
227220
XSSFCell xcell = ((XSSFCell) cell);
228221
return xcell == null ? null : xcell.getRawValue();

fastexcel-reader/src/test/java/org/dhatim/fastexcel/reader/HeaderTest.java

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ void testOOXMLZipFile() throws IOException {
4646
}
4747

4848
private static byte[] readFirstBytes(String name, int length) throws IOException {
49-
try (InputStream is = open(name)) {
49+
try (InputStream is = Resources.open(name)) {
5050
byte[] bytes = new byte[length];
5151
readNBytes(is, bytes, 0, length);
5252
return bytes;
@@ -66,12 +66,4 @@ private static int readNBytes(InputStream is, byte[] b, int off, int len) throws
6666
return n;
6767
}
6868

69-
private static InputStream open(String name) {
70-
InputStream result = HeaderTest.class.getResourceAsStream(name);
71-
if (result == null) {
72-
throw new IllegalStateException("Cannot read resource " + name);
73-
}
74-
return result;
75-
}
76-
7769
}
Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
package org.dhatim.fastexcel.reader;
2+
3+
import java.io.InputStream;
4+
5+
public class Resources {
6+
static InputStream open(String name) {
7+
InputStream result = Resources.class.getResourceAsStream(name);
8+
if (result == null) {
9+
throw new IllegalStateException("Cannot read resource " + name);
10+
}
11+
return result;
12+
}
13+
}

fastexcel-reader/src/test/java/org/dhatim/fastexcel/reader/SimpleReaderTest.java

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ class SimpleReaderTest {
4242

4343
@Test
4444
void test() throws IOException {
45-
try (InputStream is = openResource("/xlsx/simple.xlsx"); ReadableWorkbook wb = new ReadableWorkbook(is)) {
45+
try (InputStream is = Resources.open("/xlsx/simple.xlsx"); ReadableWorkbook wb = new ReadableWorkbook(is)) {
4646
Sheet sheet = wb.getFirstSheet();
4747
try (Stream<Row> rows = sheet.openStream()) {
4848
rows.forEach(r -> {
@@ -61,14 +61,6 @@ void test() throws IOException {
6161
}
6262
}
6363

64-
private static InputStream openResource(String name) {
65-
InputStream result = SimpleReaderTest.class.getResourceAsStream(name);
66-
if (result == null) {
67-
throw new IllegalStateException("Cannot read resource " + name);
68-
}
69-
return result;
70-
}
71-
7264
private static LocalDateTime date(int year, int month, int day) {
7365
return LocalDateTime.of(year, month, day, 0, 0);
7466
}

0 commit comments

Comments
 (0)