Cache openssl for Windows action (#11)

* add openssl cache
* Fail fast for win
* fix timeouts in test

Author: fafhrd91

.github/workflows/linux.yml

@@ -55,7 +55,7 @@ jobs:
           path: ~/.cargo/bin
           key: ${{ matrix.version }}-x86_64-unknown-linux-gnu-tarpaulin
 
-      - name: tests
+      - name: Run tests
         uses: actions-rs/cargo@v1
         timeout-minutes: 40
         with:
@@ -68,7 +68,7 @@ jobs:
         run: |
           cargo install cargo-tarpaulin
 
-      - name: Generate coverage file
+      - name: Generate coverage report
         if: matrix.version == '1.42.0' && (github.ref == 'refs/heads/master' || github.event_name == 'pull_request')
         continue-on-error: true
         run: |

.github/workflows/osx.yml

@@ -47,13 +47,7 @@ jobs:
           path: target
           key: ${{ matrix.version }}-x86_64-apple-darwin-cargo-build-trimmed-${{ hashFiles('**/Cargo.lock') }}
 
-      - name: check build
-        uses: actions-rs/cargo@v1
-        with:
-          command: check
-          args: --all --all-features --bins --examples --tests
-
-      - name: tests
+      - name: Run tests
         uses: actions-rs/cargo@v1
         with:
           command: test

.github/workflows/windows.yml

@@ -27,6 +27,12 @@ jobs:
           profile: minimal
           override: true
 
+      - name: Cache vcpkg
+        uses: actions/cache@v1
+        with:
+          path: C:\vcpkg\installed\x64-windows\
+          key: x86_64-pc-windows-msvc-openssl
+
       - name: Install OpenSSL
         run: |
           vcpkg integrate install
@@ -36,8 +42,8 @@ jobs:
           Get-ChildItem C:\vcpkg\installed\x64-windows\bin
           Get-ChildItem C:\vcpkg\installed\x64-windows\lib
 
-      - name: tests
+      - name: Run tests
         uses: actions-rs/cargo@v1
         with:
           command: test
-          args: --all --all-features --no-fail-fast -- --nocapture
+          args: --all --all-features -- --nocapture

ntex/tests/http_awc_client.rs

@@ -376,7 +376,7 @@ async fn test_with_query_parameter() {
         }))
     });
 
-    let res = Client::new().get(srv.url("/?qp=5")).send().await.unwrap();
+    let res = srv.get("/?qp=5").send().await.unwrap();
     assert!(res.status().is_success());
 }
 
@@ -392,7 +392,9 @@ async fn test_no_decompress() {
             })))
     });
 
-    let mut res = Client::new()
+    let client = Client::build().timeout(Duration::from_secs(30)).finish();
+
+    let mut res = client
         .get(srv.url("/"))
         .no_decompress()
         .send()
@@ -409,7 +411,7 @@ async fn test_no_decompress() {
     assert_eq!(Bytes::from(dec), Bytes::from_static(STR.as_ref()));
 
     // POST
-    let mut res = Client::new()
+    let mut res = client
         .post(srv.url("/"))
         .no_decompress()
         .send()

ntex/tests/http_rustls.rs

@@ -151,14 +151,14 @@ async fn test_h2_content_length() {
         for i in 0..4 {
             let req = srv
                 .srequest(Method::GET, &format!("/{}", i))
-                .timeout(Duration::from_secs(10))
+                .timeout(Duration::from_secs(30))
                 .send();
             let response = req.await.unwrap();
             assert_eq!(response.headers().get(&header), None);
 
             let req = srv
                 .srequest(Method::HEAD, &format!("/{}", i))
-                .timeout(Duration::from_secs(10))
+                .timeout(Duration::from_secs(30))
                 .send();
             let response = req.await.unwrap();
             assert_eq!(response.headers().get(&header), None);
@@ -167,7 +167,7 @@ async fn test_h2_content_length() {
         for i in 4..6 {
             let req = srv
                 .srequest(Method::GET, &format!("/{}", i))
-                .timeout(Duration::from_secs(10))
+                .timeout(Duration::from_secs(30))
                 .send();
             let response = req.await.unwrap();
             assert_eq!(response.headers().get(&header), Some(&value));