Skip to content

Commit

Permalink
Remove certain blocks from Azure Communication Services Phone Numbers…
Browse files Browse the repository at this point in the history
… pom.xml to try to get code coverage set up. (Azure#35847)

* Changes for SMS 1.1.2 -> 1.1.3

* Remove redundant code to set up code coverage.

---------

Co-authored-by: Adrian Tang <[email protected]>
  • Loading branch information
whisper6284 and Adrian Tang authored Jul 12, 2023
1 parent 9d58bfb commit b2a3869
Showing 1 changed file with 0 additions and 21 deletions.
21 changes: 0 additions & 21 deletions sdk/communication/azure-communication-phonenumbers/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,6 @@
<test.dir>src/test</test.dir>
<jacoco.min.linecoverage>0.68</jacoco.min.linecoverage>
<jacoco.min.branchcoverage>0.66</jacoco.min.branchcoverage>
<jacoco.skip>false</jacoco.skip>
<!-- Configures the Java 9+ run to perform the required module exports, opens, and reads that are necessary for testing but shouldn't be part of the module-info. -->
<javaModulesSurefireArgLine>
--add-opens com.azure.communication.phonenumbers/com.azure.communication.phonenumbers=ALL-UNNAMED
Expand Down Expand Up @@ -127,26 +126,6 @@

<build>
<plugins>
<plugin>
<groupId>org.jacoco</groupId>
<artifactId>jacoco-maven-plugin</artifactId>
<version>0.8.9</version> <!-- {x-version-update;org.jacoco:jacoco-maven-plugin;external_dependency} -->
<configuration>
<includes>
<include>com/azure/communication/phonenumbers/*.class</include>
<include>com/azure/communication/phonenumbers/siprouting/*.class</include>
</includes>
</configuration>
<executions>
<execution>
<id>coverage-report</id>
<phase>verify</phase>
<goals>
<goal>report</goal>
</goals>
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-enforcer-plugin</artifactId>
Expand Down

0 comments on commit b2a3869

Please sign in to comment.