Skip to content

feat: added abort api endpoint (MAPCO-8328)#59

Merged
CL-SHLOMIKONCHA merged 20 commits intoalphafrom
feature/abort-deployment
Feb 8, 2026
Merged

feat: added abort api endpoint (MAPCO-8328)#59
CL-SHLOMIKONCHA merged 20 commits intoalphafrom
feature/abort-deployment

Conversation

@roicohen326
Copy link
Contributor

Question Answer
Bug fix
New feature
Breaking change
Deprecations
Documentation
Tests added
Chore

@roicohen326 roicohen326 changed the base branch from master to alpha January 18, 2026 18:39
@CL-SHLOMIKONCHA
Copy link
Contributor

Attach the issue from jira (MAPCO) to PR title

@roicohen326 roicohen326 changed the title feat: added abort api endpoint feat: added abort api endpoint (MAPCO-8328) Jan 19, 2026
@roicohen326
Copy link
Contributor Author

Attach the issue from jira (MAPCO) to PR title

fixed

roicohen326 and others added 13 commits January 21, 2026 17:26
* test: add unit tests for abort deployment functionality

* fix: remove unused isJobAbortableSpy

* fix: return empty array from get task stub

* chore: removed unnecessary tests and rearranged in it each fromat

* chore: fixed tests description

* chore: fixed untracked jobs unit tests

* chore: fixed actions bad pattern

* chore: removed unnecessary BadRequestError

* chore: merge invalid status tests and use mock generator

* chore: fixed bad jobmanager tests patterns

* test: add job manager wrapper integration tests

* chore: removed unnecessary integration tests

* chore: fixed lint problems

* test: fixed wrong http status code

* test: add bad and sad abort integration tests

* test: fixed infocontroller unit tests

* chore: fix uncovered  vlaidation unit tests

* chore: removed unnecessary controllers unit testing due to prototype instance of conflict

* chore: refactor abort mocks and extract abort statuses as const

* test: adjust generateMockJob + abort expectations

* test: removed unnecessary as const definition

* style: fix lint errors

* style: removed unused import

* chore: fix eslint warnings

---------

Co-authored-by: shlomiko <shlomiko@rafael.co.il>
openapi3.yaml Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

please fix this description to Internal Server Error

in other places where 500 error is described fix the casing to as above

tnx

Copy link
Contributor

Choose a reason for hiding this comment

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

fixed it, thanks @vitaligi

@CL-SHLOMIKONCHA CL-SHLOMIKONCHA merged commit 345d894 into alpha Feb 8, 2026
7 checks passed
@CL-SHLOMIKONCHA CL-SHLOMIKONCHA deleted the feature/abort-deployment branch February 8, 2026 08:30
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.

3 participants