build: properly fix JaCoCo report generation.

Fix #1239

Author: php-coder

pom.xml

@@ -494,6 +494,15 @@
 		https://github.com/spring-projects/spring-boot/blob/v2.0.9.RELEASE/spring-boot-project/spring-boot-dependencies/pom.xml
 	-->
 	<properties>
+		<!--
+			Workaround for a bug in surefire plugin: it doesn't respect sourceEncoding
+			and doesn't pass it into a forked process.
+			See: https://stackoverflow.com/questions/17656475/maven-source-encoding-in-utf-8-not-working
+			
+			Note: don't put this property within surefire-maven-plugin as it would break jacoco-maven-plugin.
+		-->
+		<argLine>-Dfile.encoding=${project.build.sourceEncoding} -Xss256k</argLine>
+		
 		<!-- Redefine default value from spring-boot-dependencies -->
 		<assertj.version>3.14.0</assertj.version>
 
@@ -955,12 +964,6 @@
 				<configuration>
 					<skipTests>${skipUnitTests}</skipTests>
 					<disableXmlReport>${disableXmlReport}</disableXmlReport>
-					<!--
-						Workaround for a bug in surefire plugin: it doesn't respect sourceEncoding
-						and doesn't pass it into a forked process.
-						See: https://stackoverflow.com/questions/17656475/maven-source-encoding-in-utf-8-not-working
-					-->
-					<argLine>-Dfile.encoding=${project.build.sourceEncoding} -Xss256k</argLine>
 				</configuration>
 			</plugin>