Skip to content

Conversation

@Anshukumar123975
Copy link
Contributor

Description:

This PR adds a new validate directory that provides setup for testing the core components:

  1. modelloader
  2. factory
  3. serializer

The tests ensure proper functioning with both valid and invalid data models and json instances.

Directory structure for validate:

validate/
├── features/
│ └── validate.feature
├── models/
│ ├── valid/
│ │ ├── model.cto
│ │ └── instance.json
│ └── invalid/
│ ├── badmodel.cto
│ └── badInstance.json
└── validateSteps.js

package.json Outdated
"type": "module",
"scripts": {
"test": "cucumber-js \"semantic/features/**/*.feature\" --loader ts-node/esm --import \"semantic/features/support/Javascript/*.ts\" --tags \"not @skip\"",
"test": "npm run test:semantic && npm run test:cli",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

leave out cli part

package.json Outdated
"test": "cucumber-js \"semantic/features/**/*.feature\" --loader ts-node/esm --import \"semantic/features/support/Javascript/*.ts\" --tags \"not @skip\"",
"test": "npm run test:semantic && npm run test:cli",
"test:semantic": "cucumber-js \"semantic/features/**/*.feature\" --loader ts-node/esm --import \"semantic/features/support/Javascript/*.ts\" --tags \"not @skip\"",
"test:cli": "cucumber-js validate/features --import validate/validateSteps.js",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"test:cli": "cucumber-js validate/features --import validate/validateSteps.js",
"test:instances": "cucumber-js validate/features --import validate/validateSteps.js",

…ged the name from test:cli to test:instances

Signed-off-by: Anshukumar123975 <[email protected]>
Signed-off-by: Anshukumar123975 <[email protected]>
Signed-off-by: Anshukumar123975 <[email protected]>
Signed-off-by: Anshukumar123975 <[email protected]>
Signed-off-by: Anshukumar123975 <[email protected]>
@ekarademir ekarademir merged commit e306d6c into accordproject:main Aug 1, 2025
2 checks passed
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