-
Notifications
You must be signed in to change notification settings - Fork 26
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
test: Move from CDP to BiDi #700
Conversation
So, this looks quite okay. testUpload() cannot be run multiple times, but that feels like an unrelated test bug. |
I fixed testUpload() in #701, and included it here. |
Good, the rest are flakes and changed pixel refs. |
See cockpit-project/cockpit#20832 Explicitly add a "glob" devDependency to follow suit with cockpit-project/cockpit@680decc155a It was previously used implicitly through a transient dependency of something else, but our esbuild po-plugin uses it explicitly. Adjust `waitDownloadFile()` to the new download directory attribute. This is a `Path` object now, which is more convenient.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Enjoying the pathlib stuff :)
@@ -37,6 +36,7 @@ | |||
"eslint-plugin-react": "7.35.0", | |||
"eslint-plugin-react-hooks": "4.6.2", | |||
"gettext-parser": "8.0.0", | |||
"glob": "11.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh. Could have sworn I had already done that...
Probably in one of my many unmerged PRs ;)
See cockpit-project/cockpit#20832
Explicitly add a "glob" devDependency to follow suit with
cockpit-project/cockpit@680decc155a
It was previously used implicitly through a transient dependency of
something else, but our esbuild po-plugin uses it explicitly.
Adjust
waitDownloadFile()
to the new download directory attribute.This is a
Path
object now, which is more convenient.