Skip to content

Change the method of Add CTO, Remove vitest test files, and Workflow Updates #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Jun 19, 2025
Merged

Change the method of Add CTO, Remove vitest test files, and Workflow Updates #14

merged 3 commits into from
Jun 19, 2025

Conversation

@Anshukumar123975
Copy link
Contributor

@Anshukumar123975 Anshukumar123975 commented Jun 18, 2025 •
edited
Loading

Description (Closes #12)

Summary of Changes:

This PR includes several important updates to improve the overall structure, language support, and CI reliability of the repository:

  1. Refactored the support/ directory to accommodate step definitions and test support for multiple languages
  2. Updated model loading logic to first convert cto files to ast and then adding to the modelmanager
  3. Applied @Skip tags to scenarios that are currently failing due to known issues, ensuring green CI builds without deleting any test coverage.
  4. Removed Vitest unit test files as to completely migrate to Cucumber test suite.

@Anshukumar123975
Updated the support directory to add multiple language support , adde…
6e0ba9c 
…d the skip tags to the tests which are currently failing, remove the vitest test files, updated the way to load models using parser and modelfile

Signed-off-by: Anshukumar123975 <[email protected]>
@Anshukumar123975
Updated the scripts within the package.json
eccbcce 
Signed-off-by: Anshukumar123975 <[email protected]>
@Anshukumar123975
Updated package.json script for github actions workflow
ab1c0b5 
Signed-off-by: Anshukumar123975 <[email protected]>
@Anshukumar123975 Anshukumar123975 changed the title Change the method of LoadCTO, Remove vitest test files, and Workflow Updates Change the method of Add CTO, Remove vitest test files, and Workflow Updates Jun 18, 2025
@ekarademir ekarademir merged commit e7144da into accordproject:main Jun 19, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ekarademir ekarademir ekarademir approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Remove vitest

2 participants

@Anshukumar123975 @ekarademir