Skip to content
Snippets Groups Projects

Add test documentation

Merged Add test documentation
test-documentation into master
All threads resolved!
Merged l.dressen requested to merge test-documentation into master
All threads resolved!

Documentation for testing in PeTrack, please make me aware if I forgot something regarding tests

Reviewer Checklist

Formatting

  • the pre-build checks succeed

General code quality

  • naming conventions are met (see .clang-tidy for detailed information)
  • no static analyzer warnings in new code parts (e.g., use clang-tidy for checking)

General usability

  • old versions of pet-files are still loadable

Only if changes in UI

  • new elements are also saved and loaded from pet-file
  • check if tab order is still correct
  • all new SpinBoxes are promoted
  • new keybindings added to Petrack::keyBindings()
Edited by Tobias Arens

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Tobias Arens
  • Tobias Arens
    Tobias Arens @t.arens
    Developer

    Why did you add the same text in two different files?

  • d.kilic
  • l.dressen added 1 commit

    added 1 commit

    Compare with previous version

  • l.dressen added 27 commits

    added 27 commits

    Compare with previous version

    Toggle commit list
  • Tobias Arens marked the checklist item the pre-build checks succeed as completed

    marked the checklist item the pre-build checks succeed as completed

  • l.dressen added 1 commit

    added 1 commit

    • 634f34ba - moved test and added it to the site

    Compare with previous version

  • Tobias Arens resolved all threads

    resolved all threads

  • Tobias Arens approved this merge request

    approved this merge request

  • l.dressen added 12 commits

    added 12 commits

    Compare with previous version

    Toggle commit list
  • l.dressen enabled an automatic merge when the pipeline for 80279fb8 succeeds

    enabled an automatic merge when the pipeline for 80279fb8 succeeds

  • l.dressen aborted the automatic merge because target branch was updated

    aborted the automatic merge because target branch was updated

  • l.dressen added 10 commits

    added 10 commits

    Compare with previous version

    Toggle commit list
  • l.dressen enabled an automatic merge when the pipeline for 82ac8e00 succeeds

    enabled an automatic merge when the pipeline for 82ac8e00 succeeds

  • l.dressen mentioned in commit b0cc47e0

    mentioned in commit b0cc47e0

  • l.dressen merged

    merged

    • Please register or sign in to reply