24356ac...master · typeable/openapi3

git.iko.soy/typeable/openapi3

on master help sign in

.github/workflows/haskell-ci.yml+88−31

since splitsnapshots

unifiedsplit

@@ -8,67 +8,117 @@

9# For more information, see https://github.com/haskell-CI/haskell-ci

10#

11−# version: 0.12.1

11+# version: 0.14.3

12#

13−# REGENDATA ("0.12.1",["github","cabal.project"])

13+# REGENDATA ("0.14.3",["github","cabal.project"])

14#

15name: Haskell-CI

16on:

17− - push

17+ push:

18+ branches:

19+ - master

18− - pull_request

20+ pull_request:

21+ branches:

22+ - master

19jobs:

23jobs:

20 linux:

24 linux:

21 name: Haskell-CI - Linux - ${{ matrix.compiler }}

25 name: Haskell-CI - Linux - ${{ matrix.compiler }}

22 runs-on: ubuntu-18.04

26 runs-on: ubuntu-18.04

27+ timeout-minutes:

28+ 60

23 container:

29 container:

24− image: buildpack-deps:xenial

30+ image: buildpack-deps:focal

25 continue-on-error: ${{ matrix.allow-failure }}

31 continue-on-error: ${{ matrix.allow-failure }}

26 strategy:

32 strategy:

27 matrix:

33 matrix:

28 include:

34 include:

29− - compiler: ghc-9.0.1

35+ - compiler: ghc-9.2.2

36+ compilerKind: ghc

37+ compilerVersion: 9.2.2

38+ setup-method: ghcup

30 allow-failure: true

39 allow-failure: true

31− - compiler: ghc-8.10.4

40+ - compiler: ghc-9.0.2

41+ compilerKind: ghc

42+ compilerVersion: 9.0.2

43+ setup-method: ghcup

44+ allow-failure: false

45+ - compiler: ghc-8.10.7

46+ compilerKind: ghc

47+ compilerVersion: 8.10.7

48+ setup-method: ghcup

32 allow-failure: false

49 allow-failure: false

33 - compiler: ghc-8.8.4

50 - compiler: ghc-8.8.4

51+ compilerKind: ghc

52+ compilerVersion: 8.8.4

53+ setup-method: hvr-ppa

34 allow-failure: false

54 allow-failure: false

35 - compiler: ghc-8.6.5

55 - compiler: ghc-8.6.5

56+ compilerKind: ghc

57+ compilerVersion: 8.6.5

58+ setup-method: hvr-ppa

36 allow-failure: false

59 allow-failure: false

37 - compiler: ghc-8.4.4

60 - compiler: ghc-8.4.4

61+ compilerKind: ghc

62+ compilerVersion: 8.4.4

63+ setup-method: hvr-ppa

38 allow-failure: false

64 allow-failure: false

39 fail-fast: false

65 fail-fast: false

40 steps:

66 steps:

41 - name: apt

67 - name: apt

42 run: |

68 run: |

43 apt-get update

69 apt-get update

44− apt-get install -y --no-install-recommends gnupg ca-certificates dirmngr curl git software-properties-common

70+ apt-get install -y --no-install-recommends gnupg ca-certificates dirmngr curl git software-properties-common libtinfo5

71+ if [ "${{ matrix.setup-method }}" = ghcup ]; then

72+ mkdir -p "$HOME/.ghcup/bin"

73+ curl -sL https://downloads.haskell.org/ghcup/0.1.17.5/x86_64-linux-ghcup-0.1.17.5 > "$HOME/.ghcup/bin/ghcup"

74+ chmod a+x "$HOME/.ghcup/bin/ghcup"

75+ "$HOME/.ghcup/bin/ghcup" install ghc "$HCVER"

76+ "$HOME/.ghcup/bin/ghcup" install cabal 3.6.2.0

77+ else

45− apt-add-repository -y 'ppa:hvr/ghc'

78+ apt-add-repository -y 'ppa:hvr/ghc'

46− apt-get update

79+ apt-get update

47− apt-get install -y $CC cabal-install-3.4

80+ apt-get install -y "$HCNAME"

81+ mkdir -p "$HOME/.ghcup/bin"

82+ curl -sL https://downloads.haskell.org/ghcup/0.1.17.5/x86_64-linux-ghcup-0.1.17.5 > "$HOME/.ghcup/bin/ghcup"

83+ chmod a+x "$HOME/.ghcup/bin/ghcup"

84+ "$HOME/.ghcup/bin/ghcup" install cabal 3.6.2.0

85+ fi

48 env:

86 env:

87+ HCKIND: ${{ matrix.compilerKind }}

49− CC: ${{ matrix.compiler }}

88+ HCNAME: ${{ matrix.compiler }}

89+ HCVER: ${{ matrix.compilerVersion }}

50 - name: Set PATH and environment variables

90 - name: Set PATH and environment variables

51 run: |

91 run: |

52 echo "$HOME/.cabal/bin" >> $GITHUB_PATH

92 echo "$HOME/.cabal/bin" >> $GITHUB_PATH

53− echo "LANG=C.UTF-8" >> $GITHUB_ENV

93+ echo "LANG=C.UTF-8" >> "$GITHUB_ENV"

54− echo "CABAL_DIR=$HOME/.cabal" >> $GITHUB_ENV

94+ echo "CABAL_DIR=$HOME/.cabal" >> "$GITHUB_ENV"

55− echo "CABAL_CONFIG=$HOME/.cabal/config" >> $GITHUB_ENV

95+ echo "CABAL_CONFIG=$HOME/.cabal/config" >> "$GITHUB_ENV"

56− HCDIR=$(echo "/opt/$CC" | sed 's/-/\//')

96+ HCDIR=/opt/$HCKIND/$HCVER

57− HCNAME=ghc

97+ if [ "${{ matrix.setup-method }}" = ghcup ]; then

58− HC=$HCDIR/bin/$HCNAME

98+ HC=$HOME/.ghcup/bin/$HCKIND-$HCVER

59− echo "HC=$HC" >> $GITHUB_ENV

99+ echo "HC=$HC" >> "$GITHUB_ENV"

60− echo "HCPKG=$HCDIR/bin/$HCNAME-pkg" >> $GITHUB_ENV

100+ echo "HCPKG=$HOME/.ghcup/bin/$HCKIND-pkg-$HCVER" >> "$GITHUB_ENV"

61− echo "HADDOCK=$HCDIR/bin/haddock" >> $GITHUB_ENV

101+ echo "HADDOCK=$HOME/.ghcup/bin/haddock-$HCVER" >> "$GITHUB_ENV"

102+ echo "CABAL=$HOME/.ghcup/bin/cabal-3.6.2.0 -vnormal+nowrap" >> "$GITHUB_ENV"

103+ else

104+ HC=$HCDIR/bin/$HCKIND

105+ echo "HC=$HC" >> "$GITHUB_ENV"

106+ echo "HCPKG=$HCDIR/bin/$HCKIND-pkg" >> "$GITHUB_ENV"

107+ echo "HADDOCK=$HCDIR/bin/haddock" >> "$GITHUB_ENV"

62− echo "CABAL=/opt/cabal/3.4/bin/cabal -vnormal+nowrap" >> $GITHUB_ENV

108+ echo "CABAL=$HOME/.ghcup/bin/cabal-3.6.2.0 -vnormal+nowrap" >> "$GITHUB_ENV"

109+ fi

110+

63 HCNUMVER=$(${HC} --numeric-version|perl -ne '/^(\d+)\.(\d+)\.(\d+)(\.(\d+))?$/; print(10000 * $1 + 100 * $2 + ($3 == 0 ? $5 != 1 : $3))')

111 HCNUMVER=$(${HC} --numeric-version|perl -ne '/^(\d+)\.(\d+)\.(\d+)(\.(\d+))?$/; print(10000 * $1 + 100 * $2 + ($3 == 0 ? $5 != 1 : $3))')

64− echo "HCNUMVER=$HCNUMVER" >> $GITHUB_ENV

112+ echo "HCNUMVER=$HCNUMVER" >> "$GITHUB_ENV"

65− echo "ARG_TESTS=--enable-tests" >> $GITHUB_ENV

113+ echo "ARG_TESTS=--enable-tests" >> "$GITHUB_ENV"

66− echo "ARG_BENCH=--enable-benchmarks" >> $GITHUB_ENV

114+ echo "ARG_BENCH=--enable-benchmarks" >> "$GITHUB_ENV"

67− echo "HEADHACKAGE=false" >> $GITHUB_ENV

115+ echo "HEADHACKAGE=false" >> "$GITHUB_ENV"

68− echo "ARG_COMPILER=--$HCNAME --with-compiler=$HC" >> $GITHUB_ENV

116+ echo "ARG_COMPILER=--$HCKIND --with-compiler=$HC" >> "$GITHUB_ENV"

69− echo "GHCJSARITH=0" >> $GITHUB_ENV

117+ echo "GHCJSARITH=0" >> "$GITHUB_ENV"

70 env:

118 env:

119+ HCKIND: ${{ matrix.compilerKind }}

71− CC: ${{ matrix.compiler }}

120+ HCNAME: ${{ matrix.compiler }}

121+ HCVER: ${{ matrix.compilerVersion }}

72 - name: env

122 - name: env

73 run: |

123 run: |

74 env

124 env

@@ -91,6 +141,10 @@ jobs:

91 repository hackage.haskell.org

141 repository hackage.haskell.org

92 url: http://hackage.haskell.org/

142 url: http://hackage.haskell.org/

93 EOF

143 EOF

144+ cat >> $CABAL_CONFIG <<EOF

145+ program-default-options

146+ ghc-options: $GHCJOBS +RTS -M3G -RTS

147+ EOF

94 cat $CABAL_CONFIG

148 cat $CABAL_CONFIG

95 - name: versions

149 - name: versions

96 run: |

150 run: |

@@ -129,7 +183,8 @@ jobs:

129 - name: generate cabal.project

183 - name: generate cabal.project

130 run: |

184 run: |

131 PKGDIR_openapi3="$(find "$GITHUB_WORKSPACE/unpacked" -maxdepth 1 -type d -regex '.*/openapi3-[0-9.]*')"

185 PKGDIR_openapi3="$(find "$GITHUB_WORKSPACE/unpacked" -maxdepth 1 -type d -regex '.*/openapi3-[0-9.]*')"

132− echo "PKGDIR_openapi3=${PKGDIR_openapi3}" >> $GITHUB_ENV

186+ echo "PKGDIR_openapi3=${PKGDIR_openapi3}" >> "$GITHUB_ENV"

187+ rm -f cabal.project cabal.project.local

133 touch cabal.project

188 touch cabal.project

134 touch cabal.project.local

189 touch cabal.project.local

135 echo "packages: ${PKGDIR_openapi3}" >> cabal.project

190 echo "packages: ${PKGDIR_openapi3}" >> cabal.project

@@ -180,7 +235,9 @@ jobs:

180 rm -f cabal.project.local

235 rm -f cabal.project.local

181 - name: constraint set aeson-2

236 - name: constraint set aeson-2

182 run: |

237 run: |

238+ $CABAL v2-build $ARG_COMPILER --disable-tests --disable-benchmarks --constraint='aeson >=2.0' --dependencies-only -j2 all

183 $CABAL v2-build $ARG_COMPILER --disable-tests --disable-benchmarks --constraint='aeson >=2.0' all

239 $CABAL v2-build $ARG_COMPILER --disable-tests --disable-benchmarks --constraint='aeson >=2.0' all

184 - name: constraint set aeson-1

240 - name: constraint set aeson-1

185 run: |

241 run: |

186− $CABAL v2-build $ARG_COMPILER --disable-tests --disable-benchmarks --constraint='aeson <2.0' all

242+ if [ $((HCNUMVER < 90200)) -ne 0 ] ; then $CABAL v2-build $ARG_COMPILER --disable-tests --disable-benchmarks --constraint='aeson <2.0' --dependencies-only -j2 all ; fi

243+ if [ $((HCNUMVER < 90200)) -ne 0 ] ; then $CABAL v2-build $ARG_COMPILER --disable-tests --disable-benchmarks --constraint='aeson <2.0' all ; fi

CHANGELOG.md+14

@@ -1,6 +1,20 @@

1Unreleased

2----------

4+3.2.2

5+-----

7+- Fix bug with infinite recursion in GToSchema [#37](https://github.com/biocad/openapi3/pull/37).

9+3.2.1

10+-----

11+

12+- Support aeson-2.0.3.

13+- Support GHC-9.2 (doctests still fail).

14+

15+3.2.0

16+-----

17+

4- Support aeson-2 [#34](https://github.com/biocad/openapi3/pull/34).

18- Support aeson-2 [#34](https://github.com/biocad/openapi3/pull/34).

5- Use `SecurityDefinitions` type for `_componentsSecuritySchemes`

19- Use `SecurityDefinitions` type for `_componentsSecuritySchemes`

6 [#32](https://github.com/biocad/openapi3/pull/32).

20 [#32](https://github.com/biocad/openapi3/pull/32).

cabal.haskell-ci+4

@@ -1,7 +1,11 @@

1branches: master

3+distribution: focal

4+allow-failures: >=9.2

3constraint-set aeson-1

6constraint-set aeson-1

4 constraints: aeson <2.0

7 constraints: aeson <2.0

8+ ghc: <9.2

6constraint-set aeson-2

10constraint-set aeson-2

7 constraints: aeson >=2.0

11 constraints: aeson >=2.0

openapi3.cabal+10−9

@@ -1,6 +1,6 @@

1cabal-version: >=1.10

2name: openapi3

3−version: 3.2.0

3+version: 3.2.2

5synopsis: OpenAPI 3.0 data model

6category: Web, Swagger, OpenApi

@@ -26,8 +26,9 @@ tested-with:

26 GHC ==8.4.4

27 || ==8.6.5

28 || ==8.8.4

29− || ==8.10.4

29+ || ==8.10.7

30− || ==9.0.1

30+ || ==9.0.2

31+ || ==9.2.2

32custom-setup

33custom-setup

33 setup-depends:

34 setup-depends:

@@ -60,16 +61,16 @@ library

61 -- GHC boot libraries

62 -- GHC boot libraries

62 build-depends:

63 build-depends:

63− base >=4.11.1.0 && <4.16

64+ base >=4.11.1.0 && <4.17

64− , bytestring >=0.10.8.2 && <0.11

65+ , bytestring >=0.10.8.2 && <0.12

65 , containers >=0.5.11.0 && <0.7

66 , containers >=0.5.11.0 && <0.7

66− , template-haskell >=2.13.0.0 && <2.18

67+ , template-haskell >=2.13.0.0 && <2.19

67− , time >=1.8.0.2 && <1.10

68+ , time >=1.8.0.2 && <1.14

68 , transformers >=0.5.5.0 && <0.6

69 , transformers >=0.5.5.0 && <0.6

70 build-depends:

71 build-depends:

71 mtl >=2.2.2 && <2.3

72 mtl >=2.2.2 && <2.3

72− , text >=1.2.3.1 && <1.3

73+ , text >=1.2.3.1 && <2.1

74 -- other dependencies

75 -- other dependencies

75 build-depends:

76 build-depends:

@@ -82,7 +83,7 @@ library

82 , hashable >=1.2.7.0 && <1.5

83 , hashable >=1.2.7.0 && <1.5

83 , http-media >=0.8.0.0 && <0.9

84 , http-media >=0.8.0.0 && <0.9

84 , insert-ordered-containers >=0.2.3 && <0.3

85 , insert-ordered-containers >=0.2.3 && <0.3

85− , lens >=4.16.1 && <5.1

86+ , lens >=4.16.1 && <5.2

86 , network >=2.6.3.5 && <3.2

87 , network >=2.6.3.5 && <3.2

87 , optics-core >=0.2 && <0.5

88 , optics-core >=0.2 && <0.5

88 , optics-th >=0.2 && <0.5

89 , optics-th >=0.2 && <0.5

src/Data/OpenApi/Internal/Schema.hs+6−3

@@ -945,10 +945,13 @@ instance (Selector s, GToSchema f, GToSchema (S1 s f)) => GToSchema (C1 c (S1 s

945 case schema ^. items of

946 Just (OpenApiItemsArray [_]) -> fieldSchema

947 _ -> do

948− declare defs

948+ -- We have to run recordSchema instead of just using its defs,

949+ -- since those can be recursive and will lead to infinite loop,

950+ -- see https://github.com/biocad/openapi3/pull/37

951+ NamedSchema _ schema' <- recordSchema

949− return (unnamed schema)

952+ return (unnamed schema')

950 where

953 where

951− (defs, NamedSchema _ schema) = runDeclare recordSchema mempty

954+ (_, NamedSchema _ schema) = runDeclare recordSchema mempty

952 recordSchema = gdeclareNamedSchema opts (Proxy :: Proxy (S1 s f)) s

955 recordSchema = gdeclareNamedSchema opts (Proxy :: Proxy (S1 s f)) s

953 fieldSchema = gdeclareNamedSchema opts (Proxy :: Proxy f) s

956 fieldSchema = gdeclareNamedSchema opts (Proxy :: Proxy f) s

954

957

stack.yaml+1−1

@@ -1,4 +1,4 @@

1−resolver: lts-16.8

1+resolver: lts-16.31

2packages:

3- '.'

4extra-deps:

test/Data/OpenApi/CommonTestTypes.hs+123

@@ -863,6 +863,129 @@ singleMaybeFieldSchemaJSON = [aesonQQ|

863}

864|]

865

866+-- ========================================================================

867+-- Natural Language (single field data with recursive fields)

868+-- ========================================================================

869+

870+data Predicate

871+ = PredicateNoun Noun

872+ | PredicateOmitted Omitted

873+ deriving (Eq, Ord, Read, Show, Generic)

874+instance ToJSON Predicate

875+instance ToSchema Predicate

876+

877+data Noun

878+ = Noun

879+ { nounSurf :: LangWord

880+ , nounModify :: [Modifier]

881+ }

882+ deriving (Eq, Ord, Read, Show, Generic)

883+instance ToJSON Noun

884+instance ToSchema Noun

885+

886+data LangWord

887+ = LangWord

888+ { langWordSurf :: String

889+ , langWordBase :: String

890+ }

891+ deriving (Eq, Ord, Read, Show, Generic)

892+instance ToJSON LangWord

893+instance ToSchema LangWord

894+

895+data Modifier

896+ = ModifierNoun Noun

897+ | ModifierOmitted Omitted

898+ deriving (Eq, Ord, Read, Show, Generic)

899+instance ToJSON Modifier

900+instance ToSchema Modifier

901+

902+newtype Omitted

903+ = Omitted

904+ { omittedModify :: [Modifier]

905+ }

906+ deriving (Eq, Ord, Read, Show, Generic)

907+instance ToJSON Omitted

908+instance ToSchema Omitted

909+

910+predicateSchemaDeclareJSON :: Value

911+predicateSchemaDeclareJSON = [aesonQQ|

912+[

913+ {

914+ "Predicate": {

915+ "oneOf": [

916+ {

917+ "properties": {

918+ "contents": { "$ref": "#/components/schemas/Noun" },

919+ "tag": { "enum": ["PredicateNoun"], "type": "string" }

920+ },

921+ "required": ["tag", "contents"],

922+ "type": "object"

923+ },

924+ {

925+ "properties": {

926+ "contents": { "$ref": "#/components/schemas/Omitted" },

927+ "tag": { "enum": ["PredicateOmitted"], "type": "string" }

928+ },

929+ "required": ["tag", "contents"],

930+ "type": "object"

931+ }

932+ ],

933+ "type": "object"

934+ },

935+ "Noun": {

936+ "properties": {

937+ "nounModify": {

938+ "items": { "$ref": "#/components/schemas/Modifier" },

939+ "type": "array"

940+ },

941+ "nounSurf": { "$ref": "#/components/schemas/LangWord" }

942+ },

943+ "required": ["nounSurf", "nounModify"],

944+ "type": "object"

945+ },

946+ "LangWord": {

947+ "properties": {

948+ "langWordBase": { "type": "string" },

949+ "langWordSurf": { "type": "string" }

950+ },

951+ "required": ["langWordSurf", "langWordBase"],

952+ "type": "object"

953+ },

954+ "Modifier": {

955+ "oneOf": [

956+ {

957+ "properties": {

958+ "contents": { "$ref": "#/components/schemas/Noun" },

959+ "tag": { "enum": ["ModifierNoun"], "type": "string" }

960+ },

961+ "required": ["tag", "contents"],

962+ "type": "object"

963+ },

964+ {

965+ "properties": {

966+ "contents": { "$ref": "#/components/schemas/Omitted" },

967+ "tag": { "enum": ["ModifierOmitted"], "type": "string" }

968+ },

969+ "required": ["tag", "contents"],

970+ "type": "object"

971+ }

972+ ],

973+ "type": "object"

974+ },

975+ "Omitted": {

976+ "properties": {

977+ "omittedModify": {

978+ "items": { "$ref": "#/components/schemas/Modifier" },

979+ "type": "array"

980+ }

981+ },

982+ "required": ["omittedModify"],

983+ "type": "object"

984+ }

985+ },

986+ { "$ref": "#/components/schemas/Predicate" }

987+]

988+|]

866

989

867-- ========================================================================

990-- ========================================================================

868-- TimeOfDay

991-- TimeOfDay

test/Data/OpenApi/Schema/ValidationSpec.hs+3

@@ -301,6 +301,8 @@ instance Eq ZonedTime where

301-- Arbitrary instance for Data.Aeson.Value

302-- ========================================================================

303

304+-- These instances were introduces in aeson upstream in 2.0.3

305+#if !MIN_VERSION_aeson(2,0,3)

304instance Arbitrary Value where

306instance Arbitrary Value where

305 -- Weights are almost random

307 -- Weights are almost random

306 -- Uniform oneof tends not to build complex objects cause of recursive call.

308 -- Uniform oneof tends not to build complex objects cause of recursive call.

@@ -319,3 +321,4 @@ instance Arbitrary v => Arbitrary (KeyMap.KeyMap v) where

319instance Arbitrary Key.Key where

321instance Arbitrary Key.Key where

320 arbitrary = Key.fromText <$> arbitrary

322 arbitrary = Key.fromText <$> arbitrary

321#endif

323#endif

324+#endif

test/Data/OpenApi/SchemaSpec.hs+4

@@ -59,6 +59,9 @@ checkInlinedRecSchema proxy js = inlineNonRecursiveSchemas defs s <=> js

59 where

60 (defs, s) = runDeclare (declareSchema proxy) mempty

62+checkToSchemaDeclare :: (HasCallStack, ToSchema a) => Proxy a -> Value -> Spec

63+checkToSchemaDeclare proxy js = runDeclare (declareSchemaRef proxy) mempty <=> js

64+

62spec :: Spec

65spec :: Spec

63spec = do

66spec = do

64 describe "Generic ToSchema" $ do

67 describe "Generic ToSchema" $ do

@@ -78,6 +81,7 @@ spec = do

78 context "UserId (non-record newtype)" $ checkToSchema (Proxy :: Proxy UserId) userIdSchemaJSON

81 context "UserId (non-record newtype)" $ checkToSchema (Proxy :: Proxy UserId) userIdSchemaJSON

79 context "Player (unary record)" $ checkToSchema (Proxy :: Proxy Player) playerSchemaJSON

82 context "Player (unary record)" $ checkToSchema (Proxy :: Proxy Player) playerSchemaJSON

80 context "SingleMaybeField (unary record with Maybe)" $ checkToSchema (Proxy :: Proxy SingleMaybeField) singleMaybeFieldSchemaJSON

83 context "SingleMaybeField (unary record with Maybe)" $ checkToSchema (Proxy :: Proxy SingleMaybeField) singleMaybeFieldSchemaJSON

84+ context "Natural Language (single field data with recursive fields)" $ checkToSchemaDeclare (Proxy :: Proxy Predicate) predicateSchemaDeclareJSON

81 context "Players (inlining schema)" $ checkToSchema (Proxy :: Proxy Players) playersSchemaJSON

85 context "Players (inlining schema)" $ checkToSchema (Proxy :: Proxy Players) playersSchemaJSON

82 context "MyRoseTree (datatypeNameModifier)" $ checkToSchema (Proxy :: Proxy MyRoseTree) myRoseTreeSchemaJSON

86 context "MyRoseTree (datatypeNameModifier)" $ checkToSchema (Proxy :: Proxy MyRoseTree) myRoseTreeSchemaJSON

83 context "MyRoseTree' (datatypeNameModifier)" $ checkToSchema (Proxy :: Proxy MyRoseTree') myRoseTreeSchemaJSON'

87 context "MyRoseTree' (datatypeNameModifier)" $ checkToSchema (Proxy :: Proxy MyRoseTree') myRoseTreeSchemaJSON'