You've already forked comprehensive-rust
mirror of
https://github.com/google/comprehensive-rust.git
synced 2025-12-21 14:35:47 +02:00
GitHub actions separate web tests (#2963)
Separating the web-tests from building the tests makes it more easy to spot and override failing web-tests. This is still testing only the English translation. There are legitimate warnings coming out of the web-test infrastructure and should be blocking. But sometimes the tests fail and need to be overridden.
This commit is contained in:
16
.github/workflows/build.yml
vendored
16
.github/workflows/build.yml
vendored
@@ -149,20 +149,28 @@ jobs:
|
|||||||
MDBOOK_BOOK__LANGUAGE: ${{ matrix.language }}
|
MDBOOK_BOOK__LANGUAGE: ${{ matrix.language }}
|
||||||
run: mdbook test
|
run: mdbook test
|
||||||
|
|
||||||
|
web-tests:
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
needs: build
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v5
|
||||||
- name: Setup Node
|
- name: Setup Node
|
||||||
if: matrix.language == 'en'
|
|
||||||
uses: actions/setup-node@v5
|
uses: actions/setup-node@v5
|
||||||
with:
|
with:
|
||||||
node-version: 22
|
node-version: 22
|
||||||
cache: "npm"
|
cache: "npm"
|
||||||
cache-dependency-path: "tests/package-lock.json"
|
cache-dependency-path: "tests/package-lock.json"
|
||||||
- name: Install test framework
|
- name: Install test framework
|
||||||
if: matrix.language == 'en'
|
|
||||||
run: npm install
|
run: npm install
|
||||||
working-directory: ./tests
|
working-directory: ./tests
|
||||||
|
- name: Download english book
|
||||||
|
uses: actions/download-artifact@v6
|
||||||
|
with:
|
||||||
|
name: comprehensive-rust-en
|
||||||
|
path: book/
|
||||||
- name: Test Javascript
|
- name: Test Javascript
|
||||||
if: matrix.language == 'en'
|
run: cargo xtask web-tests --dir book/comprehensive-rust-en/html
|
||||||
run: cargo xtask web-tests --dir book/comprehensive-rust-${{ matrix.language }}/html
|
|
||||||
|
|
||||||
po-diff:
|
po-diff:
|
||||||
name: Translation diff
|
name: Translation diff
|
||||||
|
|||||||
Reference in New Issue
Block a user