feat: Generate show outputs with mappers

Makefile

@@ -114,4 +114,10 @@ generate-docs-additional-files: ## generate docs additional files
 generate-docs-additional-files-check: generate-docs-additional-files ## check that docs additional files have been generated
 	git diff --exit-code -- examples/additional
 
+generate-show-output-schemas: ## Generate show output schemas with mappers
+	go generate ./pkg/schemas/generate.go
+
+clean-show-output-schemas: ## Clean generated show output schemas
+	rm -f ./pkg/schemas/*_gen.go
+
 .PHONY: build-local clean-generator-poc dev-setup dev-cleanup docs docs-check fmt fmt-check fumpt help install lint lint-fix mod mod-check pre-push pre-push-check sweep test test-acceptance uninstall-tf

docs/data-sources/databases.md

@@ -356,9 +356,9 @@ Read-Only:
 
 - `comment` (String)
 - `created_on` (String)
+- `dropped_on` (String)
 - `is_current` (Boolean)
 - `is_default` (Boolean)
-- `is_transient` (Boolean)
 - `kind` (String)
 - `name` (String)
 - `options` (String)
@@ -367,3 +367,4 @@ Read-Only:
 - `owner_role_type` (String)
 - `resource_group` (String)
 - `retention_time` (Number)
+- `transient` (Boolean)

go.mod

@@ -27,6 +27,7 @@ require (
 	github.com/stretchr/testify v1.8.4
 	github.com/youmark/pkcs8 v0.0.0-20201027041543-1326539a0a0a
 	golang.org/x/crypto v0.23.0
+	golang.org/x/exp v0.0.0-20231006140011-7918f672742d
 	golang.org/x/text v0.15.0
 )
 
@@ -110,7 +111,6 @@ require (
 	github.com/xo/terminfo v0.0.0-20210125001918-ca9a967f8778 // indirect
 	github.com/zclconf/go-cty v1.14.2 // indirect
 	github.com/zeebo/xxh3 v1.0.2 // indirect
-	golang.org/x/exp v0.0.0-20231006140011-7918f672742d // indirect
 	golang.org/x/mod v0.17.0 // indirect
 	golang.org/x/net v0.25.0 // indirect
 	golang.org/x/sync v0.7.0 // indirect

pkg/datasources/databases.go

@@ -135,6 +135,7 @@ func ReadDatabases(ctx context.Context, d *schema.ResourceData, meta any) diag.D
 	flattenedDatabases := make([]map[string]any, len(databases))
 
 	for i, database := range databases {
+		database := database
 		var databaseDescription []map[string]any
 		if d.Get("with_describe").(bool) {
 			describeResult, err := client.Databases.Describe(ctx, database.ID())
@@ -158,7 +159,7 @@ func ReadDatabases(ctx context.Context, d *schema.ResourceData, meta any) diag.D
 		}
 
 		flattenedDatabases[i] = map[string]any{
-			"show_output":     []map[string]any{schemas.DatabaseShowToSchema(database)},
+			"show_output":     []map[string]any{schemas.DatabaseToSchema(&database)},
 			"describe_output": databaseDescription,
 			"parameters":      databaseParameters,
 		}

pkg/datasources/databases_acceptance_test.go

@@ -40,7 +40,7 @@ func TestAcc_Databases_Complete(t *testing.T) {
 					resource.TestCheckResourceAttrSet("data.snowflake_databases.test", "databases.0.show_output.0.created_on"),
 					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.name", databaseName),
 					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.kind", "STANDARD"),
-					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.is_transient", "false"),
+					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.transient", "false"),
 					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.is_default", "false"),
 					// Commenting as this value depends on the currently used database, which is different when running as a single test and multiple tests (e.g., on CI)
 					// resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.is_current", "true"),
@@ -83,7 +83,7 @@ func TestAcc_Databases_Complete(t *testing.T) {
 					resource.TestCheckResourceAttrSet("data.snowflake_databases.test", "databases.0.show_output.0.created_on"),
 					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.name", databaseName),
 					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.kind", "STANDARD"),
-					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.is_transient", "false"),
+					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.transient", "false"),
 					resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.show_output.0.is_default", "false"),
 					// Commenting for the same reason as above
 					// resource.TestCheckResourceAttr("data.snowflake_databases.test", "databases.0.is_current", "false"),