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

[Flaky Test] should display revisions UI when there is more than 1 revision #49708

Closed
github-actions bot opened this issue Apr 11, 2023 · 0 comments · Fixed by #50454
Closed

[Flaky Test] should display revisions UI when there is more than 1 revision #49708

github-actions bot opened this issue Apr 11, 2023 · 0 comments · Fixed by #50454
Assignees
Labels
[Type] Flaky Test Auto-generated flaky test report issue

Comments

@github-actions
Copy link

github-actions bot commented Apr 11, 2023

Flaky test detected. This is an auto-generated issue by GitHub Actions. Please do NOT edit this manually.

Test title

should display revisions UI when there is more than 1 revision

Test path

/test/e2e/specs/site-editor/user-global-styles-revisions.spec.js

Errors

[2023-04-11T06:10:23.601Z] Test passed after 1 failed attempt on try/user-global-styles-revisions.
[2023-04-11T06:59:32.085Z] Test passed after 1 failed attempt on try/user-global-styles-revisions.
[2023-04-20T05:46:54.224Z] Test passed after 1 failed attempt on try/user-global-styles-revisions-in-style-book-slot.
[2023-04-27T04:45:19.388Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-04-27T05:15:43.835Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-04-27T06:09:39.460Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-04-28T07:04:47.860Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-05-01T02:24:19.508Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-05-01T05:49:13.350Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-05-02T05:49:50.857Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-05-03T07:48:34.757Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-05-04T01:31:32.358Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-05-06T00:11:31.431Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-05-08T01:08:03.158Z] Test passed after 1 failed attempt on add/global-styles-revisions-ui.
[2023-05-08T03:02:28.831Z] Test passed after 1 failed attempt on trunk.
[2023-05-08T03:12:36.114Z] Test passed after 1 failed attempt on trunk.
[2023-05-08T04:20:47.397Z] Test passed after 1 failed attempt on try/visible-dimension-controls.
[2023-05-08T05:17:39.419Z] Test passed after 1 failed attempt on try/visible-dimension-controls.
[2023-05-08T05:21:51.826Z] Test passed after 1 failed attempt on trunk.
[2023-05-08T06:09:11.322Z] Test passed after 1 failed attempt on trunk.
[2023-05-08T07:29:18.977Z] Test passed after 1 failed attempt on fix/list-view-delete-and-focus.
[2023-05-08T07:42:39.456Z] Test passed after 1 failed attempt on add/elements-variations-custom-css.
[2023-05-08T09:01:51.014Z] Test passed after 1 failed attempt on add/store-open-close-api-commands.
[2023-05-08T09:05:52.812Z] Test passed after 1 failed attempt on remove/quote-transform-from-group.
[2023-05-08T09:16:19.995Z] Test passed after 1 failed attempt on update/file-block-interactivity-api.
[2023-05-08T09:59:08.896Z] Test passed after 1 failed attempt on remove/quote-transform-from-group.
[2023-05-08T10:54:38.136Z] Test passed after 1 failed attempt on use/list-view-in-inspector.
[2023-05-08T11:00:16.449Z] Test passed after 1 failed attempt on remove/offcanvaseditor-in-browse-mode.
[2023-05-08T11:40:20.119Z] Test passed after 1 failed attempt on remove/offcanvaseditor-in-browse-mode.
[2023-05-08T11:47:44.882Z] Test passed after 1 failed attempt on trunk.
[2023-05-08T13:33:54.326Z] Test passed after 1 failed attempt on refactor/block-list-uses-hooks.
[2023-05-08T13:43:22.188Z] Test passed after 1 failed attempt on try/site-edit-loading-experience-3.
[2023-05-08T14:51:16.259Z] Test passed after 1 failed attempt on fix/list-view-color.
[2023-05-08T15:10:58.069Z] Test passed after 1 failed attempt on trunk.
[2023-05-08T16:43:10.351Z] Test passed after 1 failed attempt on docs/use-atag-in-callout.
[2023-05-08T16:52:42.360Z] Test passed after 1 failed attempt on fix/multiple-tooltips-site-editor.
[2023-05-08T17:13:07.513Z] Test passed after 1 failed attempt on trunk.
[2023-05-08T17:22:43.851Z] Test passed after 1 failed attempt on rnmobile/update/setup-media-picker.
[2023-05-08T17:40:25.904Z] Test passed after 1 failed attempt on rnmobile/add/sandbox-test-id.
[2023-05-08T17:53:22.784Z] Test passed after 1 failed attempt on trunk.
[2023-05-08T18:27:45.755Z] Test passed after 1 failed attempt on rnmobile/add-embed-webview.
[2023-05-08T18:47:07.675Z] Test passed after 1 failed attempt on fix/editor-iframe-margin-override.
[2023-05-08T19:51:03.326Z] Test passed after 1 failed attempt on check-that-phpunit-runs.
[2023-05-08T20:32:37.296Z] Test passed after 1 failed attempt on remove/select-parent-block-settings.
[2023-05-08T21:40:21.599Z] Test passed after 1 failed attempt on perf/reduce-inner-blocks-tree-depth.
[2023-05-08T21:52:02.768Z] Test passed after 1 failed attempt on colorpalette-layout-update.
[2023-05-08T22:58:48.390Z] Test passed after 1 failed attempt on perf/reduce-inner-blocks-tree-depth.
[2023-05-08T23:27:15.664Z] Test passed after 1 failed attempt on fix/editor-iframe-margin-override.
[2023-05-09T02:01:02.297Z] Test passed after 1 failed attempt on behaviors-ui.
[2023-05-09T02:39:39.802Z] Test passed after 1 failed attempt on rnmobile/add-embed-webview.
[2023-05-09T04:58:56.732Z] Test passed after 1 failed attempt on fix/reset-of-spacing-sizes-control.
TimeoutError: locator.click: Timeout 10000ms exceeded.
=========================== logs ===========================
waiting for getByRole('button', { name: 'Typography styles' })
============================================================
    at /home/runner/work/gutenberg/gutenberg/test/e2e/specs/site-editor/user-global-styles-revisions.spec.js:60:6
@github-actions github-actions bot added the [Type] Flaky Test Auto-generated flaky test report issue label Apr 11, 2023
github-actions bot referenced this issue May 8, 2023
* initial commit.

* Adding state for revisions
Adding revisions global styles sidebar UI
Adding revisions fill

Adding revisions components and styles.

Added e2e tests

* Making selectors more consistent in e2e tests

* This commit:
- redesigns the revisions list to show a timeline view
- changes the revisions endpoint response (and tests) to return a human friendly diff only. The frontend can format dates on the fly
- Adds a slot to the edit side sidebar that can position a component at the bottom of the interface sidebar
- updates e2e tests

* This commit:
- adds wordpress/date to package-lock.jdon
- ensures that we correctly modify the user revisions to add unsaved and islatest flags

* This commit:
- extends the e2e test a little bit and adds a TODO to do it better
- Adds ally labels to the color panel

* This commit:
- uses a clientside human-readable time diff method instead of the server side one. This is to make the response as close to the post revisions response as possible (for now)
- makes the current timeline dot blue

This commit:

* This commit:
- removes the specific author/user properties from the rest controller
- grabs revision author information in the JS
- shows a loading wheel while we grab revisions and user data

* Label titles

* Style adjustments

Adding a reset to theme default item to the revisions list.

Adding a dependencies to the `useMemo` so that it knows to update when the number of revisions change.

This is so we can ensure we add the right metadata in the right order, e.g., `isLatest`

Extracted hooks and components from screen revisions monster file

* Adding tests for the useGetGlobalStylesRevisions hook

* Extracting SCSS to external file
Renaming date > meta in classname of DIV that wraps revision date and author meta
Extra checks so that we don't render the meta block at all if there's no data

* Reverting the reset theme default button and ensuring that revisions panel only display when there are more than two revisions.
Updated tests accordingly.

bumping gravatar size to 48

Updating other E2E tests after having added aria labels to the color controls

Update e2e button spec after changes to button labels

* This commit:
- refactors useGlobalOutput by splitting out the build logic into a separate hook so that we can pass a custom config
- uses wp.date.getSettings().formats.datetimeAbbreviated for formatted date in the revisions buttons
- removes unnecessary role on the ordered list
- updated copy
- replacing modal component with the confirm dialog component
- minor code optimizations
- making the revisions store methods stable
- updating tests
- removing shouldShowClose button prop on the editor-canvas-container

update e2e

* Tweaked the revisions buttons copy, shifting the date and meta around.
Updated tests

---------

Co-authored-by: James Koster <[email protected]>
github-actions bot referenced this issue May 8, 2023
…viewports, or in template mode (#50361)

* Update background color of post editor when viewing mobile or tablet viewports, or in template mode

* Also update site editor color
github-actions bot referenced this issue May 8, 2023
* Try always showing the dimensions controls

* Update defaults for Paragraph block
github-actions bot referenced this issue May 8, 2023
* refactor: BlockList EmptyList uses Hooks

Adopt modern React paradigms prior to refactoring the components
composition. Completed to more closely mirror other code in the code
base.

* refactor: BlockList uses Hooks

Adopt modern React paradigms prior to refactoring the components
composition. Completed to more closely mirror other code in the code
base.

* refactor: BlockList higher-order components use Hooks

Adopt modern React paradigms prior to refactoring the components
composition. Completed to more closely mirror other code in the code
base.

Additionally, this removes the `withPreferredColorScheme` higher-order
component as it appeared unused.

* fix: BlockList footer interactivity

`VirtualList` expects a React Component (e.g. `SomeComponent`), or a
React element (e.g. `<SomeComponent` />) passed to the
`ListFooterComponent` prop. Passing a function instead resulted in the
block inserter never opening when tapping the block appender button
within the footer; this appeared to be due to excessive re-renders
resetting the `isOpen` state for the inserter.

* fix: Rename Footer addBlockToEndOfPost callback prop

The typo led to an undefined callback function.

* fix: Reinstate missing prop used within store selector

This prop was originally accessed via the `withSelect` higher-order
component. When converting to Hooks, passing this prop was overlooked.

* fix: Scope timer run by `waitForStoreResolvers`

Running all timers could create testing failures in scenarios wheres
where inner blocks were inserted, e.g. Social Icons. It is not known
why, but scoping from "all" to "pending" timers resolved the following
errors, without breaking any existing tests.

Unknown origin:

```
Warning: You seem to have overlapping act() calls, this is not supported. Be sure to await previous act() calls before making a new one.
```

Originating from `waitForStoreResolvers`:

```
Ran 100000 timers, and there are still more! Assuming we've hit an infinite recursion and bailing out...
```

Originating from `packages/block-editor/src/components/block-list/test/index.native.js`:

```
thrown: "Exceeded timeout of 5000 ms for a test.
```

* test: BlockList renders footer appender

* typo: Add code comment

Added to align with sibling tests.

* test: Expand empty BlockList appender tests
github-actions bot referenced this issue May 8, 2023
Removes mention of performance testing in the Gutenberg plugin release process documentation. 

Props to @priethor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Flaky Test Auto-generated flaky test report issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants