Skip to content

Add unit tests for functions that don't require Ansys - #35

Merged
sadda merged 7 commits into
mainfrom
20-unit-tests
Sep 9, 2026
Merged

sadda merged 7 commits into
mainfrom
20-unit-tests

Conversation

@haitao-ZCU

Copy link
Copy Markdown
Collaborator

Adds unit tests (do not need to run Ansys) for :

  • barrier generation (get_arc, signed_distance, check_barrier), covering the OneLambda, SixLambdas, and 3BrokenLines generators
  • rotation_matrix, rotate, and analyze_results
  • objective_transform and init_points
  • extract_results unit conversion (both motor1 and motor2)

@sadda

sadda commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

@haitao-ZCU Have you checked (as mentioned in the issue)?

MANUALLY (that is without an AI agent) check whether all tests make sense. We had an example of a test confirming a bug.

@haitao-ZCU

Copy link
Copy Markdown
Collaborator Author

@sadda
Yes, I have manually checked the physical values and each test.

@sadda
sadda merged commit 7f8a327 into main Sep 9, 2026
6 checks passed
@haitao-ZCU
haitao-ZCU deleted the 20-unit-tests branch September 9, 2026 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants