OpenAPITools/openapi-generator

[Clojure] Add more complex spec tests to run the generator on

Open

#743 opened on Aug 6, 2018

View on GitHub
 (2 comments) (1 reaction) (0 assignees)Java (7,516 forks)batch import
Client: ClojureEnhancement: CI/TestHacktoberfesthelp wanted

Repository metrics

Stars
 (26,243 stars)
PR merge metrics
 (Avg merge 9d 1h) (123 merged PRs in 30d)

Description

Description

As we figured out in #122, the current Petstore spec is not enough to test all the edge cases of the generator.

Suggest a fix/enhancement

As @wing328 suggested in https://github.com/OpenAPITools/openapi-generator/pull/122#issuecomment-403228826, we should start testing on the fake petstore spec.

Rough list of things to do for this issue:

  • add a line to bin/clojure-petstore.sh to test against that new spec
  • add Clojure tests in samples/client/fake-petstore/clojure/test/

Contributor guide

Research direction
Explore the existing Clojure test structure in samples/client/petstore/clojure/test and the shell script bin/clojure petstore.sh. Then add a similar test setup for the fake petstore spec as described.
Tech stack
clojure
Domain
backendtooling
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Mostly clear
Prerequisites
ClojureGit
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65