remove some warnings in tests

Author: lecousin

net.lecousin.core/.checkstyle

@@ -1,10 +1,10 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
-<fileset-config file-format-version="1.2.0" simple-config="true" sync-formatter="false">
+<fileset-config file-format-version="1.2.0" simple-config="false" sync-formatter="false">
   <local-check-config name="parent-pom" location="/net.lecousin.parent-pom/src/resources/checkstyle.xml" type="project" description="">
     <additional-data name="protect-config-file" value="false"/>
   </local-check-config>
   <fileset name="all" enabled="true" check-config-name="parent-pom" local="true">
-    <file-match-pattern match-pattern="." include-pattern="true"/>
+    <file-match-pattern match-pattern="src/main/java/." include-pattern="true"/>
   </fileset>
 </fileset-config>

net.lecousin.core/pom.xml

@@ -1,7 +1,6 @@
 <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
 	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
 	<modelVersion>4.0.0</modelVersion>
-	<groupId>net.lecousin</groupId>
 	<artifactId>core</artifactId>
 	<version>0.9.10</version>
 

net.lecousin.core/src/main/java/net/lecousin/framework/io/serialization/AbstractSerializationSpecWriter.java

@@ -76,6 +76,7 @@ protected List<SerializationRule> addRulesForAttribute(Attribute a, List<Seriali
 
 	protected abstract ISynchronizationPoint<? extends Exception> specifyAnyValue(SerializationContext context);
 
+	/** Can be used when already initialized. */
 	public ISynchronizationPoint<? extends Exception> specifyValue(
 		SerializationContext context, TypeDefinition typeDef, List<SerializationRule> rules
 	) {

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/io/TestIO.java

@@ -111,6 +111,7 @@ protected long getFileSize() {
 
 	protected abstract IO getIOForCommonTests() throws Exception;
 
+	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public void testBasicCommonFunctions() throws Exception {
 		IO io = getIOForCommonTests();

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/io/TestReadWrite.java

@@ -68,7 +68,6 @@ public <T extends IO.Readable.Seekable & IO.Writable.Seekable> void testWriteThe
 		io.close();
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public <T extends IO.Readable.Seekable & IO.Writable.Seekable> void testRandomWriteAndReadAsync() throws Exception {
 		Assume.assumeTrue(nbBuf > 0);
@@ -336,7 +335,6 @@ void dichotomicWriteSync(T io, int bufStart, int bufEnd, SeekType seekType) thro
 		io.writeSync(ByteBuffer.wrap(testBuf, 0, testBuf.length / 2));
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public <T extends IO.Readable.Seekable & IO.Writable.Seekable> void testSeekAsyncWriteOddEvenThenReadReverse() throws Exception {
 		Assume.assumeTrue(nbBuf > 0);

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/io/TestReadWriteResizable.java

@@ -107,7 +107,6 @@ void testResizeSync() throws Exception {
 		io.close();
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public <T extends IO.Readable.Seekable & IO.Writable.Seekable & IO.KnownSize & IO.Resizable>
 	void testResizeAsync() throws Exception {

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/io/TestReadable.java

@@ -115,7 +115,6 @@ public void testReadableBufferByBufferFullySyncBigBuffer() throws Exception {
 		io.close();
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public void testReadableBufferByBufferFullyAsync() throws Exception {
 		IO.Readable io = createReadableFromFile(openFile(), getFileSize());
@@ -197,7 +196,6 @@ public void run() {
 		io.close();
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public void testReadableBufferByBufferFullyAsyncBigBuffer() throws Exception {
 		IO.Readable io = createReadableFromFile(openFile(), getFileSize());
@@ -258,7 +256,6 @@ public void run() {
 		io.close();
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public void testReadableBufferByBufferAsync() throws Exception {
 		IO.Readable io = createReadableFromFile(openFile(), getFileSize());
@@ -353,7 +350,6 @@ public void testReadableSkipSync() throws Exception {
 		io.close();
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public void testReadableSkipAsync() throws Exception {
 		Assume.assumeTrue(nbBuf > 0);

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/io/TestReadableSeekable.java

@@ -76,7 +76,6 @@ public void testSeekableByteByByteSync() throws Exception {
 		io.close();
 	}
 
-	@SuppressWarnings({ "resource" })
 	@Test(timeout=300000)
 	public void testSeekableByteByByteAsync() throws Exception {
 		IO.Readable.Seekable io = createReadableSeekableFromFile(openFile(), getFileSize());
@@ -186,7 +185,6 @@ public void testSeekableBufferByBufferFullySync() throws Exception {
 		io.close();
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public void testSeekableBufferByBufferFullyAsync() throws Exception {
 		IO.Readable.Seekable io = createReadableSeekableFromFile(openFile(), getFileSize());
@@ -349,7 +347,6 @@ private void testSeekSync(IO.Readable.Seekable io, SeekType type, long move, lon
 		}
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public void testSeekAsync() throws Exception {
 		long size = getFileSize();
@@ -506,7 +503,6 @@ public void testSeekSyncWrong() throws Exception {
 		io.close();
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public void testSeekAsyncWrong() throws Exception {
 		long size = getFileSize();

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/io/TestWritableToFile.java

@@ -77,7 +77,6 @@ public void testWriteBufferByBufferSync() throws Exception {
 		checkFile(file, testBuf, nbBuf);
 	}
 
-	@SuppressWarnings("resource")
 	@Test(timeout=120000)
 	public void testWriteBufferByBufferAsync() throws Exception {
 		Assume.assumeTrue(nbBuf > 0);

net.lecousin.core/src/test/java/net/lecousin/framework/core/test/serialization/TestSerialization.java

@@ -437,6 +437,7 @@ public void testMap() throws Exception {
 		map.put(Integer.valueOf(2), "World");
 		map.put(Integer.valueOf(3), "!");
 		map.put(Integer.valueOf(4), null);
+		@SuppressWarnings("resource")
 		MemoryIO io = serializeInMemory(map, new TypeDefinition(HashMap.class, new TypeDefinition(Integer.class), new TypeDefinition(String.class)));
 		print(io, map);
 		Map<Integer,String> map2 = deserialize(io, new TypeDefinition(HashMap.class, new TypeDefinition(Integer.class), new TypeDefinition(String.class)));
@@ -791,6 +792,7 @@ public void testMergeAttributesOfListElements2() throws Exception {
 		m = new Merged();
 		m.aString = "World";
 		t.list.add(new Pair<>("le monde", m));
+		@SuppressWarnings("resource")
 		MemoryIO io = serializeInMemory(t, new TypeDefinition(ToMerge3.class));
 		print(io, t);
 		deserialize(io, new TypeDefinition(Merged3.class));
@@ -978,6 +980,7 @@ protected <T> T deserialize(IO.Readable io, TypeDefinition type) throws Exceptio
 	protected void testSpec(Class<?> type, MemoryIO serialization) throws Exception {
 		SerializationSpecWriter sw = createSpecWriter();
 		if (sw == null) return;
+		@SuppressWarnings("resource")
 		MemoryIO io = new MemoryIO(1024, "test");
 		sw.writeSpecification(type, io, new ArrayList<>(0)).blockThrow(0);
 		printSpec(io, type);
@@ -1096,6 +1099,7 @@ protected void printSpec(IO.Readable.Seekable io, Class<?> type) throws Exceptio
 		System.out.println("Serialization specification for " + type.getName() + "\r\n" + content);
 	}
 
+	@SuppressWarnings("unused")
 	protected void checkSpec(IO.Readable.Seekable spec, Class<?> type, IO.Readable.Seekable serialization) throws Exception {
 	}
 }