Skip to content
Snippets Groups Projects

WorkerRun on Classification

Merged WorkerRun on Classification
classification-worker-run into master
All threads resolved!
Merged Erwan Rouchet requested to merge classification-worker-run into master
All threads resolved!

Closes #1024 (closed)

This adds a generic validator in arkindex.project.tools to validate the WorkerVersion XOR WorkerRun in the API and uses a check constraint in the DB to ensure that we always store a WorkerVersion when we store a WorkerRun.

To make unit tests simpler, this adds a new DataImport and two WorkerRuns to the test fixtures, which required a dozen fixes in some DataImport-related unit tests.

Edited by Erwan Rouchet

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Bastien AbadieBastien Abadie 2 years ago (May 13, 2022 1:26pm UTC)

Merge details

  • Changes merged into master with db86810e (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Bastien Abadie resolved all threads

    resolved all threads

  • Bastien Abadie merged

    merged

  • Bastien Abadie mentioned in commit db86810e

    mentioned in commit db86810e

  • Bastien Abadie
    Bastien Abadie @babadie ·
    Owner

    Thanks for the comments, really nice :thumbsup:

    • Please register or sign in to reply