swagger-api / swagger-api/swagger-codegen

[CI/CD] Integration-Tests / Samples execution/generation on CI env

Open
#9,823 1 comment 0 reactions 1 assignee View on GitHub

@HugoMario is already working on this.

Since May 8, 2020.

Dominant language
Mustache
Stars
17.8k
Forks
6k
PR merge metrics
No merged PRs in 30d

Description

Description

Currently to me the testing approach seems a little bit weired:

  • The generated samples are checked in
  • There are scripts within bin which use latest version
  • Not all samples are executed
Swagger-codegen version

latest 3.0.0 branch

Swagger declaration file content or url

N/A

Command line used for generation

N/A

Steps to reproduce
 mvn -q --batch-mode verify -Psamples

;)

Related issues/PRs

e.g. this error would have been found before releasing: #9822

Suggest a fix/enhancement
  1. I think the samples folder should just hold the pom.xml
  2. For the rest the scripts should run on CI
  3. All samples should be executed on travis

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.