Commit b7ae7af
authored
Add integration test CI check (#17)
### Motivation
We want to make sure that we don't unknowingly introduce changes in the
runtime
that break the generator, which relies on its SPI.
We have an package in a subdirectory in the generator repo that we can
be used
as an integration test to gate changes to the various repos in Swift
OpenAPI
project, including the runtime package.
The generator repo also contains a script that can be used to clone the
integration test package and use `swift package edit` to override a
dependency
to check it still functions with the proposed changes.
### Modifications
- Bring in the `run-integraton-test.sh` script from the generator repo.
- Run this script as part of the soundness pipeline.
### Result
On each pull request, the integration test package will be built with
the
changes proposed in the pull request.
### Test Plan
The CI pipeline for this PR will run the integration test because it's
been
added to the soundness script, which is run as part of an existing CI
pipeline.
I have also validated this locally:
```console
❯ docker-compose -f docker/docker-compose.yaml run --build
soundness
...
** Running /code/scripts/run-integration-test.sh...
...
** Extracting name for Swift package: /code
** Overriding dependency in /code/tmp.run-integration-test.sh.uyS9Hf8siA.noindex/swift-openapi-generator/IntegrationTest on swift-openapi-runtime to use /code
...
Build complete! (42.53s)
** ✅ Successfully built integration test package.
** ✅ All soundness check(s) passed.
```
### Notes
This PR adds the integration test to the soundness script, but we
probably want
to split this out into its own pipeline in the future.
Signed-off-by: Si Beaumont <beaumont@apple.com>1 parent b66f94a commit b7ae7af
File tree
3 files changed
+53
-1
lines changed- docker
- scripts
3 files changed
+53
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
| 31 | + | |
31 | 32 | | |
32 | 33 | | |
33 | 34 | | |
| |||
0 commit comments