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

Run integration tests on CI #13

Closed
2 of 7 tasks
php-coder opened this issue Aug 14, 2020 · 4 comments
Closed
2 of 7 tasks

Run integration tests on CI #13

php-coder opened this issue Aug 14, 2020 · 4 comments
Assignees
@php-coder
Labels
area/ci enhancement New feature or request
Milestone
0.0.3

Comments

@php-coder
Copy link
Owner

php-coder commented Aug 14, 2020 •
edited
Loading

@php-coder php-coder added the enhancement New feature or request label Aug 14, 2020
@php-coder php-coder self-assigned this Aug 14, 2020
@php-coder php-coder added this to the 0.0.3 milestone Aug 14, 2020
@php-coder
Copy link
Owner Author

Idea: we can generate from a single endpoints.yaml all supported types of applications, build them into docker container, run on different ports and execute tests against all of them at the same time.

@php-coder
Copy link
Owner Author

Idea: use --dest-dir option (#24)

php-coder added a commit that referenced this issue Oct 14, 2022
@php-coder
ci: add no-op workflows for testing generation of apps on different l…
ef147be 
…anguages

Part of #13
Part of #30
php-coder added a commit that referenced this issue Oct 14, 2022
@php-coder
ci: move workflow files to the appropriate directory
f6a14fd 
Correction for ef147be commit.

Part of #13
Part of #30
php-coder added a commit that referenced this issue Oct 14, 2022
@php-coder
ci: correct job's name
27ffed4 
Correction for ef147be commit.

Part of #13
Part of #30
@php-coder
Copy link
Owner Author

After adding unit tests (#31) we should run them in CI as well.

php-coder added a commit that referenced this issue Oct 27, 2022
@php-coder
ci: setup NodeJS and generare application on Python
e520355 
Part of #13
php-coder added a commit that referenced this issue Oct 28, 2022
@php-coder
ci: enable caching
7353c68 
See https://github.com/actions/setup-node/blob/main/docs/advanced-usage.md#caching-packages-data

Part of #13
php-coder added a commit that referenced this issue Oct 28, 2022
@php-coder
chore: update package-lock.json file with a newer version of NodeJS t…
e352297 
…o fix warnings on CI

npm ci complained on CI:

npm WARN old lockfile
npm WARN old lockfile The package-lock.json file was created with an old version of npm,
npm WARN old lockfile so supplemental metadata must be fetched from the registry.
npm WARN old lockfile
npm WARN old lockfile This is a one-time fix-up, please be patient...
npm WARN old lockfile

and I install a newer node version (18) and updated the lock file with the following command:

$ nvm install --lts
$ npm install --package-lock-only

See https://stackoverflow.com/questions/68260784/npm-warn-old-lockfile-the-package-lock-json-file-was-created-with-an-old-version

Part of #13
php-coder added a commit that referenced this issue Oct 28, 2022
@php-coder
ci: use the latest LTS version of NodeJS (18.x)
3cb3b2e 
Part of #13
php-coder added a commit that referenced this issue Oct 28, 2022
@php-coder
ci: use --dest-dir option instead of manually changing a working dire…
a6bf8e9 
…ctory

Part of #13
php-coder added a commit that referenced this issue Oct 28, 2022
@php-coder
ci: disable audit and fund during dependencies installation
13a3d29 
See:
- https://docs.npmjs.com/cli/v8/commands/npm-ci#audit
- https://docs.npmjs.com/cli/v8/commands/npm-ci#fund

Part of #13
php-coder added a commit that referenced this issue Nov 7, 2022
@php-coder
ci: generate applications on JavaScript and Golang
f043753 
Part of #13
php-coder added a commit that referenced this issue Nov 13, 2022
@php-coder
ci: fail a pipeline when non-committed files have been found
df22aa2 
Part of #13
php-coder added a commit that referenced this issue Nov 13, 2022
@php-coder
ci: add semicolons to make shell commands work with >- YAML notation
52fd844 
Correction for df22aa2 commit.

Part of #13
@php-coder php-coder mentioned this issue Feb 27, 2024
Closed
12 tasks
php-coder added a commit that referenced this issue Mar 19, 2024
@php-coder
ci: run integration tests for Express/JavaScript/MySQL
206d310 
Part of #13
php-coder added a commit that referenced this issue Mar 19, 2024
@php-coder
chore: disable progress meter for curl
5c5ef9d 
Relate to #13
php-coder added a commit that referenced this issue Mar 19, 2024
@php-coder
chore: fix installation of hurl
71f0994 
Error was: dpkg: error: requested operation requires superuser privilege

Correction for 206d310 commit.

Part of #13
php-coder added a commit that referenced this issue Mar 19, 2024
@php-coder
chore: fix installation of hurl because of invalid file content
18c2c9d 
Error was: dpkg-deb: error: 'hurl_4.2.0_amd64.deb' is not a Debian format archive

Correction for 206d310 commit.

Part of #13
php-coder added a commit that referenced this issue Mar 19, 2024
@php-coder
chore: don't remove base images during purging
83707b0 
Should be in 206d310 commit.

Part of #13
php-coder added a commit that referenced this issue Mar 19, 2024
@php-coder
chore: fix saving of container logs as artifact
e244cb3 
Correction for 206d310 commit.

Part of #13
php-coder added a commit that referenced this issue Mar 19, 2024
@php-coder
chore: include timestamps in container logs
7532ffb 
Should be in 206d310 commit.

Part of #13
php-coder added a commit that referenced this issue Mar 21, 2024
@php-coder
chore: app now waits till MySQL is up and runnning
097a708 
Correction for 206d310 commit.

Part of #13
php-coder added a commit that referenced this issue Mar 23, 2024
@php-coder
ci: run integration tests for Express/TypeScript/MySQL
6a5dc71 
Part of #13
php-coder added a commit that referenced this issue Mar 23, 2024
@php-coder
chore: don't publish MySQL port to the host to avoid conflict on CI w…
cae21bd 
…here 2 matrix jobs are running simultaneously

Correction for 6a5dc71 commit.

Relate to #13
php-coder added a commit that referenced this issue Mar 26, 2024
@php-coder
chore: fix artifacts uploading from matrix jobs by giving a unique na…
85feeaa 
…me to an artifact

The error was:
Error: Failed to CreateArtifact: Received non-retryable error: Failed request: (409) Conflict: an artifact with this name already exists on the workflow run

See also: actions/upload-artifact#480 (comment)

Part of #13
php-coder added a commit that referenced this issue Mar 26, 2024
@php-coder
chore: save app and db logs to separate files
c15739c 
Part of #13
php-coder added a commit that referenced this issue Mar 26, 2024
@php-coder
chore: don't show image pulling progress
7083cff 
Part of #13
php-coder added a commit that referenced this issue Mar 27, 2024
@php-coder
chore: don't run database container with docker compose
465d6fd 
Relate to #13
php-coder added a commit that referenced this issue Mar 27, 2024
@php-coder
chore: pre-create a directory for reports to make it exists even when…
cf3087b 
… integration tests weren't run

When `docker compose up` has failed because a container couldn't startup, the integration tests step
is skipped and directory hurl-reports wasn't created. In this case, "Save application logs" step
also has been failing with error:
  tee: ../hurl-reports/application-logs.txt: No such file or directory

Part of #13
php-coder added a commit that referenced this issue Mar 27, 2024
@php-coder
chore: save logs before stopping all containers
eaaa90d 
For unknown reason, sometimes "Save report" step wasn't executed. Try to move it close to the step
that was failing.

Relate to #13
php-coder added a commit that referenced this issue Mar 27, 2024
@php-coder
revert: "chore: don't run database container with docker compose"
3dcbec8 
This reverts commit 465d6fd.

Relate to #13
php-coder added a commit that referenced this issue Mar 27, 2024
@php-coder
revert: "chore: don't run database container with docker compose"
f4a97cd 
This reverts commit 465d6fd.

Relate to #13
php-coder added a commit that referenced this issue Mar 27, 2024
@php-coder
chore: allow to trigger integration tests manually
6bbf95c 
Relate to #13
php-coder added a commit that referenced this issue Mar 27, 2024
@php-coder
chore: enable debug for docker compose
5230d3c 
Relate to #13
php-coder added a commit that referenced this issue Mar 27, 2024
@php-coder
chore: save docker daemon logs
571e912 
Relate to #13
php-coder added a commit that referenced this issue Mar 28, 2024
@php-coder
fix(golang): correct endpoints.yaml to generate DTO with a type that …
70de65c 
…matches input type

The error was:
NamedExec failed: Error 1366: Incorrect integer value: '' for column 'created_by' at row 1

Relate to #9
Relate to #13
php-coder added a commit that referenced this issue Mar 28, 2024
@php-coder
chore: show container statuses only when the previous job isn't cannc…
ff02eb0 
…elled

Part of #13
php-coder added a commit that referenced this issue Mar 28, 2024
@php-coder
fix(golang): fix generated code for get_list method
2482db2 
The error was:
Select failed: scannable dest type slice with >1 columns (4) in result

Relate to #9
Relate to #13
php-coder added a commit that referenced this issue Mar 28, 2024
@php-coder
fix(golang): fix get_list method to return [] instead of null for emp…
00ef4cd 
…ty result set

Relate to #9
Relate to #13
php-coder added a commit that referenced this issue Mar 28, 2024
@php-coder
revert: "chore: enable debug for docker compose"
d16cb2e 
This reverts commit 5230d3c.

Relate to #13
php-coder added a commit that referenced this issue Mar 28, 2024
@php-coder
revert: "chore: save docker daemon logs"
64134c1 
This reverts commit 571e912.

Relate to #13
php-coder added a commit that referenced this issue Mar 29, 2024
@php-coder
ci: run integration tests for FastAPI/Python/PostgreSQL
e9d4ca1 
Part of #13
php-coder added a commit that referenced this issue Mar 29, 2024
@php-coder
refactor: move steps for setting up hurl, closer to its usage
4e2f076 
Relate to #13
php-coder added a commit that referenced this issue Mar 29, 2024
@php-coder
chore: fix running of fastapi container because postgres didn't have …
1abdf9b 
…a healthcheck

The error was:
dependency failed to start: container fastapi-postgres-1 has no healthcheck configured

Part of #13
php-coder added a commit that referenced this issue Mar 29, 2024
@php-coder
chore: fix docker-compose.yml syntax
77b1a26 
The error was:
parsing docker-compose.yaml: yaml: line 37: did not find expected '-' indicator

Correction for 1abdf9b commit.

Relate to #13
php-coder added a commit that referenced this issue Apr 1, 2024
@php-coder
chore: correct internal port for FastAPI
f001ce4 
Part of #13
php-coder added a commit that referenced this issue Apr 4, 2024
@php-coder
chore(python): make all model fields optional by default
67a664a 
Part of #16
Relate to #13
php-coder added a commit that referenced this issue Apr 4, 2024
@php-coder
chore(python): unbreak optional fields on Python 3.7
7e1ff65 
The error was: TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

See https://stackoverflow.com/questions/76712720/typeerror-unsupported-operand-types-for-type-and-nonetype

Correction for 67a664a commit.

Part of #16
Relate to #13
php-coder added a commit that referenced this issue Apr 4, 2024
@php-coder
chore: don't save logs as artifact
4e22c12 
Part of #13
php-coder added a commit that referenced this issue Apr 5, 2024
@php-coder
chore: relax check to not require "charset" for "application/json" co…
1ece6c0 
…ntent type

Because FastAPI doesn't set "charset" parameter, this change should make the tests pass.

RFC 4627 does not define a charset parameter because it requires that JSON is always encoded as
Unicode (c) request/request#383 (comment)

Part of #13
php-coder added a commit that referenced this issue Apr 6, 2024
@php-coder
chore: remove "limit" query parameter from get_list in order to make …
92ff16d 
…integration tests pass

Partically reverts changes from f524f15 commit.

Reopen #23
Part of #13
Relate to #41
@php-coder
Copy link
Owner Author

Done!

php-coder added a commit that referenced this issue Apr 17, 2024
@php-coder
chore: don't cancel all in-progress and queued jobs in the matrix if …
5185d21 
…any job in the matrix fails

Relate to #13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@php-coder php-coder

Labels
area/ci enhancement New feature or request
Projects
None yet
Milestone
0.0.3
Development

No branches or pull requests
1 participant
@php-coder