Gitlab Community Edition Instance

Skip to content
Snippets Groups Projects

Feature/#52 annotationapi tests

Merged Feature/#52 annotationapi tests
feature/#52-annotationapi-tests into develop
Merged Michelle Weidling requested to merge feature/#52-annotationapi-tests into develop

Feature

Summary

This MR provides…

  • extensive tests for the AnnotationAPI. For this, some minor changes several functions have been made and several minor bugs fixed.
  • the annotation module has been split to annotation.xqm and annotation-rest.xqm, the latter holding only the REST interface definitions. Both reside now in their own collection, modules/AnnotationAPI, which makes the app a bit more clear.
  • a better structure of the modules by moving the fragment retrieval to commons.xqm. This way, annotations.xqm doesn't have to import tapi-html.xqm which makes no sense (the HTML creation is logically independent from the annotations).
  • test-runner.xq which can be used for local unit testing now inherits from test-trigger.xqm, thus reducing duplications (DRY!). Cf. #62 (closed)
  • The output of the tests have been remodeled to be both human and machine readable which addresses #61 (closed).
  • The sample files have been renamed for clarity and slightly altered/enhanced to better depict the project's data structure.
  • The content of repo.xml has been shortened in order to avoid having version information on two places.

Compliance to “Definition of Done”

  • Unit tests passed
  • Code reviewed
  • Acceptance criteria met
  • Functional tests passed
  • Non-Functional requirements met
  • Product Owner accepts the User Story

Documentation

  • I updated existing documentation

Tests

Are we able to test this new feature?

  • Yes, everything can be done via unit tests.

Changelog

  • I added a statement to the CHANGELOG.

Version number

  • I bumped the version number in build.properties.

Related Tickets

Closes

Closes #52 (closed), #61 (closed), #62 (closed).

/cc Mathias Göbel, Frank Schneider, Michelle Weidling

Edited by Michelle Weidling

Merge request reports

Pipeline #167190 passed

Stage: unit_tests
Stage: build
Stage: package
Stage: deploy_docker
Stage: deploy_app
Stage: api_tests

Pipeline passed for 246e039e on feature/#52-annotationapi-tests

Merged by avatar (Apr 13, 2025 4:02pm UTC)

Loading

Pipeline #167257 passed

Stage: unit_tests
Stage: build
Stage: package
Stage: deploy_docker
Stage: deploy_app
Stage: api_tests

Pipeline passed for 491208cc on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply