Closed patrickcping closed 2 months ago
BUG FIX resource/davinci_flow: Fix validation to test whether the flow JSON contains multiple flows in one file. Only single flows are supported.
resource/davinci_flow
TF_ACC=1 go test -v -timeout 1800s -run ^TestAccResourceFlow_ github.com/pingidentity/terraform-provider-davinci/internal/service/davinci
Changes
BUG FIX
resource/davinci_flow: Fix validation to test whether the flow JSON contains multiple flows in one file. Only single flows are supported.Testing Shell Command
Testing Results
Expand Results
```shell === RUN TestAccResourceFlow_RemovalDrift === PAUSE TestAccResourceFlow_RemovalDrift === RUN TestAccResourceFlow_Basic_Clean === PAUSE TestAccResourceFlow_Basic_Clean === RUN TestAccResourceFlow_Basic_WithBootstrap === PAUSE TestAccResourceFlow_Basic_WithBootstrap === RUN TestAccResourceFlow_ConnectionSubflowLinks_WithMappingIDs_Clean === PAUSE TestAccResourceFlow_ConnectionSubflowLinks_WithMappingIDs_Clean === RUN TestAccResourceFlow_ConnectionSubflowLinks_WithMappingIDs_WithBootstrap === PAUSE TestAccResourceFlow_ConnectionSubflowLinks_WithMappingIDs_WithBootstrap === RUN TestAccResourceFlow_ConnectionSubflowLinks_WithoutMappingIDs_Clean === PAUSE TestAccResourceFlow_ConnectionSubflowLinks_WithoutMappingIDs_Clean === RUN TestAccResourceFlow_ConnectionSubflowLinks_WithoutMappingIDs_WithBootstrap === PAUSE TestAccResourceFlow_ConnectionSubflowLinks_WithoutMappingIDs_WithBootstrap === RUN TestAccResourceFlow_ComputeDifferences_ModifySettings === PAUSE TestAccResourceFlow_ComputeDifferences_ModifySettings === RUN TestAccResourceFlow_ComputeDifferences_CompanyId === PAUSE TestAccResourceFlow_ComputeDifferences_CompanyId === RUN TestAccResourceFlow_ComputeDifferences_Version === PAUSE TestAccResourceFlow_ComputeDifferences_Version === RUN TestAccResourceFlow_ComputeDifferences_Description === PAUSE TestAccResourceFlow_ComputeDifferences_Description === RUN TestAccResourceFlow_ComputeDifferences_AdditionalProperties === PAUSE TestAccResourceFlow_ComputeDifferences_AdditionalProperties === RUN TestAccResourceFlow_ComputeDifferences_NewNode === PAUSE TestAccResourceFlow_ComputeDifferences_NewNode === RUN TestAccResourceFlow_UnknownFlowString === PAUSE TestAccResourceFlow_UnknownFlowString === RUN TestAccResourceFlow_BrokenFlow === PAUSE TestAccResourceFlow_BrokenFlow === RUN TestAccResourceFlow_Variables_Clean === PAUSE TestAccResourceFlow_Variables_Clean === RUN TestAccResourceFlow_Variables_WithBootstrap === PAUSE TestAccResourceFlow_Variables_WithBootstrap === RUN TestAccResourceFlow_Variables_Invalid === PAUSE TestAccResourceFlow_Variables_Invalid === RUN TestAccResourceFlow_Variables_Overridden_Clean === PAUSE TestAccResourceFlow_Variables_Overridden_Clean === RUN TestAccResourceFlow_Variables_Overridden_WithBootstrap === PAUSE TestAccResourceFlow_Variables_Overridden_WithBootstrap === RUN TestAccResourceFlow_BadParameters === PAUSE TestAccResourceFlow_BadParameters === CONT TestAccResourceFlow_RemovalDrift === CONT TestAccResourceFlow_ComputeDifferences_AdditionalProperties === CONT TestAccResourceFlow_Variables_WithBootstrap === CONT TestAccResourceFlow_Variables_Overridden_WithBootstrap === CONT TestAccResourceFlow_BadParameters === CONT TestAccResourceFlow_Variables_Clean === CONT TestAccResourceFlow_Variables_Overridden_Clean === CONT TestAccResourceFlow_UnknownFlowString === CONT TestAccResourceFlow_ComputeDifferences_CompanyId === CONT TestAccResourceFlow_ComputeDifferences_NewNode === CONT TestAccResourceFlow_Variables_Invalid === CONT TestAccResourceFlow_ConnectionSubflowLinks_WithMappingIDs_Clean === CONT TestAccResourceFlow_ConnectionSubflowLinks_WithoutMappingIDs_Clean === CONT TestAccResourceFlow_BrokenFlow === CONT TestAccResourceFlow_ComputeDifferences_Version === CONT TestAccResourceFlow_ConnectionSubflowLinks_WithoutMappingIDs_WithBootstrap === NAME TestAccResourceFlow_Variables_Overridden_WithBootstrap resource_flow_test.go:1237: Skipping test with bootstrap config: https://github.com/pingidentity/terraform-provider-davinci/issues/266 --- SKIP: TestAccResourceFlow_Variables_Overridden_WithBootstrap (0.02s) === CONT TestAccResourceFlow_ConnectionSubflowLinks_WithMappingIDs_WithBootstrap === NAME TestAccResourceFlow_Variables_WithBootstrap resource_flow_test.go:1001: Skipping test with bootstrap config: https://github.com/pingidentity/terraform-provider-davinci/issues/266 --- SKIP: TestAccResourceFlow_Variables_WithBootstrap (0.02s) === CONT TestAccResourceFlow_ComputeDifferences_ModifySettings === NAME TestAccResourceFlow_ConnectionSubflowLinks_WithMappingIDs_WithBootstrap resource_flow_test.go:323: Skipping test with bootstrap config: https://github.com/pingidentity/terraform-provider-davinci/issues/266 --- SKIP: TestAccResourceFlow_ConnectionSubflowLinks_WithMappingIDs_WithBootstrap (0.01s) === CONT TestAccResourceFlow_Basic_WithBootstrap resource_flow_test.go:178: Skipping test with bootstrap config: https://github.com/pingidentity/terraform-provider-davinci/issues/266 --- SKIP: TestAccResourceFlow_Basic_WithBootstrap (0.01s) === CONT TestAccResourceFlow_ComputeDifferences_Description --- PASS: TestAccResourceFlow_Variables_Invalid (88.87s) === CONT TestAccResourceFlow_Basic_Clean --- PASS: TestAccResourceFlow_BrokenFlow (344.27s) --- PASS: TestAccResourceFlow_UnknownFlowString (437.90s) === NAME TestAccResourceFlow_RemovalDrift resource_flow_test.go:39: Skipping step 3/4 due to SkipFunc resource_flow_test.go:39: Skipping step 4/4 due to SkipFunc === NAME TestAccResourceFlow_Variables_Clean resource_flow_test.go:995: Skipping step 2/4 due to SkipFunc === NAME TestAccResourceFlow_Variables_Overridden_Clean resource_flow_test.go:1231: Skipping step 2/4 due to SkipFunc --- PASS: TestAccResourceFlow_RemovalDrift (475.37s) --- PASS: TestAccResourceFlow_BadParameters (487.94s) --- PASS: TestAccResourceFlow_ComputeDifferences_Version (511.85s) --- PASS: TestAccResourceFlow_ComputeDifferences_AdditionalProperties (517.60s) --- PASS: TestAccResourceFlow_ComputeDifferences_NewNode (533.80s) --- PASS: TestAccResourceFlow_ComputeDifferences_CompanyId (541.07s) --- PASS: TestAccResourceFlow_ComputeDifferences_ModifySettings (543.14s) --- PASS: TestAccResourceFlow_ComputeDifferences_Description (548.86s) --- PASS: TestAccResourceFlow_Variables_Clean (648.53s) --- PASS: TestAccResourceFlow_Variables_Overridden_Clean (685.42s) --- PASS: TestAccResourceFlow_ConnectionSubflowLinks_WithMappingIDs_Clean (905.42s) --- PASS: TestAccResourceFlow_ConnectionSubflowLinks_WithoutMappingIDs_Clean (910.03s) --- PASS: TestAccResourceFlow_Basic_Clean (843.75s) --- PASS: TestAccResourceFlow_ConnectionSubflowLinks_WithoutMappingIDs_WithBootstrap (1346.59s) PASS ok github.com/pingidentity/terraform-provider-davinci/internal/service/davinci 1347.539s ```