Skip to content

[#9] Add e2e tests for article read #15

Merged
idembele70 merged 2 commits into
mainfrom
feature/issue-9/e2e-test-for-article-read
Mar 7, 2026
Merged

[#9] Add e2e tests for article read #15
idembele70 merged 2 commits into
mainfrom
feature/issue-9/e2e-test-for-article-read

Conversation

@idembele70
Copy link
Copy Markdown
Owner

@idembele70 idembele70 commented Mar 6, 2026

What does this implement/fix? Explain your changes.

  • Add E2E tests for article read feature

Does this close any currently open issues?

Any other comments?

None.

I'm not a dummy, so I've checked these

  • I've made changes requested by Copilot.
  • I made a self-review
  • I updated test-inventory.xlsx
  • I updated coverages.xlsx
  • I updated features.xlsx

@idembele70 idembele70 linked an issue Mar 6, 2026 that may be closed by this pull request
@idembele70 idembele70 self-assigned this Mar 6, 2026
@idembele70 idembele70 requested a review from Copilot March 6, 2026 17:29
@idembele70 idembele70 added the enhancement New feature or request label Mar 6, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Playwright E2E coverage for the “article read” feature and refactors existing article tests/utilities to use new @article/* path aliases and updated article payload models.

Changes:

  • Introduces new Article - Read E2E specs plus supporting fixtures/page objects/components.
  • Refactors article creation tests to use CreateArticleRequest (body, tagList) and new fixtures under @article/fixtures/*.
  • Adds new shared utilities/pages (e.g., HomePage, date formatting) and extends API/front URL helpers for article list & tags.

Reviewed changes

Copilot reviewed 20 out of 22 changed files in this pull request and generated 8 comments.

Show a summary per file
File Description
tsconfig.json Adds @article/* path alias for E2E tests.
e2e/tests/auth/specifications/login.spec.ts Switches to alias-based fixture import.
e2e/tests/auth/auth.utility.ts Adds helper to mint auth tokens per test/shard/worker.
e2e/tests/article/specifications/article-read.spec.ts New E2E coverage for listing, pagination, tag filtering, and negative/edge cases.
e2e/tests/article/specifications/article-creation.spec.ts Updates to new article payload model + new fixtures/import paths.
e2e/tests/article/pages/article-editor.page.ts Updates typing to CreateArticleRequest and rewires imports.
e2e/tests/article/pages/article-details.page.ts Adds URL navigation helper and updates assertions for new model (body, tagList).
e2e/tests/article/models/article.model.ts Expands Article and introduces CreateArticleRequest.
e2e/tests/article/fixtures/article-read.fixture.ts New fixture providing token, payload, and page objects for article read tests.
e2e/tests/article/fixtures/article-creation.fixture.ts New fixture replacing old article.fixture.ts.
e2e/tests/article/components/article-preview.component.ts New component assertions for article preview rows.
e2e/tests/article/components/article-form.component.ts New component for editor form interactions/assertions.
e2e/tests/article/article.utility.ts Adds API helpers and mocks for article list and tag list.
e2e/shared/utilities/url-front.utility.ts Adds front URL constants for article details and profile.
e2e/shared/utilities/url-api.utility.ts Adds API URL helpers for article list pagination and tags list.
e2e/shared/utilities/date.utilities.ts New date formatting utility used by preview assertions.
e2e/shared/pages/home.page.ts New Home page object used by article read tests.
e2e/shared/layouts/header.component.ts Adds authenticated-header visibility assertion helper.
docs/tests-inventories/0002_article-inventory.xlsx Adds/updates test inventory documentation artifact.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread e2e/tests/article/pages/article-editor.page.ts Outdated
Comment thread e2e/tests/article/fixtures/article-creation.fixture.ts Outdated
Comment thread e2e/tests/article/fixtures/article-read.fixture.ts Outdated
Comment thread e2e/shared/pages/home.page.ts
Comment thread e2e/tests/article/components/article-form.component.ts
Comment thread e2e/tests/article/components/article-preview.component.ts
Comment thread e2e/tests/article/components/article-preview.component.ts Outdated
Comment thread e2e/tests/article/pages/article-details.page.ts
@idembele70 idembele70 merged commit eaf582a into main Mar 7, 2026
99 of 105 checks passed
@idembele70 idembele70 deleted the feature/issue-9/e2e-test-for-article-read branch March 7, 2026 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

e2e test for article read

2 participants