-
Notifications
You must be signed in to change notification settings - Fork 5
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
de80634
commit 3ff3e59
Showing
3 changed files
with
98 additions
and
151 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,73 +12,54 @@ jobs: | |
|| (github.event.pull_request.head.repo.id != github.event.pull_request.base.repo.id) | ||
name: with Plugins | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- | ||
name: Install libreoffice | ||
run: | | ||
sudo add-apt-repository -y ppa:libreoffice/ppa | ||
sudo apt update | ||
sudo apt install -y --no-install-recommends libreoffice libreoffice-pdfimport | ||
- | ||
name: Install etherpad core | ||
uses: actions/checkout@v4 | ||
with: | ||
repository: ether/etherpad-lite | ||
- | ||
name: Checkout plugin repository | ||
uses: actions/checkout@v4 | ||
with: | ||
path: ./node_modules/__tmp | ||
- | ||
name: Determine plugin name | ||
id: plugin_name | ||
run: | | ||
cd ./node_modules/__tmp | ||
npx -c 'printf %s\\n "::set-output name=plugin_name::${npm_package_name}"' | ||
- | ||
name: Rename plugin directory | ||
run: | | ||
mv ./node_modules/__tmp ./node_modules/"${PLUGIN_NAME}" | ||
env: | ||
PLUGIN_NAME: ${{ steps.plugin_name.outputs.plugin_name }} | ||
- | ||
uses: actions/setup-node@v4 | ||
- uses: pnpm/action-setup@v3 | ||
name: Install pnpm | ||
with: | ||
version: 8 | ||
run_install: false | ||
- name: Get pnpm store directory | ||
shell: bash | ||
run: | | ||
echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV | ||
- uses: actions/cache@v4 | ||
name: Setup pnpm cache | ||
with: | ||
path: ${{ env.STORE_PATH }} | ||
key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }} | ||
restore-keys: | | ||
${{ runner.os }}-pnpm-store- | ||
- | ||
name: Install plugin dependencies | ||
run: | | ||
cd ./node_modules/"${PLUGIN_NAME}" | ||
pnpm i | ||
env: | ||
PLUGIN_NAME: ${{ steps.plugin_name.outputs.plugin_name }} | ||
# Etherpad core dependencies must be installed after installing the | ||
# plugin's dependencies, otherwise npm will try to hoist common | ||
# dependencies by removing them from src/node_modules and installing them | ||
# in the top-level node_modules. As of v6.14.10, npm's hoist logic appears | ||
# to be buggy, because it sometimes removes dependencies from | ||
# src/node_modules but fails to add them to the top-level node_modules. | ||
# Even if npm correctly hoists the dependencies, the hoisting seems to | ||
# confuse tools such as `npm outdated`, `npm update`, and some ESLint | ||
# rules. | ||
- | ||
name: Install Etherpad core dependencies | ||
run: ./bin/installDeps.sh | ||
- | ||
name: Run the backend tests | ||
run: cd src && pnpm run test | ||
- | ||
name: Install libreoffice | ||
uses: awalsh128/[email protected] | ||
with: | ||
packages: libreoffice libreoffice-pdfimport | ||
version: 1.0 | ||
- | ||
name: Install etherpad core | ||
uses: actions/checkout@v3 | ||
with: | ||
repository: ether/etherpad-lite | ||
- | ||
name: Checkout plugin repository | ||
uses: actions/checkout@v3 | ||
with: | ||
path: ./node_modules/__tmp | ||
- | ||
name: Determine plugin name | ||
id: plugin_name | ||
run: | | ||
cd ./node_modules/__tmp | ||
npx -c 'printf %s\\n "::set-output name=plugin_name::${npm_package_name}"' | ||
- | ||
name: Rename plugin directory | ||
run: | | ||
mv ./node_modules/__tmp ./node_modules/"${PLUGIN_NAME}" | ||
env: | ||
PLUGIN_NAME: ${{ steps.plugin_name.outputs.plugin_name }} | ||
- uses: pnpm/action-setup@v3 | ||
name: Install pnpm | ||
with: | ||
version: 8 | ||
run_install: false | ||
- name: Get pnpm store directory | ||
shell: bash | ||
run: | | ||
echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV | ||
- uses: actions/cache@v4 | ||
name: Setup pnpm cache | ||
with: | ||
path: ${{ env.STORE_PATH }} | ||
key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }} | ||
restore-keys: | | ||
${{ runner.os }}-pnpm-store- | ||
- | ||
name: Install Etherpad core dependencies | ||
run: bin/installDeps.sh | ||
- | ||
name: Run the backend tests | ||
run: pnpm run test |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,33 +1,24 @@ | ||
# Publicly credit Sauce Labs because they generously support open source | ||
# projects. | ||
name: "frontend tests powered by Sauce Labs" | ||
name: "Frontend Tests" | ||
|
||
on: [push] | ||
|
||
jobs: | ||
test: | ||
runs-on: ubuntu-latest | ||
if: ${{ github.actor != 'dependabot[bot]' }} | ||
|
||
steps: | ||
- | ||
name: Generate Sauce Labs strings | ||
id: sauce_strings | ||
run: | | ||
printf %s\\n '::set-output name=name::${{github.event.repository.name}} ${{ github.workflow }} - ${{ github.job }}' | ||
printf %s\\n '::set-output name=tunnel_id::${{ github.run_id }}-${{ github.run_number }}-${{ github.job }}' | ||
- | ||
name: Check out Etherpad core | ||
uses: actions/checkout@v4 | ||
uses: actions/checkout@v3 | ||
with: | ||
repository: ether/etherpad-lite | ||
- | ||
uses: actions/setup-node@v4 | ||
- uses: pnpm/action-setup@v3 | ||
name: Install pnpm | ||
with: | ||
version: 8 | ||
run_install: false | ||
version: 8 | ||
run_install: false | ||
- name: Get pnpm store directory | ||
shell: bash | ||
run: | | ||
|
@@ -41,7 +32,7 @@ jobs: | |
${{ runner.os }}-pnpm-store- | ||
- | ||
name: Check out the plugin | ||
uses: actions/checkout@v4 | ||
uses: actions/checkout@v3 | ||
with: | ||
path: ./node_modules/__tmp | ||
- | ||
|
@@ -80,31 +71,23 @@ jobs: | |
# rules. | ||
- | ||
name: Install Etherpad core dependencies | ||
run: ./bin/installDeps.sh | ||
- | ||
name: Create settings.json | ||
run: cp settings.json.template settings.json | ||
- | ||
name: Disable import/export rate limiting | ||
run: | | ||
sed -e '/^ *"importExportRateLimiting":/,/^ *\}/ s/"max":.*/"max": 0/' -i settings.json | ||
- | ||
name: Remove standard frontend test files | ||
run: rm -rf src/tests/frontend/specs | ||
- | ||
uses: saucelabs/[email protected] | ||
with: | ||
username: ${{ secrets.SAUCE_USERNAME }} | ||
accessKey: ${{ secrets.SAUCE_ACCESS_KEY }} | ||
tunnelIdentifier: ${{ steps.sauce_strings.outputs.tunnel_id }} | ||
- | ||
name: Run the frontend tests | ||
run: bin/installDeps.sh | ||
- name: Create settings.json | ||
run: cp ./src/tests/settings.json settings.json | ||
- name: Run the frontend tests | ||
shell: bash | ||
env: | ||
SAUCE_USERNAME: ${{ secrets.SAUCE_USERNAME }} | ||
SAUCE_ACCESS_KEY: ${{ secrets.SAUCE_ACCESS_KEY }} | ||
SAUCE_NAME: ${{ steps.sauce_strings.outputs.name }} | ||
TRAVIS_JOB_NUMBER: ${{ steps.sauce_strings.outputs.tunnel_id }} | ||
GIT_HASH: ${{ steps.environment.outputs.sha_short }} | ||
run: | | ||
src/tests/frontend/travis/runner.sh | ||
pnpm run dev & | ||
connected=false | ||
can_connect() { | ||
curl -sSfo /dev/null http://localhost:9001/ || return 1 | ||
connected=true | ||
} | ||
now() { date +%s; } | ||
start=$(now) | ||
while [ $(($(now) - $start)) -le 15 ] && ! can_connect; do | ||
sleep 1 | ||
done | ||
cd src | ||
pnpm exec playwright install chromium --with-deps | ||
pnpm run test-ui --project=chromium |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -12,6 +12,7 @@ on: | |
|
||
jobs: | ||
test: | ||
name: "Test release" | ||
runs-on: ubuntu-latest | ||
steps: | ||
# Clone ether/etherpad-lite to ../etherpad-lite so that ep_etherpad-lite | ||
|
@@ -22,7 +23,7 @@ jobs: | |
# conflicts with this plugin's clone, etherpad-lite must be cloned and | ||
# moved out before this plugin's repo is cloned to $GITHUB_WORKSPACE. | ||
- | ||
uses: actions/checkout@v4 | ||
uses: actions/checkout@v3 | ||
with: | ||
repository: ether/etherpad-lite | ||
path: etherpad-lite | ||
|
@@ -31,45 +32,35 @@ jobs: | |
# etherpad-lite has been moved outside of $GITHUB_WORKSPACE, so it is now | ||
# safe to clone this plugin's repo to $GITHUB_WORKSPACE. | ||
- | ||
uses: actions/checkout@v4 | ||
# This is necessary for actions/setup-node because '..' can't be used in | ||
# cache-dependency-path. | ||
- | ||
name: Create ep_etherpad-lite symlink | ||
uses: actions/checkout@v3 | ||
- uses: pnpm/action-setup@v3 | ||
name: Install pnpm | ||
with: | ||
version: 8 | ||
run_install: false | ||
- name: Get pnpm store directory | ||
shell: bash | ||
run: | | ||
mkdir -p node_modules | ||
ln -s ../../etherpad-lite/src node_modules/ep_etherpad-lite | ||
- | ||
uses: actions/setup-node@v4 | ||
echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV | ||
- uses: actions/cache@v4 | ||
name: Setup pnpm cache | ||
with: | ||
node-version: 20 | ||
cache: 'npm' | ||
cache-dependency-path: | | ||
node_modules/ep_etherpad-lite/package-lock.json | ||
node_modules/ep_etherpad-lite/bin/doc/package-lock.json | ||
package-lock.json | ||
- run: npm install [email protected] -g | ||
name: Install legacy npm for correct dependency resolution | ||
path: ${{ env.STORE_PATH }} | ||
key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }} | ||
restore-keys: | | ||
${{ runner.os }}-pnpm-store- | ||
# All of ep_etherpad-lite's devDependencies are installed because the | ||
# plugin might do `require('ep_etherpad-lite/node_modules/${devDep}')`. | ||
# Eventually it would be nice to create an ESLint plugin that prohibits | ||
# Etherpad plugins from piggybacking off of ep_etherpad-lite's | ||
# devDependencies. If we had that, we could change this line to only | ||
# install production dependencies. | ||
- run: pnpm i && pnpm link --global | ||
- | ||
run: cd ../etherpad-lite/src && npm ci | ||
- | ||
run: npm ci | ||
# This runs some sanity checks and creates a symlink at | ||
# node_modules/ep_etherpad-lite that points to ../../etherpad-lite/src. | ||
# This step must be done after `npm ci` installs the plugin's dependencies | ||
# because npm "helpfully" cleans up such symlinks. :( Installing | ||
# ep_etherpad-lite in the plugin's node_modules prevents lint errors and | ||
# unit test failures if the plugin does `require('ep_etherpad-lite/foo')`. | ||
- | ||
run: npm install --no-save ep_etherpad-lite@file:../etherpad-lite/src | ||
run: cd ../etherpad-lite/src && pnpm i && pnpm link $PLUGIN_NAME | ||
|
||
- | ||
run: npm test | ||
run: pnpm run test | ||
- | ||
run: npm run lint | ||
|
||
|
@@ -79,17 +70,9 @@ jobs: | |
runs-on: ubuntu-latest | ||
steps: | ||
- | ||
uses: actions/checkout@v4 | ||
uses: actions/checkout@v3 | ||
with: | ||
fetch-depth: 0 | ||
- | ||
uses: actions/setup-node@v4 | ||
with: | ||
node-version: 20 | ||
registry-url: https://registry.npmjs.org/ | ||
cache: 'npm' | ||
- run: npm install [email protected] -g | ||
name: Install legacy npm for correct dependency resolution | ||
- | ||
name: Bump version (patch) | ||
run: | | ||
|
@@ -98,13 +81,13 @@ jobs: | |
[ "${NEW_COMMITS}" -gt 0 ] || exit 0 | ||
git config user.name 'github-actions[bot]' | ||
git config user.email '41898282+github-actions[bot]@users.noreply.github.com' | ||
npm ci | ||
npm version patch | ||
pnpm i | ||
pnpm version patch | ||
git push --follow-tags | ||
# This is required if the package has a prepare script that uses something | ||
# in dependencies or devDependencies. | ||
- | ||
run: npm ci | ||
run: pnpm i | ||
# `npm publish` must come after `git push` otherwise there is a race | ||
# condition: If two PRs are merged back-to-back then master/main will be | ||
# updated with the commits from the second PR before the first PR's | ||
|