Skip to content

Commit b51f968

Browse files
committed
Attempt continued
1 parent 7d2266f commit b51f968

File tree

3 files changed

+14
-8
lines changed

3 files changed

+14
-8
lines changed

.github/workflows/build.yml

+11-6
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
name: Build
22

3-
on: [push, pull_request]
3+
on: [push]
44

55
jobs:
66
build:
@@ -52,6 +52,15 @@ jobs:
5252
strategy:
5353
matrix:
5454
node-version: [7.x, 8.x, 9.x, 10.x, 11.x, 12.x, 13.x, 14.x, 15.x, 16.x, 17.x, 18.x, 19.x]
55+
include:
56+
- command: test
57+
- command: test:use-openssl-ca
58+
node-version: 7.x
59+
- command: test:use-openssl-ca
60+
node-version: 8.x
61+
- command: test:use-openssl-ca
62+
node-version: 9.x
63+
5564
steps:
5665
- name: Checkout repo
5766
uses: actions/checkout@v3
@@ -73,14 +82,10 @@ jobs:
7382
- name: Run smoke tests
7483
env:
7584
SERPAPI_TEST_KEY: ${{ secrets.SERPAPI_TEST_KEY }}
76-
NODE_OPTIONS: --use-openssl-ca
7785
run: |
7886
cd smoke_tests/commonjs
79-
if [ "$(node -v)" == "v7.10.1" ]; then
80-
sed -i.orig -e 's/node commonjs.js/node --use-openssl-ca commonjs.js/' package.json
81-
fi
8287
npm i
83-
npm test
88+
npm ${{ matrix.command }}
8489
8590
smoke-tests-esm:
8691
name: "Smoke tests (ESM)"

smoke_tests/commonjs/package.json

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
"serpapi": "../../npm"
55
},
66
"scripts": {
7-
"test": "node commonjs.js"
7+
"test": "node commonjs.js",
8+
"test:use-openssl-ca": "node --use-openssl-ca commonjs.js"
89
}
910
}

smoke_tests/esm/package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,6 @@
55
"serpapi": "../../npm"
66
},
77
"scripts": {
8-
"test": "node esm.js"
8+
"test": "node --use-openssl-ca esm.js"
99
}
1010
}

0 commit comments

Comments
 (0)