Contributing
September 27, 2024 ยท View on GitHub
Thank you for taking your time to contribute to the SAP Cloud SDK!
Please make sure, that you are familiar with our style guide.
Project Structure
This project contains multiple packages, that are managed using turborepo and yarn workspaces. Productive packages are located in the packages directory, test packages are located in the test-packages directory.
All dependencies that are used in more than one of the packages are hoisted into the root project. Dependencies, that occur only once and binaries will be placed in the node_modules of that specific package. Some of the packages are interdependent, therefore yarn install won't work from within those packages. Run yarn install in the root directory instead.
Testing
All (new) functionality shall be covered by tests.
In order to run all tests, execute:
$ yarn test
This will run unit tests for all our packages as well as integration tests and type tests. You can run those individually as described in the following.
Unit Tests
Unit tests shall test specific modules of a package, units that are tested for behavior. You can run all unit tests by executing:
$ yarn test:unit
To run unit tests for a specific package add the workspace name to the command. For the core package this would be:
$ yarn @sap-cloud-sdk/core run test
Integration Tests
Integration tests shall test how modules behave in combination. The integration tests are located in test-packages/integration-tests.
To run the integration tests, execute:
$ yarn test:integration
Type Tests
As this project is written in TypeScript, it shall be consumable by other TypeScript projects. We use dtslint to test that our resulting API meets our expectations.
The type tests are located at test-packages/type-tests.
To run the integration tests, execute:
$ yarn test:type
Test Services
To simplify testing we are using tests services.
The specifications for those can be found in test-resources.
They are used from different tests on different levels.
The service specifications are directly used by the unit tests of the generator.
From the specifications, we generate two OData clients.
First, we generate type script sources, that are generated into the test-utils of the core, where we replace all references to the @sap-cloud-sdk core packages with a reference to the local sources.
The resulting test service is used in the unit tests of the core packages.
Second, we generate a transpiled version of a non-modified OData client based on the specifications.
This is used in the integration tests and type tests.
If you need to extend the existing services, run the following to regenerate the OData clients.
$ yarn generate
End to End (E2E) tests
The E2E are the most realistic tests included in this repo and run on each pull request. There are also nightly tests executed on the internal jenkins - see the internal repo for more details on these. These are also called E2E tests but are not meant here.
The E2E tests are based on a locally running server providing an OData interface using CAP and a REST interface using OpenAPI.
This server is used by the E2E tests located at test-packages/e2e-tests.
Attention The imports in the E2E tests use the root packages e.g. @sap-cloud-sdk/core to mimic the way a customer would use it.
So if you made code changes in one of the packages you need to run yarn compile to make the changes take effect.
For manual E2E to a real remote system we have also some tests against the TripPin service which is the standard OData V4 sample service. Since the remote service is not really stable we commented out the tests under test-packages/e2e-tests/test/TripPin but for manual testing they can be useful.
Checks
To perform all available checks, run:
$ yarn checks
Warning Please note the plural s which is needed because yarn has its own (legacy) 'check' command which is not recommended to be used.
Individual checks can be run with individual scripts:
$ yarn check:MY_CHECK
Where MY_CHECK can be any of the checks defined in the top level package.json file.
Linting
To fix all linting issues, run:
$ yarn lint:fix
How to contribute
When contributing to this repository, please first discuss the changes you wish to make through an issue, email, or any other method with the owners of this repository.
Please note, that we have a code of conduct, please follow it in all your interactions with the project.
Once you are ready to make a change, please test it appropriately, create a pull request and describe your change in the pull request. The owners of the repository will review your changes as soon as possible.
Developer Certificate of Origin (DCO)
Due to legal reasons, contributors will be asked to accept a DCO before they submit the first pull request to this project. This happens in an automated fashion during the submission process. SAP uses the standard DCO text of the Linux Foundation.
Release Process
Information on the release process can be found in the release documentation.
Contributing with AI-generated Code
As artificial intelligence evolves, AI-generated code is becoming valuable for many software projects, including open-source initiatives. While we recognize the potential benefits of incorporating AI-generated content into our open-source projects there a certain requirements that need to be reflected and adhered to when making contributions.
Please see our Guideline for AI-generated code contributions to SAP Open Source Software Projects for more details.