Skip to content

Commit b7830db

Browse files
Vladimir.ShapkinVladimir.Shapkin
authored andcommitted
Moved test-lib from src/test/resources to src/test.
Replaced while and if/else to stream.
1 parent 633b721 commit b7830db

File tree

6 files changed

+21
-46
lines changed

6 files changed

+21
-46
lines changed

pom.xml

Lines changed: 7 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,8 @@ SOFTWARE.
8282
</site>
8383
</distributionManagement>
8484
<properties>
85-
<additionalClasspathElements>${project.build.directory}/${project.artifactId}-test-lib-${project.version}.jar</additionalClasspathElements>
85+
<testLibPath>src/test/test-lib</testLibPath>
86+
<testLibJarPath>${project.build.directory}/${project.artifactId}-test-lib-${project.version}.jar</testLibJarPath>
8687
</properties>
8788
<dependencies>
8889
<dependency>
@@ -133,24 +134,6 @@ SOFTWARE.
133134
</dependency>
134135
</dependencies>
135136
<build>
136-
<resources>
137-
<resource>
138-
<directory>${basedir}/src/test/resources</directory>
139-
<filtering>true</filtering>
140-
<excludes>
141-
<exclude>test-lib/**</exclude>
142-
</excludes>
143-
</resource>
144-
</resources>
145-
<testResources>
146-
<testResource>
147-
<directory>${basedir}/src/test/resources</directory>
148-
<filtering>true</filtering>
149-
<excludes>
150-
<exclude>test-lib/**</exclude>
151-
</excludes>
152-
</testResource>
153-
</testResources>
154137
<plugins>
155138
<plugin>
156139
<groupId>org.pitest</groupId>
@@ -165,7 +148,9 @@ SOFTWARE.
165148
<threads>8</threads>
166149
<mutationThreshold>75</mutationThreshold>
167150
<timeoutConstant>500</timeoutConstant>
168-
<additionalClasspathElements>${additionalClasspathElements}</additionalClasspathElements>
151+
<additionalClasspathElements>
152+
<additionalClasspathElement>${testLibJarPath}</additionalClasspathElement>
153+
</additionalClasspathElements>
169154
<useClasspathJar>true</useClasspathJar>
170155
</configuration>
171156
</execution>
@@ -221,7 +206,7 @@ SOFTWARE.
221206
<configuration>
222207
<finalName>${project.artifactId}-test-lib-${project.version}</finalName>
223208
<descriptors>
224-
<descriptor>${basedir}/src/test/resources/test-lib/assembly.xml</descriptor>
209+
<descriptor>${testLibPath}/assembly.xml</descriptor>
225210
</descriptors>
226211
<appendAssemblyId>false</appendAssemblyId>
227212
</configuration>
@@ -239,7 +224,7 @@ SOFTWARE.
239224
<goal>install-file</goal>
240225
</goals>
241226
<configuration>
242-
<file>${additionalClasspathElements}</file>
227+
<file>${testLibJarPath}</file>
243228
<groupId>org.cactoos</groupId>
244229
<artifactId>${project.artifactId}-test-lib</artifactId>
245230
<version>${project.version}</version>

src/main/java/org/cactoos/io/ResourceOf.java

Lines changed: 13 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,6 @@
2828
import java.net.URISyntaxException;
2929
import java.net.URL;
3030
import java.nio.charset.StandardCharsets;
31-
import java.util.ArrayList;
32-
import java.util.Enumeration;
33-
import java.util.List;
3431
import java.util.jar.JarEntry;
3532
import java.util.jar.JarFile;
3633
import java.util.stream.Collectors;
@@ -293,28 +290,21 @@ private static final class JarDirectoryFileNameStream {
293290
* @throws Exception If something goes wrong
294291
*/
295292
public InputStream files() throws Exception {
296-
final List<String> names = new ArrayList<>(2);
297293
try (JarFile jar = new JarFile(this.extract())) {
298-
final Enumeration<JarEntry> entries = jar.entries();
299-
while (entries.hasMoreElements()) {
300-
final JarEntry entry = entries.nextElement();
301-
final String name = entry.getName();
302-
if (this.path.equals(name)) {
303-
continue;
304-
} else if (name.lastIndexOf(this.path) >= 0) {
305-
names.add(name.substring(this.path.length()));
306-
}
307-
}
294+
return
295+
new InputStreamOf(
296+
jar
297+
.stream()
298+
.map(JarEntry::getName)
299+
.filter(
300+
name -> !this.path.equals(name)
301+
&& name.lastIndexOf(this.path) >= 0
302+
)
303+
.map(name -> name.substring(this.path.length()))
304+
.collect(Collectors.joining("\n"))
305+
.getBytes(StandardCharsets.UTF_8)
306+
);
308307
}
309-
return
310-
new InputStreamOf(
311-
names
312-
.stream()
313-
.collect(
314-
Collectors.joining("\n")
315-
)
316-
.getBytes(StandardCharsets.UTF_8)
317-
);
318308
}
319309

320310
/**

src/test/resources/test-lib/assembly.xml renamed to src/test/test-lib/assembly.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ SOFTWARE.
3030
<includeBaseDirectory>false</includeBaseDirectory>
3131
<fileSets>
3232
<fileSet>
33-
<directory>${project.basedir}/src/test/resources/test-lib</directory>
33+
<directory>${project.basedir}/${testLibPath}</directory>
3434
<outputDirectory>/</outputDirectory>
3535
<includes>
3636
<include>org/cactoos/io/small-text-file.txt</include>
File renamed without changes.
File renamed without changes.
File renamed without changes.

0 commit comments

Comments
 (0)