From 75d7fd54c2758783f448626165062bc8f1c8ebf1 Mon Sep 17 00:00:00 2001 From: Scott Winkler Date: Mon, 7 Nov 2022 13:39:38 -0800 Subject: [PATCH] fix: format for go ci (#1349) --- pkg/datasources/roles.go | 3 ++- pkg/datasources/roles_acceptance_test.go | 6 +++--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/pkg/datasources/roles.go b/pkg/datasources/roles.go index 4d2d03b701..b3839264d5 100644 --- a/pkg/datasources/roles.go +++ b/pkg/datasources/roles.go @@ -2,9 +2,10 @@ package datasources import ( "database/sql" + "log" + "github.com/Snowflake-Labs/terraform-provider-snowflake/pkg/snowflake" "github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema" - "log" ) const ( diff --git a/pkg/datasources/roles_acceptance_test.go b/pkg/datasources/roles_acceptance_test.go index c21095df7c..b4f4921a32 100644 --- a/pkg/datasources/roles_acceptance_test.go +++ b/pkg/datasources/roles_acceptance_test.go @@ -26,15 +26,15 @@ func TestAcc_Roles(t *testing.T) { Check: resource.ComposeTestCheckFunc( resource.TestCheckResourceAttrSet("data.snowflake_roles.r", "roles.#"), resource.TestCheckResourceAttrSet("data.snowflake_roles.r", "roles.0.name"), - //resource.TestCheckTypeSetElemAttr("data.snowflake_roles.r", "roles.*", "name"), - //TODO SHOW ROLES output also includes built in roles, i.e. ACCOUNTADMIN, SYSADMIN, etc. + // resource.TestCheckTypeSetElemAttr("data.snowflake_roles.r", "roles.*", "name"), + // TODO SHOW ROLES output also includes built in roles, i.e. ACCOUNTADMIN, SYSADMIN, etc. ), }, { Config: rolesPattern(), Check: resource.ComposeTestCheckFunc( resource.TestCheckResourceAttrSet("data.snowflake_roles.r", "roles.#"), - //resource.TestCheckResourceAttrSet("data.snowflake_roles.r", "roles.0.name"), + // resource.TestCheckResourceAttrSet("data.snowflake_roles.r", "roles.0.name"), resource.TestCheckResourceAttr("data.snowflake_roles.r", "roles.#", "1"), resource.TestCheckResourceAttr("data.snowflake_roles.r", "roles.0.name", accountAdmin), ),