Skip to content

Add e2e spec for "sticky posts" feature - #36

Open
goldenapples wants to merge 1 commit into
mainfrom
sticky-posts-e2e-tests
Open

Add e2e spec for "sticky posts" feature#36
goldenapples wants to merge 1 commit into
mainfrom
sticky-posts-e2e-tests

Conversation

@goldenapples

Copy link
Copy Markdown
Contributor

I added a "sticky posts" control in #35, but didn't include any tests.

This PR adds a test spec for this feature, to run as part of the e2e test harness in the future.

$ npx playwright test
Importing database fixture
Logging in to WordPress at http://localhost:8889...
WordPress authentication state saved successfully.

Running 26 tests using 1 worker

...

  ✓  17 …e2e/sticky-posts.spec.js:100:2 › Sticky Posts › pins posts to the front of the loop in the chosen order (4.8s)
  ✓  18 … tests/e2e/sticky-posts.spec.js:138:2 › Sticky Posts › leaves the loop untouched when nothing is pinned (4.8s)
  ✓  19 …ests/e2e/sticky-posts.spec.js:156:2 › Sticky Posts › does not pull in a post the query would not return (5.0s)
  ✓  20 …/sticky-posts.spec.js:193:2 › Sticky Posts › pins, reorders and unpins posts from the inspector control (6.6s)
...

I added a "sticky posts" control in #35, but didn't include any tests.
This PR adds a test spec for this feature, to run in future test
harness.
@github-actions

Copy link
Copy Markdown

Playwright test results

passed  26 passed

Details

stats  26 tests across 8 suites
duration  1 minute, 16 seconds
commit  414f661

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant