Skip to content

feat: adjust e2e to pagination api MAPCO-9304 - #9

Open
ronenkapelian wants to merge 7 commits into
masterfrom
feat/adjust-e2e-to-pagination-api
Open

feat: adjust e2e to pagination api MAPCO-9304#9
ronenkapelian wants to merge 7 commits into
masterfrom
feat/adjust-e2e-to-pagination-api

Conversation

@ronenkapelian

Copy link
Copy Markdown
Contributor

List endpoints now return { total, items } instead of a raw array (breaking
change introduced in jobnik-manager PR #263).

Changes:

  • tests/multipleStagesWorkflow.spec.ts — update GET /v1/jobs/{jobId}/stages
    usages to access .items and .total via PaginatedResponse cast
  • tests/pagination.spec.ts — new dedicated spec with two tests:
    • stage ordering via the list endpoint
    • page_size slicing and total accuracy across pages
  • infrastructure/types.ts — shared PaginatedResponse<T> bridge type;
    marked @deprecated with a cleanup checklist for when the SDK ships updated
    types (page/page_size query params + response shape)

@ronenkapelian ronenkapelian changed the title feat: adjust e2e to pagination api feat: adjust e2e to pagination api MAPCO-9304 Jul 23, 2026
Comment thread package.json Outdated
"homepage": "https://github.com/MapColonies/jobnik-e2e#readme",
"dependencies": {
"@map-colonies/jobnik-sdk": "^0.3.0",
"@map-colonies/jobnik-sdk": "file:../jobnik-sdk/map-colonies-jobnik-sdk-0.3.0.tgz",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is this on purpose or just for your tests?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed with new release of sdk 1.0.1

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.

2 participants