Merge pull request #3835 from senivam/travisBuildProfile

Travis CI build script + fixes to pass the tests

Author: jansupol

.travis.yml

@@ -0,0 +1,13 @@
+language: java
+
+jdk:
+  - oraclejdk8
+
+cache:
+  directories:
+    - .autoconf
+
+install: true
+
+script:
+  - bash travis.sh

README.md

@@ -12,6 +12,8 @@
 [//]: # "  "
 [//]: # " SPDX-License-Identifier: EPL-2.0 OR GPL-2.0 WITH Classpath-exception-2.0 "
 
+[![Build Status](https://travis-ci.org/eclipse-ee4j/jersey.svg?branch=master)](https://travis-ci.org/eclipse-ee4j/jersey)
+
 ### About Jersey
 
 Jersey is a REST framework that provides JAX-RS Reference Implementation and more.

connectors/jdk-connector/src/main/java/org/glassfish/jersey/jdk/connector/internal/HttpConnection.java

@@ -127,7 +127,7 @@ public void completed(HttpRequest result) {
         });
     }
 
-    synchronized void close() {
+    void close() {
         if (state == State.CLOSED) {
             return;
         }
@@ -218,6 +218,9 @@ protected Filter<HttpRequest, HttpResponse, HttpRequest, HttpResponse> createFil
     }
 
     private void changeState(State newState) {
+        if (state == State.CLOSED) {
+            return;
+        }
         State old = state;
         state = newState;
 
@@ -414,9 +417,11 @@ void processSslHandshakeCompleted() {
 
         @Override
         void processConnectionClosed() {
-            cancelAllTimeouts();
-            changeState(State.CLOSED_BY_SERVER);
-            HttpConnection.this.close();
+            synchronized (HttpConnection.this) {
+                cancelAllTimeouts();
+                changeState(State.CLOSED_BY_SERVER);
+                HttpConnection.this.close();
+            }
         }
 
         @Override

pom.xml

@@ -764,6 +764,7 @@
             <properties>
                 <release.tests.args>-Dskip.tests=true</release.tests.args>
                 <skip.tests>true</skip.tests>
+                <skip.e2e>true</skip.e2e>
             </properties>
         </profile>
         <profile>
@@ -1172,6 +1173,37 @@
                 </pluginManagement>
             </build>
         </profile>
+        <profile>
+            <!--
+            Profile is aimed to run the build on travis
+            due to travis limitations for output (max 4MB) this profile is used along with grep which reduces
+            the output.
+            However some e2e tests produce output which is not grepped (thus is not visible) and run longer than
+            10 minutes which results in the whole build is being murdered by Travis because of death suspection
+
+            the whole build is run as clean install but excludes several e2e tests because of the not grepped output
+            -->
+            <id>travis_e2e_skip</id>
+            <properties>
+                <skip.e2e>true</skip.e2e>
+            </properties>
+        </profile>
+        <profile>
+            <!--
+             Profile is aimed to run the build on travis
+            due to travis limitations for output (max 4MB) this profile is used to run e2e tests only.
+
+            the only thing which is happen using throfile is run of e2e tests (without additional build or so)
+            everything is already build using travis_e2e_skip profile
+
+            the whole build is run as test -Ptravis_e2e
+            -->
+            <id>travis_e2e</id>
+            <properties>
+                <skip.e2e>false</skip.e2e>
+                <skip.tests>true</skip.tests>
+            </properties>
+        </profile>
     </profiles>
 
     <reporting>
@@ -1963,5 +1995,6 @@
         <xerces.version>2.11.0</xerces.version>
         <xmlunit.version>1.6</xmlunit.version>
         <yasson.version>1.0.1</yasson.version>
+        <skip.e2e>false</skip.e2e>
     </properties>
 </project>

tests/e2e-client/pom.xml

@@ -41,6 +41,7 @@
                     <forkCount>1</forkCount>
                     <reuseForks>false</reuseForks>
                     <enableAssertions>false</enableAssertions>
+                    <skipTests>${skip.e2e}</skipTests>
                 </configuration>
             </plugin>
         </plugins>

tests/e2e-entity/pom.xml

@@ -41,6 +41,7 @@
                     <forkCount>1</forkCount>
                     <reuseForks>false</reuseForks>
                     <enableAssertions>false</enableAssertions>
+                    <skipTests>${skip.e2e}</skipTests>
                 </configuration>
             </plugin>
         </plugins>

tests/e2e-entity/src/test/java/org/glassfish/jersey/tests/e2e/entity/filtering/EntityFilteringClientTest.java

@@ -77,7 +77,12 @@ public String post(final String value) {
 
     @Test
     public void testEntityAnnotationsPrimaryView() throws Exception {
-        final String fields = target()
+        final ClientConfig config = new ClientConfig()
+                .property(EntityFilteringFeature.ENTITY_FILTERING_SCOPE, PrimaryDetailedView.Factory.get());
+        configureClient(config);
+
+        final String fields = ClientBuilder.newClient(config)
+                .target(getBaseUri())
                 .request()
                 .post(Entity.entity(
                                 new OneFilteringOnClassEntity(),

tests/e2e-server/pom.xml

@@ -41,6 +41,7 @@
                     <forkCount>1</forkCount>
                     <reuseForks>false</reuseForks>
                     <enableAssertions>false</enableAssertions>
+                    <skipTests>${skip.e2e}</skipTests>
                 </configuration>
             </plugin>
         </plugins>

tests/e2e-testng/pom.xml

@@ -45,6 +45,7 @@
                     <forkCount>1</forkCount>
                     <reuseForks>false</reuseForks>
                     <enableAssertions>false</enableAssertions>
+                    <skipTests>${skip.e2e}</skipTests>
                 </configuration>
                 <dependencies>
                     <dependency>

tests/e2e/pom.xml

@@ -41,6 +41,7 @@
                     <forkCount>1</forkCount>
                     <reuseForks>false</reuseForks>
                     <enableAssertions>false</enableAssertions>
+                    <skipTests>${skip.e2e}</skipTests>
                 </configuration>
             </plugin>
         </plugins>