FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add solver input validation and test scripts by ferrari212 · Pull Request #83 · FEAScript/FEAScript-core · GitHub

Add solver input validation and test scripts - #83

Merged
nikoscham merged 10 commits into
FEAScript:mainfrom
ferrari212:feature/tests
Jul 9, 2026
Merged

Add solver input validation and test scripts#83
nikoscham merged 10 commits into
FEAScript:mainfrom
ferrari212:feature/tests

Conversation

Copy link
Copy Markdown
Collaborator

Description

This PR introduces an initial regression and unity testing structure for FEAScript-core as part of the ongoing improvements proposed in #50.

The main goal is to establish a foundation for validating existing functionality and preventing unintended changes in future development. These tests provide baseline coverage that can be expanded as new features, bug fixes, and improvements are introduced.

Changes

  • Added an initial regression test structure.
  • Added baseline test cases to verify existing behavior.
  • Added reference cases for future comparison and validation.
  • Improved the testing workflow to support future development and refactoring.

Motivation

As FEAScript-core continues to evolve, maintaining a reliable regression testing suite is important to ensure that existing functionality remains stable while new capabilities are added.

This PR represents a first step toward broader automated validation coverage and provides a framework where additional regression cases can be included incrementally.

Testing

  • Add regression tests covering initial scenarios.
  • Add unity tests covering initial scenarios.
  • Verified that the current implementation passes the added tests.

Related Issue

Partially addresses #50

nikoscham self-assigned this Jun 17, 2026
nikoscham added the enhancement New feature or request label Jun 17, 2026
nikoscham linked an issue Jun 17, 2026 that may be closed by this pull request

nikoscham commented Jun 24, 2026
edited
Loading

Copy link
Copy Markdown
Member

Thanks for the pull request @ferrari212. Very useful to have a reference solution for the test cases. Before merging can you please rename the src/methods/jacobiMethodScript.js to src/methods/jacobiSolver.js ? I have renamed the jacobi method file in the main branch and the old name creates a lot of conflicts. Thanks!

nikoscham merged commit 57d94ff into FEAScript:main Jul 9, 2026
2 checks passed
ferrari212 deleted the feature/tests branch July 9, 2026 09:38
nikoscham linked an issue Jul 14, 2026 that may be closed by this pull request
2 tasks
nikoscham added a commit that referenced this pull request Jul 14, 2026
Co-authored-by: ferrari212 <felipe.ferrari.212@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Performance Benchmarking Build Data-Driven Testing & Quality Assurance Infrastructure

2 participants


Back | FazBrowse Home | New Git URL