Skip to content

Commit ac718ee

Browse files
committed
Run embedded JS API test on both dart and node compiler
1 parent 7ffed6e commit ac718ee

File tree

1 file changed

+5
-9
lines changed

1 file changed

+5
-9
lines changed

Diff for: .github/workflows/test.yml

+5-9
Original file line numberDiff line numberDiff line change
@@ -126,13 +126,14 @@ jobs:
126126
working-directory: sass-spec
127127

128128
sass_spec_js_embedded:
129-
name: 'JS API Tests | Embedded | Node ${{ matrix.node-version }} | ${{ matrix.os }}'
129+
name: "JS API Tests | Embedded ${{ matrix.js && 'Pure JS' || 'Dart' }} | Node ${{ matrix.node-version }} | ${{ matrix.os }}"
130130
runs-on: ${{ matrix.os }}
131131
if: "github.event_name != 'pull_request' || !contains(github.event.pull_request.body, 'skip sass-embedded')"
132132

133133
strategy:
134134
fail-fast: false
135135
matrix:
136+
js: [true, false]
136137
os: [ubuntu-latest, windows-latest, macos-latest]
137138
node-version: ['lts/*']
138139
include:
@@ -166,19 +167,14 @@ jobs:
166167
- name: Initialize embedded host
167168
run: |
168169
npm install
169-
npm run init -- --compiler-path=.. --language-path=../build/language
170+
npm run init -- --compiler-path=.. --language-path=../build/language --compiler-js=${{ matrix.js && 'true' || 'false' }}
170171
npm run compile
171-
mv {`pwd`/,dist/}lib/src/vendor/dart-sass
172172
working-directory: embedded-host-node
173173

174174
- name: Version info
175175
run: |
176-
path=embedded-host-node/dist/lib/src/vendor/dart-sass/sass
177-
if [[ -f "$path.cmd" ]]; then "./$path.cmd" --version
178-
elif [[ -f "$path.bat" ]]; then "./$path.bat" --version
179-
elif [[ -f "$path.exe" ]]; then "./$path.exe" --version
180-
else "./$path" --version
181-
fi
176+
npx sass --version
177+
working-directory: embedded-host-node
182178

183179
- name: Run tests
184180
run: npm run js-api-spec -- --sassPackage ../embedded-host-node --sassSassRepo ../build/language

0 commit comments

Comments
 (0)