Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get subdomain multisite tests working properly #148

Merged
merged 22 commits into from
Aug 2, 2024
Merged
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
ff53ea9
add copying the changes in the PR to the subdomain tests
jazzsequence Jul 31, 2024
75e933f
switch to git mode before we try pushing things
jazzsequence Jul 31, 2024
20daa86
remove color tags
jazzsequence Jul 31, 2024
b425a1d
exclude node_modules from rsync
jazzsequence Jul 31, 2024
3f0e604
add multisite config to the fixture site
jazzsequence Jul 31, 2024
7405de3
fix quoting
jazzsequence Jul 31, 2024
90296bb
remove lingering ${type} vars and stray quote
jazzsequence Jul 31, 2024
1619e06
don't run playwright tests if files we don't care about were affected
jazzsequence Jul 31, 2024
3312f0d
remove color tags
jazzsequence Jul 31, 2024
fe79af5
don't save in a variable
jazzsequence Jul 31, 2024
f8ae367
exclude node_modules from rsync
jazzsequence Jul 31, 2024
47e8d92
fix quoting
jazzsequence Jul 31, 2024
3f4d299
remove lingering ${type} vars and stray quote
jazzsequence Jul 31, 2024
dffdcfb
don't run playwright tests if files we don't care about were affected
jazzsequence Jul 31, 2024
2899f36
move the wait to earlier
jazzsequence Jul 31, 2024
60c3da8
give up on the multisite checking
jazzsequence Jul 31, 2024
fc08498
install graphql as part of the workflow
jazzsequence Jul 31, 2024
38c9386
clear ache after installing graphql
jazzsequence Jul 31, 2024
17d4860
break the subdomain sites up
jazzsequence Jul 31, 2024
46aaa50
allow the graphql endpoint to be passed
jazzsequence Jul 31, 2024
22c3022
copy the application.php for subdom
jazzsequence Jul 31, 2024
4e9d827
add more debugging info to failure output
jazzsequence Aug 1, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
don't save in a variable
Co-authored-by: Phil Tyler <philip@tylerdigital.com>
  • Loading branch information
jazzsequence and pwtyler committed Jul 31, 2024
commit fe79af5a2cb3ce46c10ea1a4997342b32e71e7a4
4 changes: 1 addition & 3 deletions .github/workflows/playwright.yml
Original file line number Diff line number Diff line change
@@ -439,9 +439,7 @@ jobs:
exit 1
fi
# Check SUBDOMAIN_INSTALL value
SUBDOMAIN_INSTALL=$(terminus wp "${{ env.SITE_ID }}".dev -- config get SUBDOMAIN_INSTALL)
if [[ "${SUBDOMAIN_INSTALL}" != "1" ]]; then
# SUBDOMAIN_INSTALL should be true.
if ! terminus wp "${{ env.SITE_ID }}".dev -- config is-true SUBDOMAIN_INSTALL; then
echo -e "Subdomain configuration not found!"
exit 1
fi