Releases: dikhan/terraform-provider-openapi
v0.21.1
What's new?
- Feature #164: Migrate OpenAPI plugin terraform core imports to Terraform Plugin SDK
- Bug Fix #172: provider name and version not parsed correctly when any part of version more than 1 digit
Changelog
d4eeccb pump up minor and bug version
71f1a04 pump up bug version number
dcf9b98 Merge pull request #170 from dikhan/fix-goreportcard-issues-2
340f389 Merge branch 'master' into fix-goreportcard-issues-2
d177dc6 Merge pull request #173 from dikhan/bugfix/allow-multiple-digit-version
54ede22 update test name
c92455d Merge branch 'master' into bugfix/allow-multiple-digit-version
df038d9 enable multiple digits in regex that checks binary name and extracts the provider name
13935ba do not use iterating object for assignment
2a9f4d8 handle error
b7e7c52 Merge pull request #168 from dikhan/fix-goreportcard-issues
cdeebf1 ineffassign
4214882 run go fmt simplify
8ed6a93 fix misspells
8df5c40 fix 'occured' misspells
5e6c2a1 fix 'reponse' misspells
2168001 fix 'Adn' misspells
f091831 fix 'shoudl' misspells
5950901 Merge pull request #167 from dikhan/making-repo-awesome
41c0da0 Merge branch 'making-repo-awesome' of github.com:dikhan/terraform-provider-openapi into making-repo-awesome
70e2ca3 remove hidden dot
e3ea0a2 Merge branch 'master' into making-repo-awesome
46fac18 push code cov report from ci build too
efc30f2 Merge pull request #166 from dikhan/making-repo-awesome
fe1c3a2 add CodeCov link and badge
8c521c1 add go doc and goreportcard links/badges
ae07d24 Merge pull request #165 from dikhan/feature/use-terraform-plugin-sdk
127011a go mod deps updates
3a4ce45 ran tf-sdk-migrator migrate
da82815 fix more issues related to the arrays of interfaces
a36fc77 fix issues after upgrading terraform to 0.12.10
v0.20.0
What's new?
- Added support for security definition x-terraform-refresh-token-url extension
Changelog
4abb5c8 pump minor version
8bed9e5 Merge pull request #163 from dikhan/feature/refresh-token-authenticator
25187b5 rename variable name to make govet happy
0c9360f Merge branch 'master' into feature/refresh-token-authenticator
9de6447 add tests for specAPIKeyHeaderRefreshTokenSecurityDefinition
03b105a go mod dep updates
8a8845b add docu for x-terraform-refresh-token-url extension
e5b7c04 remove x-terraform-refresh-token-url from example
7e9d49f add response status check
706b033 quick refactor in assertion
1d94875 removed unapplicatle Todo, that struct feels a lot like a Java class with all private methods and because it doesn't shows behaviors it shouldn't be "tested directly"
c8e7041 unittested apiRefreshTokenAuthenticator.prepareAuth()
58b1073 removed unapplicable todo
6ed5f89 add coverage for APIKeyHeaderRefreshTokenSecurityDefinition
9dad72c remove no longer applable TODO
2d2bdba cover use case where location is not supported
2e6ac2c add test for refresh token auth
26872dd add coverage for bearer query sec def
4c181f8 update dup test to cover bearer header security def
3c59631 Add TestIsRefreshTokenAuth
d5226c2 add TODOs as a reference to the missing bits and bobs
2afb6ac integrate refresh token api call to prepareAuth and then attach to the auth context the access token received
1806721 update also input arguments after specAPIKeyHeaderRefreshTokenSecurityDefinition contructor signature refactoring
b5650d9 quick struct rename to specAPIKeyHeaderRefreshTokenSecurityDefinition
8716336 create int test to cover refresh token authenticator
60631de update cons names to be more explicit
dc49dfd Merge branch 'master' into feature/refresh-token-authenticator
f91faa8 design proposal for refresh token authenticator
v0.19.1
What’s new?
- Bug Fix #159: immutable properties where the value has not changed not working properly.
- Bug Fix #160: readonly properties inside objects are included in the payload and sent to the API when a resource is updated.
- Bug Fix #161: immutable properties within objects and lists of objects are not working as expected.
Changelog
5422349 pumping up bug version number
12ff85c Merge pull request #162 from dikhan/various-bug-fix
0b544cf update readme with object and optional property child info
8f88674 go mod dep updates
8f031bb fix issue where immutable check was done after the list size was checked resulting into an error being thrown for props that are actually mutable
aff7008 Merge branch 'master' into various-bug-fix
9927fd3 update docu for x-terraform-immutable
6796f8d Merge pull request #158 from dikhan/bugfix/install-script
20631d8 fetch version from github instead of relying on local dir
49e700b Merge pull request #157 from childish-sambino/patch-1
a805164 update error message in int test
d7b8150 check if property (list of primitives) is immutable before checking for diffs
39151b6 fix bug where properties within objects were always treated as immutable
052034a add special case in float where if actual float is being compared then check directly the values (with no casting)
c3d409e add new test mutable list of objects where some properties are immutable and values are not updated
d976047 fix issue where json marshalling always converts int into float64
ab45e06 update go mod deps
ddb84b2 go vet name update
01ceecd fix bug where read remote was not considering parentIDs (only for subresources)
0e03254 fix put tests
f660861 add test when api returns more props than the ones specified in the schema definition
fc7d1bb cover when client returns an error
7c01af7 added coverage for immutable list size is updated
0f1c65a fix issue where objects that were not immutable but had properties immutable were being ignored
154fd61 fixed docu for isComputed and isOptionalComputed methods
645b73a add missing prop to the test
378839a check size of local list and remote before going through the list elems
c9efbae move readOnly checks inside validateImmutableProperty so recursive calls also ensure ignore readonly and parent props
7a142ac panic when marshalling fails
1d42689 add support for list with objects immutable checks
78b505f refactor code removing not needed extra method validateImmutableProperties
ce186ad add coverage for immutable object property is updated
5fb697b add support and coverate for immutable objects
388269e fix checkImmutableFields bugs
8e37718 reverting changes related to post/put having different methods to build the payload
bf877db push changes for createPayloadFromLocalStateData
62faaa0 refactoring on createPayloadFromLocalStateData
3ff73d4 add better logging
516a963 create string zeo prop
8bdbda2 use computed param not used before
c4ebe4f create helper function to build parent props
9a4773e go mod deps
0fbd841 fix issue with version
5b5db35 do not inject readonly properties into the payload
14d9d75 Add git dir to script installation instructions
v0.19.0
What’s new?
Changelog
b482df7 pump up minor version
e4dec0c Merge pull request #155 from dikhan/feature/data-source-instance
4325ef7 add documentation
1ba6f3b go mod dep updates
02c7778 remove default values when processing data source computed properties
3f02a7d go mod dep updates
f50fa49 add coverage for data source instance to fat unit test
5633465 add subresource test for DataSourceInstanceRead
c320918 refactor createTerraformProviderResourceMap to return both resources and data source instance
622654f handle createDataSourceSchema error returned
d071248 create createTerraformProviderDataSourceInstanceMap
9afcba7 add read support for data source instance
d4fdfbc create data source instance factory
f4a8dc2 add integration test for new data source type (instance)
v0.18.0
What’s new?
Changelog
c1505b6 Merge pull request #152 from dikhan/feature/data-source-support
328c37b Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
75e8e25 Update TestCreateTerraformProviderDataSourceMap to use dataSources on the specAnalyserStub
2876859 add info that subresource paths are also supported in data sources
f2b1886 Fix case in TestCreateProvider for createTerraformProviderDataSourceMap failure
0f95b4d Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
7c9c762 WIP add case to TestCreateProvider to exercise error when createTerraformProviderDataSourceMap fails
3a6736d go mod update
4cf3593 add timing telemetry and debug info
2504e34 Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
61e6129 add fat test for subresource data sources
ffca75e Expand TestOpenAPIProvider to check more data types
a4264f7 add TestDataSourceRead_Subresource
0a1ff7e add test TestCreateDataSourceSchema_Subresources
f53c126 use get property based on terraform name
96c7d94 do not override parent id property config if found
2750c71 update test to represent the schema input configuration
427e2af some more dry up
18b8389 Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
c524b82 create helper function to assert data source schema configs
62906c9 [wip] applying a filter on a nested struct while reading from a DataSource
70a0965 fixed case in which the datasource contains Elems which are primitive
e7f55ba [wip] all the nested object properties attributes are set to the correct values for the data source
b1dd16c [wip] adding nested object support
9d4b3e5 Merge branch 'master' into feature/data-source-support
03cf8a2 added todo to cover nested objects and make sure they are made computed too
0944432 update the schema to incluide an array of strings and assert the state contains the owners too
4696163 extend model def to include array of strings prop
6bad267 Merge pull request #151 from dikhan/update-version-to-0.18.0
714ee7d Update version to 0.18.0
d45cc04 Merge pull request #150 from dikhan/feature/data-source-support
51e7256 Add case to TestCreateProvider where createTerraformProviderResourceMap fails
e257db3 Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
d7e9ece make assertions on data source map too
bc534db Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
f2ffbf8 WIP add coverage to TestCreateProvider for createTerraformProviderResourceMap failure
f5fe35f Remove error check from GetTerraformCompliantDataSources - no longer returns an error
145cd1f remove empty line
ecdef72 add data source documentation
221d817 Add case to TestCreateProvider where specAnalyser has an error on the backendConfiguration
8e3125c create specific int test for data source
9a7c567 remove not needed assertions
c3833a1 Add case to TestCreateProvider where specAnalyser has an error
01b5009 Add integration test for data source
9f79f4f add todo with missing coverage
7d14132 remove redundant test (perhaps copy pasta mistake)
ece4e31 TestOpenAPIProvider now includes the Integration without mock of DataSources
dec17f7 removed unapplicable todo
15936e8 covered happy and crappy paths for createTerraformProviderDataSourceMap
104047e implemented logic remaining in createTerraformProviderDataSourceMap and completed its "integration" test
8d2468e [wip] the integration test is now running properly
267e900 [wip] attempt to test the createProvider() methods widthou that much stubbing this technically will increase confidence in the code as a real SPECV2Analyzer is being use to test prod code
7ef9606 Added tests for TestGetTerraformCompliantDataSources() and changed its signature to return NO error
97f9389 Remove invalid specSchemaDefinition testcase from TestFilterMatch
faa8d18 mod dep
794a927 fix lint issue
09908a7 fix failing test after making list operation rely on root level get response operation config
5db8c13 Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
d55cb54 Remove error checking from filterMatch and validateInput - already caught when data source constructed
1550a54 fix issue with test assertion
ef9be96 add more todos
bc056da add implementation to GetTerraformCompliantDataSources
1496b34 create constructor for newSpecV2DataSource
9fb9a03 use List operation when figuring out configuration for list operation
bdfe95e Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
69309e6 update test to check expected struct hearirchy for data sources
d998691 add todo as reminder to add a test for subresoruce data source too
0fb66ab add logic to check if a path is data source compliant
840ed38 Add TestDataSourceRead_Fails_Because_Bad_Status_Code
7df603f Add TestCreateTerraformDataSource
d32ba16 skeleton design to integrate data source functionality
1563b4f Add TestDataSourceRead_Fails_Because_List_Operation_Returns_Err for List returning error
0958c3d [wip] covering crappy paths in read()
32f78d1 added the ID to the data source and improved assertions
5664259 [wip] extracted setStateID() from resource_factory.go to common.go as it will be used in data_source_factory.go too
d6280d1 [wip] introduced new todos as annotation on code coverage and correctness
4954c34 removed unapplicable or old todos
8627def Fixed filter for float property
6becdd7 [wip] polishing and removing unapplicable or solved todos
75074a3 Update TestCreateTerraformDataSourceSchema to check for an expectedError
5027d68 Update createTerraformDataSourceSchema to return an error
e63fc0d Remove todo to get and return the data source schema in createTerraformDataSourceSchema
02a719c Update createTerraformDataSourceSchema to return the created data source schema
d1a6920 [wip] on createTerraformDataSourceSchema
9589386 both sad and happy path covered for createDataSourceSchema()
cd28208 both sad and happy path covered for createDataSourceSchema()
c3a5172 [wip] happy path for createDataSourceSchema()
e032097 add support for createDataSourceSchema
ffae702 Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
efdfb7b add test for TestCreateTerraformDataSourceSchema that documents what is the expectation for data source schema configuration
f4ba8af Merge branch 'feature/data-source-support' of github.com:dikhan/terraform-provider-openapi into feature/data-source-support
3249311 Add happy path testcases to TestFilterMatch for int, float, bool properties
e020425 add some info why the test fails
7bf7c12 flesh out todo comment
262e44a add ref todo comment with missing logic to make the read operation work fine
d52c9ce integrate updateStateWithPayloadData so state gets updated with the filtered result
26aa11e fix assertion issue and only assert state when error is nil
fd7e365 add test case for validate input fails
d97d477 remove no longer applicable TODO
6ca9596 add test coverage for:
79a96b1 Add TestFilterMatch to validate filterMatch on dataSourceFactory
07534ed Add error property to specStubResource so getResourceSchema() can return an error
a046e70 refactor TestCheckHTTPStatusCode and TestResponseContainsExpectedStatus
6af5c24 move common functionality for datasource and resources to commons
1ad586d add filterMatch logic
636e1ff make validateInput return list of filters with name and value
d45f505 add test for TestDataSourceRead
750aa62 remove redundant err check and update error message returned
078546e update response payload to be array of items
5b6e347 enable list testing adding responseListPayload to clientOpenAPIStub
230c94e dry up label set up
56fab15 add test coverage for isPrimitiveProperty
22246e5 remove n/a todo comment
267e978 add check for primitive and values size
78cfe37 refactor test to use test driven table
c666a98 use const for schema property names to avoid bugs due to typos (reused names across)
c5c40da add some test for validateInput
5428780 add dataSourceFactory struct
90dd66a extend ClientAPI with new operation to list instances
d5d26c6 move common functionality for resource and data source factories to commons
v0.17.0
What's New?
Changelog
5a6e1e7 Merge pull request #149 from dikhan/bumping-ver-0.17.0
759d05b added details in the release.md doc
91a8ef0 bump version to 0.17.0
e4b109a Merge pull request #148 from dikhan/investigation/nested-objects-readonly-props-diffing
75351e0 Merge branch 'investigation/nested-objects-readonly-props-diffing' of github.com:dikhan/terraform-provider-openapi into investigation/nested-objects-readonly-props-diffing
fea7dde fix broken link
f0c1c65 Merge branch 'investigation/nested-objects-readonly-props-diffing' of github.com:dikhan/terraform-provider-openapi into investigation/nested-objects-readonly-props-diffing
cf2b1f6 added a new test case to cover //unstested path
e5b20c2 add documentation about
0db47f5 Dry-ed test case
34a181f remove //untested comments for tested code
53cf534 add a test for invalid terraform names for getPropertyPayload()
61851ab add TestGetPropertyBasedOnTerraformName()
de2b288 replace git.apache.org dependency to github
c2ece77 Remove obsolete //untested marker
cde9294 Merge remote-tracking branch 'origin/investigation/nested-objects-readonly-props-diffing' into investigation/nested-objects-readonly-props-diffing
6e81694 merge master
accb1b7 Merge remote-tracking branch 'origin/investigation/nested-objects-readonly-props-diffing' into investigation/nested-objects-readonly-props-diffing
71f66d4 if a datavalue is null, then test the return behavior
6983dee Mark untested logic in updated function
7f54cfa Fix "swagger schema definition property that is not of type 'object'", which was passing for the wrong reason
17804e3 De-convey TestSpecSchemaDefinitionIsPropertyWithNestedObjects
65e7e38 Simplify TestSpecSchemaDefinitionIsPropertyWithNestedObjects
021567f Introduce "Given a swagger schema definition property that is not of type object" in TestSpecSchemaDefinitionIsPropertyWithNestedObjects
c0c1c13 Simplify TestIsLegacyComplexObjectExtensionEnabled
d6bbee2 tested objectSchema, err := s.terraformObjectSchema()
2bec2cb Merge remote-tracking branch 'origin/investigation/nested-objects-readonly-props-diffing' into investigation/nested-objects-readonly-props-diffing
ede3a27 De-Convey TestIsLegacyComplexObjectExtensionEnabled
40a707a Merge branch 'investigation/nested-objects-readonly-props-diffing' of github.com:dikhan/terraform-provider-openapi into investigation/nested-objects-readonly-props-diffing
c0a2d21 Mark untested logic in new and updated functions
d568b4f quick convey test name update to make it more explicit difference bw simple and complex objects
de84c8f Clean up obsolete TODO; introduce new tests for getPropertyPayload.
85e6ab8 update go mod deps
8174b68 remove unnecessary error returned from isPropertyWithNestedObjects
a9aae5c Merge branch 'investigation/nested-objects-readonly-props-diffing' of github.com:dikhan/terraform-provider-openapi into investigation/nested-objects-readonly-props-diffing
d3e90b3 added test coverage for complex object types in TestConvertPayloadToLocalStateDataValue
36c7624 handled error when a resource miss the specSchemaDefinition
d5beb47 Merge branch 'investigation/nested-objects-readonly-props-diffing' of github.com:dikhan/terraform-provider-openapi into investigation/nested-objects-readonly-props-diffing
3106508 Add a test scenario where a nested object contains a complex object
9ce8475 Introduce Test_shouldUseLegacyTerraformSDKBlockApproachForComplexObjects
aa754e9 Add a test scenario where a nested object contains a complex object
36a53b7 fix merge conflicts - replaced name to isLegacyComplexObjectExtensionEnabled
8be8c38 Merge branch 'investigation/nested-objects-readonly-props-diffing' of github.com:dikhan/terraform-provider-openapi into investigation/nested-objects-readonly-props-diffing
cb28dc2 add test coverage for shouldEnableLegacyComplexObjectBlockConfiguration
a82a318 simplified function name
e23c975 Merge remote-tracking branch 'origin/investigation/nested-objects-readonly-props-diffing' into investigation/nested-objects-readonly-props-diffing
afde7ce simplified return boolean values
591caff remove no longer applicable todo commnet
15d6fa6 add test coverage for isBoolExtensionEnabled
bd2a77d centralise bool check logic into its own method isBoolExtensionEnabled
ad216da remove unneeded fmt print line
9b4bd4b add test coverage for x-terraform-complex-object-legacy-config extension
1a99932 add TODOs with remaining missing coverage
31d3398 centralised all the logic related to legacy block approach for complex objecs under one method (shouldUseLegacyTerraformSDKBlockApproachForComplexObjects)
e79b4eb update extension name
7b59209 delete no longer needed test file
d52bf67 add two option on how to handle object types without diffs
c8e7d6c better docs
ac7facf That's my take on this issues, check my TODOs
fcc823c add some bits to the test to reproduce the error
cdc7361 add readonly YAML to the grey box testing
a11201c add new unit test to prove that nested objects with readonly props are configured properly in terraform schema
v0.16.0
What's new?
- Added support for nested structs #108. More info here.
- Added support for sub-resource #129. More info here.
- [Bug Fix] #140 : When there is naming collision between resource, the provider behaves indeterminately
- [Bug Fix] #137: For resources that are multi-region but the host is not correct the error returned is being ignored resulting into a resource configured with the wrong configuration.
- [Bug Fix] #130: Fixed erratic behaviour where given a swagger file with no global http schemes defined the open api provider would default to http. Instead, the provider should work purely based on the openapi doc, and if it's missing http schemes containing supported protocols like http/https then the provider now fails to let the service provider know about it.
NOTE: This release includes a non backwards compatible change for property names that had a number and were not terraform name compatible following the snake case pattern, were translated automatically to a compatible name but not very user friendly. For instance a property with name ipv4 would be translated automatically to ipv_4. The property is now translated to ipv4. If you have state files that contain resources with properties where names where automatically translated like the example aforementioned, you can ensure backwards compatibility before updating to use the new plugin version using the 'x-terraform-field-name extension' and setting the current name generated. This will ensure backwards compatibility before updating to the new version of the plugin. Refer to the x-terraform-field-name extension docs for more info.
Changelog
7dba62c Merge pull request #146 from dikhan/bugfix/remove-unneded-checl
22ac510 Updates to reflect including the version with preferred names
469a39c fix bug where the resource name used to match against the regex was the prefer one instead of the uri one
63a1c53 remove not needed check that was preventing resources from being registered if the definitinos were externalised
49efeb6 pump up release version
e1fa811 Fix typo in error message (#145)
00f5cec Merge pull request #144 from dikhan/bug/improve-collision-doc
3477b29 Improve doc
b10991b Improve doc
3bb796a Feature/handle resource name collisions (#141)
a8aeb99 Merge pull request #138 from dikhan/techdebt-addMoreTestsToGetTerraformComplianceResources
fa2f2e6 kick travis
25125f2 Merge branch 'master' into techdebt-addMoreTestsToGetTerraformComplianceResources
656bb80 add another state to the regex to avoid failing tests
fa6d881 Merge pull request #139 from dikhan/feature/add-int-test-for-timeout-user-override
5d65f55 due to the nature of the async call the error returned could contain the last state or not
b538581 update name for TestAccLB_CreateDoesNotTimeoutDueToUserSpecifyingAShorterTimeout to be shorter rather than longer
f59ae29 remove non needed step
b98468e quick typo fix
52d0fd3 fix instance path reference
26b83e5 update docs with more specific info on how the timeouts are treated and handled
f1f2816 remove non needed step
579e0ec add coverage for use case where user tries to set timeout for operation that does not support timeout configuration
e6d49fc add int test that shows how users can override timeouts from the config
313f787 rename TestNewSpecV2Resource to TestNewSpecV2ResourceWithConfig
285d8eb go mod version
1f3351e refactor newSpecV2ResourceWithRegion to check if region is empty
e527e0a go mod version updates
72f3245 remove unneeded extra branch that was adding more complexity just to log the regionHost
6330d56 use ShouldBeNil instead of ShouldEqual, nil
51018af add to cover that schemaDefinitions is not nil
887467b add test to cover case where resource creation fails
b863013 add test to TestGetTerraformCompliantResources for a multi-region resource with an empty string for region (x-terraform-resource-regions-serviceProviderName)
d894784 add test to TestSpecV2ResourceGetHost to cover a SpecV2Resource with a multi-region host with region specified
5e85763 add test to TestSpecV2ResourceGetHost to cover a SpecV2Resource with a multi-region host but no region specified
57555b3 refactor TestSpecV2ResourceGetHost test case with missing POST operation to pass in nil POST
374b51b refactor getResourceOverrideHost to return empty string for resource with nil rootPathItemPost
3be2fbd add test to TestGetResourceOverrideHost for resource with nil POST operation
1dba8d4 add test to TestSpecV2ResourceGetHost for resource with no POST operation
319eaa5 add test to TestSpecV2ResourceGetHost when override host (extTfResourceURL) isn't specified
c6f6d64 start adding test coverage for SpecV2Resource getHost()
b896e7b integrate createMultiRegionResources into GetTerraformCompliantResources function
f6a2277 add todo comment to add missing tests for getHost()
8db96c4 update isMultiRegionResource to return a list of strings instead of a map
30e98c7 return error when path is empty
4f96e6f expect error when path is empty
df87321 bug-fix: update GetMultiRegionHost to return error rather than nil
c96a416 add new test for createMultiRegionResources to cover when a list of regions containing a empty string is passed in
d8f5b4f add test coverage to createMultiRegionResources to cover when resourceRootPath is empty
a18708c attempt to refactor createMultiRegionResources logic
2e0c1ee add test coverage to check isMultiRegionResource
b14fe99 Merge pull request #135 from dikhan/feature/parent_preferred_names2
c008b91 add logging in case an error is returned from buildResourceNameFromPath
92ef5f0 add coverage TestGetTerraformCompliantResources to include multi region resource
6805523 fixed todo updating test to considered preferred parent name
5846a83 Remove unused resourceVersionRegex
82b90ff Mark untested new lines
47c799d fix go vet issues
c2279a1 updated docs with more info regarding how resource names are constructed
6dad1d0 update how_to_subresources doc
d8a2052 ignore error returned by Compile
a8413b7 Add more test coverage to TestBuildResourceName (negative scenario)
f28d8b5 Add more test coverage to TestBuildResourceNameFromPath
666161c Remove redundant tests in TestBuildResourceName and added coverage for different use cases
18db11e Merge branch 'feature/parent_preferred_names2' of github.com:dikhan/terraform-provider-openapi into feature/parent_preferred_names2
e35cdb0 Add coverage for when paths is nil
fcfb5b8 remove unused const
4cfc8bd mod version updates
aa128bb update integration tests failing to use preferred cdn name instead
556a924 add unit test for buildResourceNameFromPath
1d549e0 add coverage for paths using trailing slashes
dbd80d2 add test covering parent path containing more than just the version and the resource name
4903237 remove todo comment as it's no longer applicable (fixed)
9545628 fix failing test in getSchemaDefinition unit test
07338cd WIP on TestGetSchemaDefinition when we have preferred names
1d672f0 fixed different test related to the existences of preferred names. All tests are green
861ce7e fixed todo (and regex) in TestParenResourceInfo
5ff0f1c update resourceParentNameRegex to handle base paths too
6566185 Accept paths in the newSpecV2Resource constructor
47bc854 Update TestBuildResourceName to incporate preferred name case
6746d34 Update TestBuildResourceName to include paths
de78a96 depend on map[string]spec.PathItem{} instead of the whole swagger object
37be48d reuse logic on how resource name is build into buildResourceNameFromPath
20040e7 add test to cover parent preferred names use case
562ef3f Merge pull request #131 from dikhan/feature/default-to-https
789a25f Update mods
ce10d4e Merge branch 'master' into feature/default-to-https
887dbea Handle error returned by getHTTPScheme
883641a Remove old TODO
22aa1cd Replace httpSchemes arg to newStubBackendConfiguration with httpScheme
25505e4 Replace httpSchemes in specStubBackendConfiguration with httpScheme
78425c3 Rename getHTTPSchemes to getHTTPScheme
db3835f Remove getHTTPSchemes from interface and impls
818e182 Introduce getHTTPSchemes2 on stub; cut over from getHTTPSchemes
d53684d Simplify test by merging single line Thens.
5f34eab Simplify test by inlining single-use temp var.
c5be074 Add "mix of schemes configured including supported ones with https"
f8776c9 Improve readbility of test by removing unneeded empty property declarations
cc43e51 Replace conditional error assertion with general one
00988ac Add missing verification that no error is raised if none is expected
5852f07 make error message more helpful
6596cea add TODOs comments
bc55868 remove not needed check
e9816be Updated getHTTPSchemes2 signature to return one element instead of a list
c0d8691 fix buggy logic in test where errors were not being caught properly
0a8be12 create getHTTPSchemes2
6bb7973 refactor TestGetHTTPSchemes to use table driven test
2b5959e Merge pull request #133 from dikhan/feature/subresource-validation
102c4ac update docs to include new validation behaviour
38c0f5a Replace last couple occurrences of subresource with parentResource.
1c26cb5 Revert "Rename subResource to parentResourceInfo"
dc44a95 Reduce noise around conditional assertion of expected errors
30427dd Rename subResource to parentResourceInfo
c23f8a5 Rename isSubResource to getParentResourceInfo
41d992f Rename isSubResource to getParentResourceInfo; subResourceInfo to parentResourceInfo
549f6f7 Improve documentation value of tests by removing unneeded clutter.
eca4b4e...
v0.15.0
What's new?
- Added support for exposing plugin version in the open api plugin configuration. More info at plugin_version service schema configuration.
- Fixed bug where wrong service configurations that were not the plugin under execution would affect the successful execution of the openapi plugin due to validation errors. More info in this commit
Changelog
c2bc9b5 update version number to 0.15.0
bcac8a3 Merge pull request #125 from dikhan/feature/expose-plugin-version-in-config-file
31df6dc Merge branch 'feature/expose-plugin-version-in-config-file' of github.com:dikhan/terraform-provider-openapi into feature/expose-plugin-version-in-config-file
bc517da updating plugin_version schema config property doc
64ebf86 cleanup stale filehandles in TestServiceConfigV1Validate()
36a00a2 Add an additional testcase for TestServiceConfigV1Validate()
0891781 fix docs typo
8d07133 reverting latest commits on mod that introduced wrong dependencies
7fe1fac fixed go.mod: it was showing problems with Goadesign package, again x2
50c9e28 fixed go.mod: it was showing problems with Goadesign package, again
45284e5 update documentation about the new config property exposed plugin_version
1c24f9d add unit tests to cover use case where plugin version config does not match the version of the ope api being executed
0abc512 integrated validation on service level before returning
bb2b126 remove no longer needed tests (moved to service config level)
33becb8 update marshal tests covering use case of plugin version being defined
5ca306a validation on URL has been moved to the service config itself
98dbbf1 update stub with new methods to implement
0f759aa added unit tests for new functionliy added to ServiceConfigV1
e99dbfe add validation method on plugin config service level
v0.14.0
What's new?
This release adds support for Terraform 0.12 SDK. Terraform 0.12 is not backwards compatible, this means that configurations created prior to this version may no longer be compatible and will have to be updated to match the new HCL syntax released with Terraform v0.12. In order to assist with transitioning to the newer version, the following guide upgrading to terraform 0.12 can be used as a reference to understand how to make older terraform configuration compliant with the new version of Terraform v0.12.
Changelog
f2bde8c Merge pull request #123 from dikhan/feature/update-to-0.12.0-not-rc
fbe3b53 update golang image to latest and make apt-0get install non interactive
2499545 update example container to use terraform 0.12.0 version
55b672d update readme terraform version with v
bb66300 use latest terraform 0.12.0 release version instead of release candidate (rc)
48c0460 Merge pull request #121 from dikhan/feature/upgrade-example-configs-to-0.12upgrade
e230c45 update readme with a link to how to migrate to 0.12
7ec17b0 create docs for migrating to Terraform 0.12
25219fc update example config to terraform 0.12 configuration compatible syntax
1161ea1 Merge pull request #117 from dikhan/update-go
80f22d8 Merge branch 'master' into update-go
12312f3 Merge pull request #119 from dikhan/pump-version-0140
e65cf27 pump version 0140
aeb675a Update docs to reflect go version dependency change to 1.12.4
b9693a2 Merge pull request #116 from dikhan/feature/upgrade-to-terraform-sdk-0.12
a063076 update terraform version in the readme
a212fed TODO comment as reminder
7b52e1d updated tf file to describe arrays objects using block sections instead
a3a87b6 fix issue with how object bools are treated by the new sdk
f0ef546 fix integration tests after terraform sdk upgrade to 0.12
9d5cf43 Upgrade terraform sdk version to 0.12 following Terraform instructions
v0.13.0
What's New?
- Added support for External $ref, see: #93
Changelog
581c1dd Merge pull request #113 from dikhan/version/pump-0.13.0
d6c1a27 pumping version to 0.13.0
c89ab86 Merge pull request #110 from dikhan/feature/ref-first-level-expansion-support
ee057e9 removing as already documented in the official doc
b4f43c1 remove no longer needed TestGetPayloadDefName
1d494bf remove unreachable code
4b69616 use newSpecAnalyserV2 instead of stubbed specV2Analyser
8590252 this test is covered by the constructor
1c88ae0 wip: add expansion to initAPISpecAnalyser and fix tests
3768911 fix TestValidateRootPath test
82fced8 refactor getBodyParameterBodySchema() and add tests
5567820 Remove getResourcePayloadSchemaRef() as it's no longer needed
00962fb add test coverage for getBodyParameterBodySchema
5ef69a6 remove getResourcePayloadSchemaDef as it is no longer applicable
89969ad Merge branch 'feature/ref-first-level-expansion-support' of https://github.com/dikhan/terraform-provider-openapi into feature/ref-first-level-expansion-support
980e4f9 Remove unnecessary error check.
4802db9 Remove redundant nil check
e413fb1 Add back accidentally deleted integration tests
f48c6e4 update docs with info about external $ref
efca209 Add TODOs for missing tests
448ed21 clone specific goa version that has cors package
b5971dc go fmt
a8d9f5b extract out the sample json blob into createExternalSwaggerContent
4646045 update Test_getBodyParameterBodySchema unit test to assert on actual errors
edf8653 return specific errors when getBodyParameterBodySchema is not able to find the post schema
f3886c7 DRYing up a little test duplication
445ec7e Restore accidentally deleted integration tests
b3a4fa2 Remove spurious TODOs
08d35b7 Finish Test_getBodyParameterBodySchema
3ebc4d5 Introduce Test_getBodyParameterBodySchema
1d17422 fix tab issues
699e20a Add cases of bad URLs
ed84ccb add unit test covering file containing a schema ref that points to a non schema definition
022ff69 fix convey description to match the expected assertion
0c49488 add test coverage for a swagger file containing a schema ref that is empty
e38a227 add test coverage for ref to a definition is wrongly formatted (no empty string)
c3ed08b add test coverage for ref to a definition that does not exists
3e197be fix tab issue
c0a02ca openapi_v2_spec_analyzer_test: TestNewSpecAnalyserV2() with a bad remote ref
642667b openapi_v2_spec_analyzer_test: Test GetTerraformCompliantResources() with an externally hosted resource
d65ba4f Merge branch 'feature/ref-first-level-expansion-support' of https://github.com/dikhan/terraform-provider-openapi into feature/ref-first-level-expansion-support
b0923db circular ref to itself
834e313 openapi_v2_spec_analyzer_test: Fix the ref expansion over http
2401922 wip: add tests for external refs hosted with a http server
395b271 DRY up some duplication
bc41e60 Fix a compile issue.
244fe9a Introduce several new test cases
d5d3d54 Introduce go.mod, go.sum
2c8735e Improve spec label
f89b72c Prevent TestOpenAPIProvider from reaching out to a real domain.
225031e update Status definitions now that the schema is expanded
c8e93d4 add unit tests
4cee2bd expand schema
7879930 go fmt
3b944e9 move initAPISpecFile to test utils
8a1414d fix bug where just ReadOnly properties would not have been marked as computed
24dc2be move initAPISpecFile to test utils
d5a723f fix version of github.com/go-openapi/spec to v0.19.0