fix: #3622247 Fix the RightUp site template install and add functional CI

Issue: https://www.drupal.org/project/rightup/issues/3622247

What changed

  1. Removed 21 broken Canvas component config files (config/canvas.component.sdc.vartheme_bs5_rightup.*.yml with active_version: fallback and an empty last_active_version) and their 21 matching disable: [] actions in recipe.yml. Both halves have to go together — deleting only the files fails the install with Entity ... does not exist.
  2. RightUp now ships its own media: 18 media entities, 17 file entities and 17 binaries under content/, keeping the existing UUIDs so the Canvas pages resolve unchanged. The recipe is self-contained — no varbase_media_assets or varbase_demo_content dependency.
  3. canvas_page_template_component added to install:, so Canvas does not install it during RecipeAppliedEvent and rebuild the container mid-dispatch.
  4. CI replaced with the Drupal CMS pipeline — install job 🧩 Drupal CMS - RightUp, plus the functional and functional (smoke) jobs running the varbase-e2e suite under tests/. The content guard checks Canvas pages only, since RightUp ships no demo nodes by design (#3621686).

Also renames Rightup to RightUp in the three scripts/ files.

Verification

Run locally today with no patches and nothing pinned — Drupal core 11.4.6, PHP 8.4, canvas 1.11.0:

Base Before After
Drupal CMS exit 1, 379 warnings, 1 error exit 0, 0 warnings, 0 errors
Varbase project — exit 0, 0 warnings, 0 errors

On both, after install: default theme vartheme_bs5_rightup, 18 media, 5 Canvas pages, and /, /features, /blog, /about-varbase, /contact-us, /user/login all HTTP 200. The Drupal CMS browser installer path was also driven through Playwright and produces the same working site.

AI-Generated: Yes

Checkpoints:

  • File an issue
  • Addition/Change/Update/Fix
  • Testing to ensure no regression
  • Automated unit testing coverage
  • Automated functional testing coverage
  • UX/UI designer responsibilities
  • Readability
  • Accessibility
  • Performance
  • Security
  • Developer Documentation
  • User Guide Documentation
  • Reviewed by human
  • Code review by maintainers
  • Full testing and approval
  • Credit contributors
  • Review with the product owner
  • Release notes snippet
  • Release
Edited by Rajab Natshah

Merge request reports

Loading
Loading