Merge pull request #312 from Automattic/fix-playwright

fix(playwright): remove `dbus` from autostart

features/src/playwright/devcontainer-feature.json

@@ -3,7 +3,7 @@
     "name": "Playwright",
     "description": "Sets up Playwright into the Dev Environment",
     "documentationURL": "https://github.com/Automattic/vip-codespaces/tree/trunk/features/src/playwright",
-    "version": "1.1.0",
+    "version": "1.1.1",
     "options": {
         "enabled": {
             "type": "boolean",

features/src/playwright/install.sh

@@ -52,6 +52,8 @@ if [ "${ENABLED}" = "true" ]; then
             playwright-core install-deps
             su -s /bin/sh -c 'playwright-core install' "${_REMOTE_USER}"
 
+            update-rc.d -f dbus remove
+
             apt-get clean
             rm -rf /var/lib/apt/lists/*
         ;;

features/test/playwright/checks.sh

@@ -5,3 +5,9 @@ check "node exists and is executable" which node
 check "npm exists and is executable" which npm
 
 check "Smoke test" playwright-core --version
+
+# Microsoft's base images contain zsh. We don't want to run this check for MS images because we have no control over the installed services.
+if test -d /etc/rc2.d && ! test -e /usr/bin/zsh; then
+    dir="$(ls -1 /etc/rc2.d)"
+    check "/etc/rc2.d is empty" test -z "${dir}"
+fi