Skip to content
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

Validate OpenAPI in the CI #603

Open
wants to merge 61 commits into
base: main
Choose a base branch
from
Open

Validate OpenAPI in the CI #603

wants to merge 61 commits into from

Conversation

JVT038
Copy link
Collaborator

@JVT038 JVT038 commented Mar 25, 2024

This PR is based on #592

It adds a new job which uses IBM's OpenAPI validator to check if the OpenAPI specs are valid.

@JVT038
Generated HTTP tests based on OpenAPI
3e85e52 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Add HTTP test to GitHub workflow
4796832 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Merge branch 'main' into add-http-tests-ci
68f4000
@JVT038
Try to unify the tests and use mariadb
4b48b02 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Fix environment variables
01fe434 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Allow empty root password
e1b29b5 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Fix environment variables
f9fb000 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Fix composer cache directory
c5bf63b 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Update dependencies and switch to maintained version of php_codesniffer.
fc84675 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Downgrade phpunit in test workflow
627936e 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Hopefully fixed caching for composer
88123d8 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Some more trial-and-error...
6fc862c 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Try running a docker image instead.
278acbb 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Fix docker image?
cd3770e 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Switch to sqlite
a464957 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Set sqlite path
4e8a4d6 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Once again back to mariadb
ff10295 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Maybe this fixes mariadb?
3f08864 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Fix ijhttp alias
d147860 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Fix ijhttp command
cba8710 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Maybe the native PHP server suddenly works again?
ed20506 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
fix variables
3d573df 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
fix variables?
3253b92 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Switch back to docker due to lack of permission for listening on port 80
a5b34a6 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Fix typo
c4fd3cc 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Add test assertions for GET /authentication/token endpoint.
05156a5 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Fix assertions
d5d9414 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Merge branch 'main' into add-http-tests-ci
84c648a
@JVT038
Try to run in-built server with sudo instead
72bbf0f 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Hopefully the PHP server stops now.
cb14de7 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Added cs2pr for printing
f8fa0b3
@JVT038
remove cs2pr
db452ed
@JVT038
Fix preparation
a618d0b
@JVT038
Re-add cs2pr
033f46f
@JVT038
Fix cs2pr and networking
b15a0f0
@JVT038
fix network host
333f259
@JVT038
Fix networking
2471764
@JVT038
Fix networking
7d33e54
@JVT038
Remove cs2pr from phpstan
0e04ce3
@JVT038
Add docker container for http tests
622dbbb
@JVT038
Fix docker container ports
9e1223c
@JVT038
Switch to bridge mode
3981e71
@JVT038
Test creating the container with container key
5eb35dd
@JVT038
Remove apk add command
7e0309f
@JVT038
Don't store env variables in env file
4313845
@JVT038
Return to old method of docker containers
8ea6c57
@JVT038
Fix network name in variable
9d00a62
@JVT038
Fix network name variable output
28ccf08
@JVT038
Remove quotes from network name
ce6f977
@JVT038
Remove ampersands
fd7eab7
@JVT038
Resolve 127.0.0.1 to movary container
c983c02
@JVT038
Bind movary container to database network
dba9e06
@JVT038
Remove host gateway dns
cbd889f
@JVT038
Fix ports
99c056f
@JVT038
Remove last docker commands
837da27
@JVT038
Clean up
264a681
@JVT038
Add job to validate OpenAPI specs
7907a4e 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038 JVT038 requested a review from leepeuker as a code owner March 25, 2024 15:10
@JVT038
Remove npm cache
38b7f06 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
@JVT038
Only include summary in output
519347d 
Signed-off-by: JVT038 <47184046+JVT038@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@leepeuker leepeuker Awaiting requested review from leepeuker leepeuker is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@JVT038