fix - multiple typos (#1644)

Co-authored-by: Miguel Lorek <mlorek@tenable.com>

README.md

@@ -94,7 +94,7 @@ The following environment variables need to be set for acceptance tests to run:
 * `SNOWFLAKE_PASSWORD` - Password for that user.
 * `SNOWFLAKE_ROLE` - Needs to be ACCOUNTADMIN or similar.
 * `SNOWFLAKE_REGION` - Default is us-west-2, set this if your snowflake account is in a different region.
-* `TEST_ACC` - to enable acc tests.
+* `TF_ACC` - to enable acc tests.
 
 e.g.
 
@@ -104,7 +104,7 @@ export SNOWFLAKE_USER=TEST_USER
 export SNOWFLAKE_PASSWORD=hunter2
 export SNOWFLAKE_ROLE=ACCOUNTADMIN
 export SNOWFLAKE_REGION=us-west-2
-export TEST_ACC=true
+export TF_ACC=true
 ```
 
 **Note: PRs for new resources will not be accepted without passing acceptance tests.**

pkg/datasources/external_functions.go

@@ -68,7 +68,7 @@ func ReadExternalFunctions(d *schema.ResourceData, meta interface{}) error {
 
 	currentExternalFunctions, err := snowflake.ListExternalFunctions(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] external functions in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/external_tables.go

@@ -63,7 +63,7 @@ func ReadExternalTables(d *schema.ResourceData, meta interface{}) error {
 
 	currentExternalTables, err := snowflake.ListExternalTables(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] external tables in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/file_formats.go

@@ -68,7 +68,7 @@ func ReadFileFormats(d *schema.ResourceData, meta interface{}) error {
 
 	currentFileFormats, err := snowflake.ListFileFormats(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] file formats in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/masking_policies.go

@@ -68,7 +68,7 @@ func ReadMaskingPolicies(d *schema.ResourceData, meta interface{}) error {
 
 	currentMaskingPolicies, err := snowflake.ListMaskingPolicies(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] masking policies in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/materialized_views.go

@@ -63,7 +63,7 @@ func ReadMaterializedViews(d *schema.ResourceData, meta interface{}) error {
 
 	currentViews, err := snowflake.ListMaterializedViews(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] materialized views in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/pipes.go

@@ -68,7 +68,7 @@ func ReadPipes(d *schema.ResourceData, meta interface{}) error {
 
 	currentPipes, err := snowflake.ListPipes(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] pipes in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/procedures.go

@@ -76,7 +76,7 @@ func ReadProcedures(d *schema.ResourceData, meta interface{}) error {
 
 	currentProcedures, err := snowflake.ListProcedures(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] procedures in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/resource_monitors.go

@@ -61,7 +61,7 @@ func ReadResourceMonitors(d *schema.ResourceData, meta interface{}) error {
 
 	currentResourceMonitors, err := snowflake.ListResourceMonitors(db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] no resource monitors found in account (%s)", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/row_access_policies.go

@@ -63,7 +63,7 @@ func ReadRowAccessPolicies(d *schema.ResourceData, meta interface{}) error {
 
 	currentRowAccessPolicies, err := snowflake.ListRowAccessPolicies(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] row access policy in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/schemas.go

@@ -54,7 +54,7 @@ func ReadSchemas(d *schema.ResourceData, meta interface{}) error {
 
 	currentSchemas, err := snowflake.ListSchemas(databaseName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] schemas in database (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/sequences.go

@@ -63,7 +63,7 @@ func ReadSequences(d *schema.ResourceData, meta interface{}) error {
 
 	currentSequences, err := snowflake.ListSequences(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] sequences in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/stages.go

@@ -68,7 +68,7 @@ func ReadStages(d *schema.ResourceData, meta interface{}) error {
 
 	currentStages, err := snowflake.ListStages(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] stages in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/storage_integrations.go

@@ -61,7 +61,7 @@ func ReadStorageIntegrations(d *schema.ResourceData, meta interface{}) error {
 
 	currentStorageIntegrations, err := snowflake.ListStorageIntegrations(db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] no storage integrations found in account (%s)", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/streams.go

@@ -68,7 +68,7 @@ func ReadStreams(d *schema.ResourceData, meta interface{}) error {
 
 	currentStreams, err := snowflake.ListStreams(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] streams in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/system_generate_scim_access_token.go

@@ -38,7 +38,7 @@ func ReadSystemGenerateSCIMAccessToken(d *schema.ResourceData, meta interface{})
 	row := snowflake.QueryRow(db, sel)
 	accessToken, err := snowflake.ScanSCIMAccessToken(row)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] system_generate_scim_access_token (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/system_get_aws_sns_iam_policy.go

@@ -39,7 +39,7 @@ func ReadSystemGetAWSSNSIAMPolicy(d *schema.ResourceData, meta interface{}) erro
 	row := snowflake.QueryRow(db, sel)
 	policy, err := snowflake.ScanAWSSNSIAMPolicy(row)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] system_get_aws_sns_iam_policy (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/system_get_privatelink_config.go

@@ -81,7 +81,7 @@ func ReadSystemGetPrivateLinkConfig(d *schema.ResourceData, meta interface{}) er
 	rawConfig, err := snowflake.ScanPrivateLinkConfig(row)
 
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Println("[DEBUG] system_get_privatelink_config not found")
 		d.SetId("")
 		return nil

pkg/datasources/system_get_snowflake_platform_info.go

@@ -40,7 +40,7 @@ func ReadSystemGetSnowflakePlatformInfo(d *schema.ResourceData, meta interface{}
 
 	acc, err := snowflake.ReadCurrentAccount(db)
 	if err != nil {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		d.SetId("")
 		log.Println("[DEBUG] current_account failed to decode")
 		return fmt.Errorf("error current_account err = %w", err)
@@ -50,7 +50,7 @@ func ReadSystemGetSnowflakePlatformInfo(d *schema.ResourceData, meta interface{}
 
 	rawInfo, err := snowflake.ScanSnowflakePlatformInfo(row)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Println("[DEBUG] system_get_snowflake_platform_info not found")
 		return fmt.Errorf("error system_get_snowflake_platform_info err = %w", err)
 	}

pkg/datasources/tables.go

@@ -63,7 +63,7 @@ func ReadTables(d *schema.ResourceData, meta interface{}) error {
 
 	currentTables, err := snowflake.ListTables(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] tables in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/tasks.go

@@ -68,7 +68,7 @@ func ReadTasks(d *schema.ResourceData, meta interface{}) error {
 
 	currentTasks, err := snowflake.ListTasks(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] tasks in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/users.go

@@ -120,7 +120,7 @@ func ReadUsers(d *schema.ResourceData, meta interface{}) error {
 
 	currentUsers, err := snowflake.ListUsers(userPattern, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] no users found in account (%s)", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/views.go

@@ -63,7 +63,7 @@ func ReadViews(d *schema.ResourceData, meta interface{}) error {
 
 	currentViews, err := snowflake.ListViews(databaseName, schemaName, db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] views in schema (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/datasources/warehouses.go

@@ -71,7 +71,7 @@ func ReadWarehouses(d *schema.ResourceData, meta interface{}) error {
 
 	currentWarehouses, err := snowflake.ListWarehouses(db)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] no warehouses found in account (%s)", d.Id())
 		d.SetId("")
 		return nil

pkg/resources/account.go

@@ -20,7 +20,7 @@ var accountSchema = map[string]*schema.Schema{
 		Required:     true,
 		Description:  "Specifies the identifier (i.e. name) for the account; must be unique within an organization, regardless of which Snowflake Region the account is in. In addition, the identifier must start with an alphabetic character and cannot contain spaces or special characters except for underscores (_). Note that if the account name includes underscores, features that do not accept account names with underscores (e.g. Okta SSO or SCIM) can reference a version of the account name that substitutes hyphens (-) for the underscores.",
 		ValidateFunc: snowflakeValidation.ValidateAccountIdentifier,
-		// Name is automatically uppercased by Snowflake
+		// Name is automatically uppercase by Snowflake
 		StateFunc: func(val interface{}) string {
 			return strings.ToUpper(val.(string))
 		},
@@ -30,7 +30,7 @@ var accountSchema = map[string]*schema.Schema{
 		Required:     true,
 		Description:  "Login name of the initial administrative user of the account. A new user is created in the new account with this name and password and granted the ACCOUNTADMIN role in the account. A login name can be any string consisting of letters, numbers, and underscores. Login names are always case-insensitive.",
 		ValidateFunc: snowflakeValidation.ValidateAdminName,
-		// We have no way of assuming a role into this account to change the admin user name so this has to be ForceNew even though its not ideal
+		// We have no way of assuming a role into this account to change the admin user name so this has to be ForceNew even though it's not ideal
 		ForceNew:              true,
 		DiffSuppressOnRefresh: true,
 		DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
@@ -48,7 +48,7 @@ var accountSchema = map[string]*schema.Schema{
 		Sensitive:    true,
 		Description:  "Password for the initial administrative user of the account. Optional if the `ADMIN_RSA_PUBLIC_KEY` parameter is specified. For more information about passwords in Snowflake, see [Snowflake-provided Password Policy](https://docs.snowflake.com/en/sql-reference/sql/create-account.html#:~:text=Snowflake%2Dprovided%20Password%20Policy).",
 		AtLeastOneOf: []string{"admin_password", "admin_rsa_public_key"},
-		// We have no way of assuming a role into this account to change the password so this has to be ForceNew even though its not ideal
+		// We have no way of assuming a role into this account to change the password so this has to be ForceNew even though it's not ideal
 		ForceNew:              true,
 		DiffSuppressOnRefresh: true,
 		DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
@@ -66,7 +66,7 @@ var accountSchema = map[string]*schema.Schema{
 		Sensitive:    true,
 		Description:  "Assigns a public key to the initial administrative user of the account in order to implement [key pair authentication](https://docs.snowflake.com/en/sql-reference/sql/create-account.html#:~:text=key%20pair%20authentication) for the user. Optional if the `ADMIN_PASSWORD` parameter is specified.",
 		AtLeastOneOf: []string{"admin_password", "admin_rsa_public_key"},
-		// We have no way of assuming a role into this account to change the admin rsa public key so this has to be ForceNew even though its not ideal
+		// We have no way of assuming a role into this account to change the admin rsa public key so this has to be ForceNew even though it's not ideal
 		ForceNew:              true,
 		DiffSuppressOnRefresh: true,
 		DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
@@ -83,7 +83,7 @@ var accountSchema = map[string]*schema.Schema{
 		Required:     true,
 		Description:  "Email address of the initial administrative user of the account. This email address is used to send any notifications about the account.",
 		ValidateFunc: snowflakeValidation.ValidateEmail,
-		// We have no way of assuming a role into this account to change the admin email so this has to be ForceNew even though its not ideal
+		// We have no way of assuming a role into this account to change the admin email so this has to be ForceNew even though it's not ideal
 		ForceNew:              true,
 		DiffSuppressOnRefresh: true,
 		DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
@@ -106,7 +106,7 @@ var accountSchema = map[string]*schema.Schema{
 		Type:        schema.TypeString,
 		Optional:    true,
 		Description: "First name of the initial administrative user of the account",
-		// We have no way of assuming a role into this account to change the admin first name so this has to be ForceNew even though its not ideal
+		// We have no way of assuming a role into this account to change the admin first name so this has to be ForceNew even though it's not ideal
 		ForceNew:              true,
 		DiffSuppressOnRefresh: true,
 		DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
@@ -122,7 +122,7 @@ var accountSchema = map[string]*schema.Schema{
 		Type:        schema.TypeString,
 		Optional:    true,
 		Description: "Last name of the initial administrative user of the account",
-		// We have no way of assuming a role into this account to change the admin last name so this has to be ForceNew even though its not ideal
+		// We have no way of assuming a role into this account to change the admin last name so this has to be ForceNew even though it's not ideal
 		ForceNew:              true,
 		DiffSuppressOnRefresh: true,
 		DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {
@@ -139,7 +139,7 @@ var accountSchema = map[string]*schema.Schema{
 		Optional:    true,
 		Default:     false,
 		Description: "Specifies whether the new user created to administer the account is forced to change their password upon first login into the account.",
-		// We have no way of assuming a role into this account to change the admin password policy so this has to be ForceNew even though its not ideal
+		// We have no way of assuming a role into this account to change the admin password policy so this has to be ForceNew even though it's not ideal
 		ForceNew:              true,
 		DiffSuppressOnRefresh: true,
 		DiffSuppressFunc: func(k, old, new string, d *schema.ResourceData) bool {

pkg/resources/database.go

@@ -227,7 +227,7 @@ func ReadDatabase(d *schema.ResourceData, meta interface{}) error {
 	database, err := snowflake.ScanDatabase(row)
 	if err != nil {
 		if errors.Is(err, sql.ErrNoRows) {
-			// If not found, mark resource to be removed from statefile during apply or refresh
+			// If not found, mark resource to be removed from state file during apply or refresh
 			log.Printf("[DEBUG] database (%s) not found", d.Id())
 			d.SetId("")
 			return nil

pkg/resources/file_format.go

@@ -539,7 +539,7 @@ func ReadFileFormat(d *schema.ResourceData, meta interface{}) error {
 
 	f, err := snowflake.ScanFileFormatShow(row)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] file_format (%s) not found", d.Id())
 		d.SetId("")
 		return nil

pkg/resources/function.go

@@ -283,7 +283,7 @@ func ReadFunction(d *schema.ResourceData, meta interface{}) error {
 	}
 	rows, err := snowflake.Query(db, stmt)
 	if err != nil && snowflake.IsResourceNotExistOrNotAuthorized(err.Error(), "Function") {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] function (%s) not found or we are not authorized.Err:\n%s", d.Id(), err.Error())
 		d.SetId("")
 		return nil
@@ -381,7 +381,7 @@ func ReadFunction(d *schema.ResourceData, meta interface{}) error {
 	q := funct.Show()
 	showRows, err := snowflake.Query(db, q)
 	if errors.Is(err, sql.ErrNoRows) {
-		// If not found, mark resource to be removed from statefile during apply or refresh
+		// If not found, mark resource to be removed from state file during apply or refresh
 		log.Printf("[DEBUG] function (%s) not found", d.Id())
 		d.SetId("")
 		return nil