Skip to content

Commit

Permalink
[Misc] Excluding crypto modules from pitest checks, see comment in co…
Browse files Browse the repository at this point in the history
  • Loading branch information
vmassol committed Mar 12, 2019
1 parent 902dd38 commit 962e394
Show file tree
Hide file tree
Showing 6 changed files with 24 additions and 5 deletions.
24 changes: 24 additions & 0 deletions xwiki-commons-core/xwiki-commons-crypto/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -69,4 +69,28 @@
<module>xwiki-commons-crypto-signer</module>
<module>xwiki-commons-crypto-store</module>
</modules>
<build>
<pluginManagement>
<plugins>
<!-- Exclude Crypto tests from PIT/Descartes since 1) those tests are special (if they don't have enough entropy
they execute slowyl which can let to timeouts and break mutation scores and 2) they are very slow to execute
and thus take even more time with mutations... -->
<plugin>
<groupId>org.pitest</groupId>
<artifactId>pitest-maven</artifactId>
<executions>
<execution>
<id>pitest-check</id>
<goals>
<goal>mutationCoverage</goal>
</goals>
<configuration>
<skip>true</skip>
</configuration>
</execution>
</executions>
</plugin>
</plugins>
</pluginManagement>
</build>
</project>
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@
<description>Provides confidentiality throught encryption</description>
<properties>
<xwiki.jacoco.instructionRatio>0.70</xwiki.jacoco.instructionRatio>
<xwiki.pitest.mutationThreshold>36</xwiki.pitest.mutationThreshold>
<!-- Name to display by the Extension Manager -->
<xwiki.extension.name>Cryptographic Services for Confidentiality</xwiki.extension.name>
</properties>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@
<description>API common to all crypto modules</description>
<properties>
<xwiki.jacoco.instructionRatio>0.62</xwiki.jacoco.instructionRatio>
<xwiki.pitest.mutationThreshold>69</xwiki.pitest.mutationThreshold>
<!-- Name to display by the Extension Manager -->
<xwiki.extension.name>Cryptographic Services Common API</xwiki.extension.name>
<!-- Cryptography requires complex expressions, allow a few such expressions -->
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@
<description>Provides passwords encryption and validation</description>
<properties>
<xwiki.jacoco.instructionRatio>0.65</xwiki.jacoco.instructionRatio>
<xwiki.pitest.mutationThreshold>81</xwiki.pitest.mutationThreshold>
<!-- Name to display by the Extension Manager -->
<xwiki.extension.name>Cryptographic Services for Password Encryption</xwiki.extension.name>
</properties>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@
<description>Provides X.509 certificates management</description>
<properties>
<xwiki.jacoco.instructionRatio>0.76</xwiki.jacoco.instructionRatio>
<xwiki.pitest.mutationThreshold>82</xwiki.pitest.mutationThreshold>
<!-- Name to display by the Extension Manager -->
<xwiki.extension.name>Cryptographic Services for Authentication</xwiki.extension.name>
<!-- Working with cryptography requires a lot of classes, relax the coupling requirements -->
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@
<description>Provides integrity using signature</description>
<properties>
<xwiki.jacoco.instructionRatio>0.71</xwiki.jacoco.instructionRatio>
<xwiki.pitest.mutationThreshold>75</xwiki.pitest.mutationThreshold>
<!-- Name to display by the Extension Manager -->
<xwiki.extension.name>Cryptographic Services for Integrity</xwiki.extension.name>
</properties>
Expand Down

0 comments on commit 962e394

Please sign in to comment.