Skip to content

Conversation

Tom-Willemsen
Copy link
Member

@Tom-Willemsen Tom-Willemsen commented May 20, 2025

Copyedit GUI -> Testing section. Summary of changes below.


Adding unit tests

  • General sentence-level/formatting changes

An introduction to unit testing

  • General sentence-level/formatting changes
  • Run through all examples to check they still work, update where they didn't
  • Update instructions and screenshots for newer versions of Eclipse

Mockito

  • Run through examples to check they work, update where they didn't
  • Corrections/updates for newer Mockito APIs
    • e.g. recommend @Mock over Mockito.mock() as it's type-safe with generics
  • Link much more prominently to authoritative upstream documentation
    • I'm not actually sure we really want to keep much of this page - it's mostly a duplicate of the (higher-quality) upstream documentation. But have kept for now as it's "only" a few hundred lines of duplication.

Squish

  • General formatting improvements
  • Correct various dead links

Squish BDD tools

  • General formatting improvements

Test naming

  • General sentence-level/formatting changes
  • No substantive change to the documented conventions
  • Correct misleading example

@Tom-Willemsen Tom-Willemsen force-pushed the gui_testing_copyedit branch from 980053b to efd7f23 Compare May 20, 2025 17:10
@Tom-Willemsen Tom-Willemsen marked this pull request as ready for review May 29, 2025 08:50
@Tom-Willemsen Tom-Willemsen moved this to Review in PI_2025_02 May 29, 2025
@rerpha rerpha merged commit e48a9ce into master Jun 5, 2025
2 checks passed
@github-project-automation github-project-automation bot moved this from Review to Done in PI_2025_02 Jun 5, 2025
@rerpha rerpha deleted the gui_testing_copyedit branch June 5, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants