Skip to content

Commit

Permalink
Drop Playwright support (#1484)
Browse files Browse the repository at this point in the history
## Problem

- For the integration tests we have switched to Puppeteer
- So let's drop all Playwright code and references
  • Loading branch information
lslezak authored Jul 18, 2024
2 parents 9be2884 + 397be24 commit 1b1044a
Show file tree
Hide file tree
Showing 19 changed files with 7 additions and 1,147 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/obs-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ on:
- v[0-9]*

jobs:
# Note: cockpit-agama-playwright is currently not submitted
# Note: agama-integration-tests and the Live ISO are currently not submitted

update_rust:
uses: ./.github/workflows/obs-staging-shared.yml
Expand Down
22 changes: 0 additions & 22 deletions .github/workflows/obs-staging-playwright.yml

This file was deleted.

3 changes: 0 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,6 @@ interfaces over it.
| ------------------------------------------------------ | --------------------------------------------------------------- |
| ![Installing](./doc/images/screenshots/installing.png) | ![Installation finished](./doc/images/screenshots/finished.png) |

_Note for developers: For updating the screenshots see the
[integration test documentation](playwright/README.md#updating-the-screenshots)._

</details>

## Why a New Installer
Expand Down
9 changes: 0 additions & 9 deletions Rakefile
Original file line number Diff line number Diff line change
Expand Up @@ -177,15 +177,6 @@ if ENV["YUPDATE_FORCE"] == "1" || File.exist?("/.packages.initrd") || live_iso?
end
end

# update also the tests if they are present in the system
if ENV["YUPDATE_SKIP_TESTS"] != "1" && File.exist?("/usr/share/agama-playwright")
puts "Installing the integration tests..."

# we are installing into an empty chroot, make sure the target exists
FileUtils.mkdir_p(File.join(destdir, "/usr/share"))
FileUtils.cp_r("playwright/.", File.join(destdir, "/usr/share/agama-playwright"))
end

if ENV["YUPDATE_SKIP_PRODUCTS"] != "1"
files = Dir.glob("products.d/*.y{a}ml")
files.each do |f|
Expand Down
4 changes: 0 additions & 4 deletions doc/yupdate.md
Original file line number Diff line number Diff line change
Expand Up @@ -59,10 +59,6 @@ You can modify the update process with these environment variables:
- `YUPDATE_SKIP_BACKEND=1` - Skip updating the D-Bus service backend. This is
similar to the previous option, use it when you do want to keep the D-Bus
service unchanged.
- `YUPDATE_SKIP_TESTS=1` - Skip updating the integration tests if they are
installed. If the tests are not installed they are not added automatically.
If you want to add them first create the `/usr/share/agama-playwright`
directory where the tests will be added and then run the `yupdate` command.

## Notes

Expand Down
6 changes: 6 additions & 0 deletions live/src/agama-installer-openSUSE.changes
Original file line number Diff line number Diff line change
@@ -1,3 +1,9 @@
-------------------------------------------------------------------
Thu Jul 18 15:25:39 UTC 2024 - Ladislav Slezák <[email protected]>

- Include Puppeteer in all ISO images (gh#openSUSE/agama#1477)
- Drop Playwright ISO flavor (gh#openSUSE/agama#1481)

-------------------------------------------------------------------
Tue Jul 9 13:26:38 UTC 2024 - Knut Anderssen <[email protected]>

Expand Down
6 changes: 0 additions & 6 deletions playwright/.gitignore

This file was deleted.

339 changes: 0 additions & 339 deletions playwright/LICENSE

This file was deleted.

233 changes: 0 additions & 233 deletions playwright/README.md

This file was deleted.

Loading

0 comments on commit 1b1044a

Please sign in to comment.