Skip to content

Add unit tests #20

Closed
Closed

Description

@sadda

Short summary

The unit tests are quite limited.

Long description

  • MANUALLY (that is without an AI agent) check whether all tests make sense. We had an example of a test confirming a bug.
  • Add short unit tests for functions which do not need to run Ansys.
  • Add tests for the computation of motor2 (or whatever it was renamed to). Create a new golden directory and copy the baseline for motor2 there as the first commit, which worked for motor2 (that means not the current commit).

Estimated time investment

4 hours.

Activity

  1. sadda commented on Sep 9, 2026

    @sadda
    CollaboratorAuthor

    @haitao-ZCU Thanks for the PR 35. Please also finish the last bullet point about adding the test for the second motor. It would probably be nice to precompute the desired value so that Ansys is not run repeatedly for the old model and store it in some text-readable format (not pickle).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions