Skip to content

Commit 61228e2

Browse files
RomanNikitenkoRomanNikitenko
committed
[fork] Simplify Testing
1 parent 585f6ce commit 61228e2

7 files changed

Lines changed: 16 additions & 194 deletions

File tree

‎.github/workflows/pull-request-check-publish.yml‎

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -71,13 +71,20 @@ jobs:
7171
run: |
7272
docker load -i che-code-amd64.tgz
7373
74+
- name: Login to Quay.io
75+
uses: docker/login-action@v3
76+
with:
77+
registry: quay.io
78+
username: ${{ secrets.QUAY_USERNAME }}
79+
password: ${{ secrets.QUAY_PASSWORD }}
80+
7481
- name: Push che-code-amd64 docker image
7582
run: |
76-
export IMAGE=quay.io/che-incubator-pull-requests/che-code:pr-${{env._PR_NUMBER}}-amd64
83+
export IMAGE=quay.io/${{ secrets.QUAY_USERNAME }}/che-code:pr-${{env._PR_NUMBER}}-amd64
7784
docker tag che-code-amd64 ${IMAGE}
7885
docker push ${IMAGE}
7986
echo "_CHE_CODE_AMD64_IMAGE=${IMAGE}" >> $GITHUB_ENV
80-
87+
8188
- name: Remove che-code-amd64 and prune
8289
run: |
8390
docker image rm che-code-amd64 || true
@@ -93,7 +100,7 @@ jobs:
93100
docker tag che-code-arm64 ${IMAGE}
94101
docker push ${IMAGE}
95102
echo "_CHE_CODE_ARM64_IMAGE=${IMAGE}" >> $GITHUB_ENV
96-
103+
97104
- name: Remove che-code-arm64 and prune
98105
run: |
99106
docker image rm che-code-arm64 || true
@@ -109,7 +116,7 @@ jobs:
109116
docker tag che-dev ${IMAGE}
110117
docker push ${IMAGE}
111118
echo "_CHE_DEV_IMAGE=${IMAGE}" >> $GITHUB_ENV
112-
119+
113120
- name: Remove che-dev and prune
114121
run: |
115122
docker image rm che-dev || true
@@ -124,5 +131,5 @@ jobs:
124131
issue_number: process.env._PR_NUMBER,
125132
owner: context.repo.owner,
126133
repo: context.repo.repo,
127-
body: `Pull Request images published ✨\n\nEditor amd64: [${process.env._CHE_CODE_AMD64_IMAGE}](https://${process.env._CHE_CODE_AMD64_IMAGE})\nEditor arm64: [${process.env._CHE_CODE_ARM64_IMAGE}](https://${process.env._CHE_CODE_ARM64_IMAGE})\nDev image: [${process.env._CHE_DEV_IMAGE}](https://${process.env._CHE_DEV_IMAGE})`
134+
body: `Pull Request images published ✨\n\nEditor amd64: [${process.env._CHE_CODE_AMD64_IMAGE}](https://${process.env._CHE_CODE_AMD64_IMAGE})`
128135
})

‎.github/workflows/pull-request-check.yml‎

Lines changed: 2 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ jobs:
4242
fail-fast: false
4343
matrix:
4444
dist: ['libc-ubi8', 'libc-ubi9', 'musl']
45-
runners: ['ubuntu-22.04', 'ubuntu-22.04-arm']
45+
runners: ['ubuntu-22.04']
4646
runs-on: ${{matrix.runners}}
4747
steps:
4848
- name: Set arch environment variable
@@ -79,7 +79,7 @@ jobs:
7979
strategy:
8080
fail-fast: false
8181
matrix:
82-
runners: ['ubuntu-22.04', 'ubuntu-22.04-arm']
82+
runners: ['ubuntu-22.04']
8383
runs-on: ${{matrix.runners}}
8484
steps:
8585
- name: Set arch environment variable
@@ -138,36 +138,3 @@ jobs:
138138
with:
139139
name: pull-request-number
140140
path: PR_NUMBER
141-
142-
dev:
143-
name: build-che-dev-image
144-
runs-on: ubuntu-22.04
145-
steps:
146-
- name: Checkout che-code source code
147-
uses: actions/checkout@v4
148-
149-
- name: Cleanup docker images
150-
run: |
151-
docker system prune -af
152-
153-
- name: Build Che-Code Docker image
154-
run: |
155-
docker buildx build \
156-
--platform linux/amd64 \
157-
--progress=plain \
158-
-f build/dockerfiles/dev.Dockerfile \
159-
-t che-dev .
160-
161-
- name: Display docker images
162-
run: |
163-
docker images
164-
165-
- name: Compress che-dev image to a file
166-
run: |
167-
docker save che-dev | gzip > che-dev.tgz
168-
169-
- name: Upload che-dev docker image artifact
170-
uses: actions/upload-artifact@v4
171-
with:
172-
name: che-dev
173-
path: che-dev.tgz

‎build/dockerfiles/linux-libc-ubi8.Dockerfile‎

Lines changed: 0 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -91,58 +91,6 @@ RUN chmod a+x /checode/out/server-main.js \
9191
### Beginning of tests
9292
# Do not change line above! It is used to cut this section to skip tests
9393

94-
# Compile tests
95-
RUN ./node_modules/.bin/gulp compile-extension:vscode-api-tests \
96-
compile-extension:markdown-language-features \
97-
compile-extension:typescript-language-features \
98-
compile-extension:emmet \
99-
compile-extension:git \
100-
compile-extension:ipynb \
101-
compile-extension-media \
102-
compile-extension:configuration-editing
103-
104-
# Compile test suites
105-
# https://github.com/microsoft/vscode/blob/cdde5bedbf3ed88f93b5090bb3ed9ef2deb7a1b4/test/integration/browser/README.md#compile
106-
RUN if [ "$(uname -m)" = "x86_64" ]; then npm --prefix test/smoke run compile && npm --prefix test/integration/browser run compile; fi
107-
108-
# install test dependencies
109-
ENV PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=0
110-
RUN if [ "$(uname -m)" = "x86_64" ]; then npm run playwright-install; fi
111-
# Install procps to manage to kill processes and centos stream repository
112-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
113-
ARCH=$(uname -m) && \
114-
yum install --nobest -y procps \
115-
https://vault.centos.org/centos/8/extras/${ARCH}/os/Packages/epel-release-8-11.el8.noarch.rpm \
116-
https://vault.centos.org/8-stream/BaseOS/${ARCH}/os/Packages/centos-gpg-keys-8-3.el8.noarch.rpm \
117-
https://vault.centos.org/8-stream/BaseOS/${ARCH}/os/Packages/centos-stream-repos-8-3.el8.noarch.rpm; \
118-
fi
119-
120-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
121-
sed -i 's|#baseurl=http://mirror.centos.org|baseurl=http://vault.centos.org|g' /etc/yum.repos.d/CentOS-* \
122-
&& yum install -y chromium \
123-
&& PLAYWRIGHT_CHROMIUM_PATH=$(echo /opt/app-root/src/.cache/ms-playwright/chromium-*/) \
124-
&& rm "${PLAYWRIGHT_CHROMIUM_PATH}/chrome-linux/chrome" \
125-
&& ln -s /usr/bin/chromium-browser "${PLAYWRIGHT_CHROMIUM_PATH}/chrome-linux/chrome"; \
126-
fi
127-
128-
# use of retry and timeout
129-
COPY /build/scripts/helper/retry.sh /opt/app-root/src/retry.sh
130-
RUN chmod u+x /opt/app-root/src/retry.sh
131-
132-
# Run integration tests (Browser)
133-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
134-
NODE_ARCH=$(echo "console.log(process.arch)" | node) \
135-
VSCODE_REMOTE_SERVER_PATH="$(pwd)/../vscode-reh-web-linux-${NODE_ARCH}" \
136-
/opt/app-root/src/retry.sh -v -t 3 -s 2 -- timeout -v 5m ./scripts/test-web-integration.sh --browser chromium; \
137-
fi
138-
139-
# Run smoke tests (Browser)
140-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
141-
NODE_ARCH=$(echo "console.log(process.arch)" | node) \
142-
VSCODE_REMOTE_SERVER_PATH="$(pwd)/../vscode-reh-web-linux-${NODE_ARCH}" \
143-
/opt/app-root/src/retry.sh -v -t 3 -s 2 -- timeout -v 5m npm run smoketest-no-compile -- --web --headless --electronArgs="--disable-dev-shm-usage --use-gl=swiftshader"; \
144-
fi
145-
14694
# Do not change line below! It is used to cut this section to skip tests
14795
### Ending of tests
14896

‎build/dockerfiles/linux-libc-ubi9.Dockerfile‎

Lines changed: 0 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -91,57 +91,6 @@ RUN chmod a+x /checode/out/server-main.js \
9191
### Beginning of tests
9292
# Do not change line above! It is used to cut this section to skip tests
9393

94-
# Compile tests
95-
RUN ./node_modules/.bin/gulp compile-extension:vscode-api-tests \
96-
compile-extension:markdown-language-features \
97-
compile-extension:typescript-language-features \
98-
compile-extension:emmet \
99-
compile-extension:git \
100-
compile-extension:ipynb \
101-
compile-extension-media \
102-
compile-extension:configuration-editing
103-
104-
# # Compile test suites
105-
# https://github.com/microsoft/vscode/blob/cdde5bedbf3ed88f93b5090bb3ed9ef2deb7a1b4/test/integration/browser/README.md#compile
106-
RUN if [ "$(uname -m)" = "x86_64" ]; then npm --prefix test/smoke run compile && npm --prefix test/integration/browser run compile; fi
107-
108-
# install test dependencies
109-
ENV PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=0
110-
RUN if [ "$(uname -m)" = "x86_64" ]; then npm run playwright-install; fi
111-
# Install procps to manage to kill processes and centos stream repository
112-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
113-
ARCH=$(uname -m) && \
114-
yum install --nobest -y procps \
115-
https://rpmfind.net/linux/epel/9/Everything/x86_64/Packages/e/epel-release-9-10.el9.noarch.rpm \
116-
https://rpmfind.net/linux/centos-stream/9-stream/BaseOS/x86_64/os/Packages/centos-gpg-keys-9.0-32.el9.noarch.rpm \
117-
https://rpmfind.net/linux/centos-stream/9-stream/BaseOS/x86_64/os/Packages/centos-stream-repos-9.0-32.el9.noarch.rpm; \
118-
fi
119-
120-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
121-
yum install -y chromium && \
122-
PLAYWRIGHT_CHROMIUM_PATH=$(echo /opt/app-root/src/.cache/ms-playwright/chromium-*/) && \
123-
rm "${PLAYWRIGHT_CHROMIUM_PATH}/chrome-linux/chrome" && \
124-
ln -s /usr/bin/chromium-browser "${PLAYWRIGHT_CHROMIUM_PATH}/chrome-linux/chrome"; \
125-
fi
126-
127-
# use of retry and timeout
128-
COPY /build/scripts/helper/retry.sh /opt/app-root/src/retry.sh
129-
RUN chmod u+x /opt/app-root/src/retry.sh
130-
131-
# Run integration tests (Browser)
132-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
133-
NODE_ARCH=$(echo "console.log(process.arch)" | node) \
134-
VSCODE_REMOTE_SERVER_PATH="$(pwd)/../vscode-reh-web-linux-${NODE_ARCH}" \
135-
/opt/app-root/src/retry.sh -v -t 3 -s 2 -- timeout -v 5m ./scripts/test-web-integration.sh --browser chromium; \
136-
fi
137-
138-
# Run smoke tests (Browser)
139-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
140-
NODE_ARCH=$(echo "console.log(process.arch)" | node) \
141-
VSCODE_REMOTE_SERVER_PATH="$(pwd)/../vscode-reh-web-linux-${NODE_ARCH}" \
142-
/opt/app-root/src/retry.sh -v -t 3 -s 2 -- timeout -v 5m npm run smoketest-no-compile -- --web --headless --electronArgs="--disable-dev-shm-usage --use-gl=swiftshader"; \
143-
fi
144-
14594
# Do not change line below! It is used to cut this section to skip tests
14695
### Ending of tests
14796

‎build/dockerfiles/linux-musl.Dockerfile‎

Lines changed: 0 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -64,55 +64,6 @@ RUN cp -r ../vscode-reh-web-linux-alpine /checode
6464
RUN chmod a+x /checode/out/server-main.js \
6565
&& chgrp -R 0 /checode && chmod -R g+rwX /checode
6666

67-
# Compile tests
68-
RUN ./node_modules/.bin/gulp compile-extension:vscode-api-tests \
69-
compile-extension:markdown-language-features \
70-
compile-extension:typescript-language-features \
71-
compile-extension:emmet \
72-
compile-extension:git \
73-
compile-extension:ipynb \
74-
compile-extension-media \
75-
compile-extension:configuration-editing
76-
77-
# Compile test suites
78-
# https://github.com/microsoft/vscode/blob/cdde5bedbf3ed88f93b5090bb3ed9ef2deb7a1b4/test/integration/browser/README.md#compile
79-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
80-
npm --prefix test/smoke run compile && npm --prefix test/integration/browser run compile; \
81-
fi
82-
# use of retry and timeout
83-
COPY /build/scripts/helper/retry.sh /usr/bin/retry
84-
RUN chmod u+x /usr/bin/retry
85-
86-
# install test dependencies
87-
# chromium for tests and procps as tests are using kill commands and it does not work with busybox implementation
88-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
89-
apk add --update --no-cache chromium procps; \
90-
fi
91-
ENV PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD=0
92-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
93-
npm run playwright-install; \
94-
fi
95-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
96-
PLAYWRIGHT_HEADLESS_PATH=$(echo /root/.cache/ms-playwright/chromium_headless_shell-*/chrome-linux) && \
97-
echo "Found headless_shell path: $PLAYWRIGHT_HEADLESS_PATH" && \
98-
rm -f "$PLAYWRIGHT_HEADLESS_PATH/headless_shell" && \
99-
ln -sf /usr/bin/chromium-browser "$PLAYWRIGHT_HEADLESS_PATH/headless_shell" && \
100-
ln -sf /usr/bin/chromium-browser "$PLAYWRIGHT_HEADLESS_PATH/chrome" && \
101-
ls -la "$PLAYWRIGHT_HEADLESS_PATH"; \
102-
fi
103-
104-
# Run integration tests (Browser)
105-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
106-
VSCODE_REMOTE_SERVER_PATH="/vscode-reh-web-linux-alpine" \
107-
retry -v -t 3 -s 2 -- timeout 5m ./scripts/test-web-integration.sh --browser chromium; \
108-
fi
109-
110-
# Run smoke tests (Browser)
111-
RUN if [ "$(uname -m)" = "x86_64" ]; then \
112-
VSCODE_REMOTE_SERVER_PATH="/vscode-reh-web-linux-alpine" \
113-
retry -v -t 3 -s 2 -- timeout 5m npm run smoketest-no-compile -- --web --headless --electronArgs="--disable-dev-shm-usage --use-gl=swiftshader"; \
114-
fi
115-
11667
#########################################################
11768
#
11869
# Copy VS Code launcher to the container

‎code/build/gulpfile.compile.js‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,6 @@ gulp.task(compileBuildWithoutManglingTask);
3030
exports.compileBuildWithoutManglingTask = compileBuildWithoutManglingTask;
3131

3232
// CI compile, including nls and inline sources in sourcemaps, mangling, minification, for build
33-
const compileBuildWithManglingTask = task.define('compile-build-with-mangling', makeCompileBuildTask(false));
33+
const compileBuildWithManglingTask = task.define('compile-build-with-mangling', makeCompileBuildTask(true));
3434
gulp.task(compileBuildWithManglingTask);
3535
exports.compileBuildWithManglingTask = compileBuildWithManglingTask;

‎launcher/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
"format:fix": "prettier --write '{src,tests}/**/*.ts' package.json",
2424
"lint": "eslint --cache=true --no-error-on-unmatched-pattern=true '{src,tests}/(!model|**)/*.ts'",
2525
"lint:fix": "eslint --fix --cache=true --no-error-on-unmatched-pattern=true \"{src,tests}/(!model|**)/*.{ts,tsx}\"",
26-
"test": "jest --forceExit",
26+
"test": "echo 'tests were skipped'",
2727
"watch": "tsc -w"
2828
},
2929
"repository": {

0 commit comments

Comments
 (0)