Skip to content

Commit

Permalink
Fix certification failure for Commercetools Connect (#1103)
Browse files Browse the repository at this point in the history
* Fix documentation format

* Move c8 node module to be devDependencies in package.json

* Add connect-dedicated build step

* Fix mock config json format

* Fix notification/package.json

* Update documentation

* Revert change in mock-default-config.js
  • Loading branch information
leungkinghin-ct committed Jul 28, 2023
1 parent f9a2ed0 commit 02002de
Show file tree
Hide file tree
Showing 9 changed files with 300 additions and 84 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ jobs:
--runtime nodejs16 \
--trigger-http \
--allow-unauthenticated
- run: npm run test
- run: npm run test-ci

notification_test:
name: Test for Notification Module
Expand All @@ -115,4 +115,4 @@ jobs:
node-version: '16.x'
- run: npm ci
- run: npx lockfile-lint --path package-lock.json --allowed-schemes "https:" "git+https:" --allowed-hosts npm github.com
- run: npm run test
- run: npm run test-ci
5 changes: 3 additions & 2 deletions docs/ContributionGuide.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,13 +23,14 @@ npm install

## Development
While developing project you can use some predefined commands for running tests, running linter or generating coverage.
- Execute `npm run test` to run all tests and print the code coverage report.

- Execute `npm run test-ci` to run all tests and print the code coverage report.
- Execute `npm run unit` to run Unit tests.
- Execute `npm run integration` to run Integration tests. (environment variable for [extension](../extension/docs/HowToRun.md#environment-variable) and [notification](../notification/docs/HowToRun.md#environment-variable) needs to be set)
- Extension module only: Execute `npm run e2e` to run E2e tests. (environment variable for [extension](../extension/docs/HowToRun.md#environment-variable) needs to be set)
- Execute `npm run lint` to show lint errors in the code.
- Execute `npm run format` to format the code before committing.
- Execute `npm run test` to pre-run the build before the deployment in [Commercetools Connect](https://docs.commercetools.com/connect)

### Debugging

Expand Down
Loading

0 comments on commit 02002de

Please sign in to comment.