docs: remove if exist from example. since default terraform behavior …

…is to fail when the resource already exists. instead i updated the import section to include an example using idempotent snowsql clauses.

Makefile

@@ -3,7 +3,7 @@ REGISTRY=registry.terraform.io
 HOSTNAME=aidanmelen
 NAME=snowsql
 BINARY=terraform-provider-${NAME}
-VERSION=1.3.2
+VERSION=1.3.3
 OS_ARCH=darwin_amd64
 
 .PHONY: help

docs/guides/snowflake-provider-migration-guide.md

@@ -18,7 +18,7 @@ terraform {
   required_providers {
     snowsql = {
       source  = "aidanmelen/snowsql"
-      version = ">= 1.3.2"
+      version = ">= 1.3.3"
     }
   }
 }

docs/resources/exec.md

@@ -2,7 +2,7 @@
 page_title: "snowsql_exec Resource - terraform-provider-snowsql"
 subcategory: ""
 description: |-
-
+  
 ---
 
 # snowsql_exec (Resource)
@@ -16,11 +16,11 @@ This basic example shows how to manage an arbitrary Snowflake object.
 ```terraform
 resource "snowsql_exec" "role" {
   create {
-    statements = "CREATE ROLE IF NOT EXISTS my_role"
+    statements = "CREATE ROLE my_role"
   }
 
   delete {
-    statements = "DROP ROLE IF EXISTS my_role"
+    statements = "DROP ROLE my_role"
   }
 }
 ```
@@ -38,15 +38,15 @@ This resource allows you to execute arbitrary SnowSQL queries and use the result
 ```terraform
 resource "snowsql_exec" "role" {
   create {
-    statements = "CREATE ROLE IF NOT EXISTS my_role"
+    statements = "CREATE ROLE my_role"
   }
 
   read {
     statements = "SHOW ROLES LIKE 'my_role'"
   }
 
   delete {
-    statements = "DROP ROLE IF EXISTS my_role"
+    statements = "DROP ROLE my_role"
   }
 }
 
@@ -86,7 +86,7 @@ Execute the update statements as an in-place Terraform change by adding or modif
 ```terraform
 resource "snowsql_exec" "role" {
   create {
-    statements = "CREATE ROLE IF NOT EXISTS my_role"
+    statements = "CREATE ROLE my_role"
   }
 
   read {
@@ -95,11 +95,11 @@ resource "snowsql_exec" "role" {
 
   # uncomment to update role in-place
   # update {
-  #   statements = "ALTER ROLE IF EXISTS my_role SET COMMENT = 'updated with terraform'"
+  #   statements = "ALTER ROLE my_role SET COMMENT = 'updated with terraform'"
   # }
 
   delete {
-    statements = "DROP ROLE IF EXISTS my_role"
+    statements = "DROP ROLE my_role"
   }
 }
 ```
@@ -109,19 +109,19 @@ resource "snowsql_exec" "role" {
 ```terraform
 resource "snowsql_exec" "role" {
   create {
-    statements = "CREATE ROLE IF NOT EXISTS my_role"
+    statements = "CREATE ROLE my_role"
   }
 
   read {
     statements = "SHOW ROLES LIKE 'my_role'"
   }
 
   update {
-    statements = "ALTER ROLE IF EXISTS my_role SET COMMENT = 'updated with terraform'"
+    statements = "ALTER ROLE my_role SET COMMENT = 'updated with terraform'"
   }
 
   delete {
-    statements = "DROP ROLE IF EXISTS my_role"
+    statements = "DROP ROLE my_role"
   }
 }
 ```
@@ -210,7 +210,7 @@ resource "snowsql_exec" "function" {
     statements = <<-EOT
       USE SCHEMA ${snowflake_database.database.name}.PUBLIC;
 
-      CREATE OR REPLACE FUNCTION js_factorial(d double)
+      CREATE FUNCTION js_factorial(d double)
         RETURNS double
         LANGUAGE JAVASCRIPT
         STRICT
@@ -236,10 +236,7 @@ resource "snowsql_exec" "function" {
   }
 
   delete {
-    statements = <<-EOT
-      DROP FUNCTION IF EXISTS
-        ${snowflake_database.database.name}.PUBLIC.js_factorial(FLOAT);
-    EOT
+    statements = "DROP FUNCTION ${snowflake_database.database.name}.PUBLIC.js_factorial(FLOAT);"
   }
 }
 ```
@@ -272,3 +269,17 @@ Import is supported using the following syntax:
 ```shell
 terraform import snowsql_exec.name name
 ```
+
+However, since the default method only imports the object without controlling the read/refresh logic, it is recommended to use snowflake clauses to interact with existing objects, such as:
+
+```terraform
+resource "snowsql_exec" "role" {
+  create {
+    statements = "CREATE ROLE my_role"
+  }
+
+  delete {
+    statements = "DROP ROLE my_role"
+  }
+}
+```

examples/data-sources/query/provider.tf

@@ -8,7 +8,7 @@ terraform {
     }
     snowsql = {
       source  = "aidanmelen/snowsql"
-      version = ">= 1.3.2"
+      version = ">= 1.3.3"
     }
     random = ">= 2.1"
   }

examples/resources/exec/basic/.create-delete.tf.docs

@@ -1,9 +1,9 @@
 resource "snowsql_exec" "role" {
   create {
-    statements = "CREATE ROLE IF NOT EXISTS my_role"
+    statements = "CREATE ROLE my_role"
   }
 
   delete {
-    statements = "DROP ROLE IF EXISTS my_role"
+    statements = "DROP ROLE my_role"
   }
 }

examples/resources/exec/basic/.import.tf.docs

@@ -0,0 +1,9 @@
+resource "snowsql_exec" "role" {
+  create {
+    statements = "CREATE ROLE my_role"
+  }
+
+  delete {
+    statements = "DROP ROLE my_role"
+  }
+}

examples/resources/exec/basic/.read.tf.docs

@@ -1,14 +1,14 @@
 resource "snowsql_exec" "role" {
   create {
-    statements = "CREATE ROLE IF NOT EXISTS my_role"
+    statements = "CREATE ROLE my_role"
   }
 
   read {
     statements = "SHOW ROLES LIKE 'my_role'"
   }
 
   delete {
-    statements = "DROP ROLE IF EXISTS my_role"
+    statements = "DROP ROLE my_role"
   }
 }
 

examples/resources/exec/basic/.update.tf.docs

@@ -1,17 +1,17 @@
 resource "snowsql_exec" "role" {
   create {
-    statements = "CREATE ROLE IF NOT EXISTS my_role"
+    statements = "CREATE ROLE my_role"
   }
 
   read {
     statements = "SHOW ROLES LIKE 'my_role'"
   }
 
   update {
-    statements = "ALTER ROLE IF EXISTS my_role SET COMMENT = 'updated with terraform'"
+    statements = "ALTER ROLE my_role SET COMMENT = 'updated with terraform'"
   }
 
   delete {
-    statements = "DROP ROLE IF EXISTS my_role"
+    statements = "DROP ROLE my_role"
   }
 }

examples/resources/exec/basic/main.tf

@@ -1,6 +1,6 @@
 resource "snowsql_exec" "role" {
   create {
-    statements = "CREATE ROLE IF NOT EXISTS my_role"
+    statements = "CREATE ROLE my_role"
   }
 
   read {
@@ -9,10 +9,10 @@ resource "snowsql_exec" "role" {
 
   # uncomment to update role in-place
   # update {
-  #   statements = "ALTER ROLE IF EXISTS my_role SET COMMENT = 'updated with terraform'"
+  #   statements = "ALTER ROLE my_role SET COMMENT = 'updated with terraform'"
   # }
 
   delete {
-    statements = "DROP ROLE IF EXISTS my_role"
+    statements = "DROP ROLE my_role"
   }
 }

examples/resources/exec/basic/provider.tf

@@ -8,7 +8,7 @@ terraform {
     }
     snowsql = {
       source  = "aidanmelen/snowsql"
-      version = ">= 1.3.2"
+      version = ">= 1.3.3"
     }
     random = ">= 2.1"
   }

examples/resources/exec/complete/function.tf

@@ -5,7 +5,7 @@ resource "snowsql_exec" "function" {
     statements = <<-EOT
       USE SCHEMA ${snowflake_database.database.name}.PUBLIC;
 
-      CREATE OR REPLACE FUNCTION js_factorial(d double)
+      CREATE FUNCTION js_factorial(d double)
         RETURNS double
         LANGUAGE JAVASCRIPT
         STRICT
@@ -31,9 +31,6 @@ resource "snowsql_exec" "function" {
   }
 
   delete {
-    statements = <<-EOT
-      DROP FUNCTION IF EXISTS
-        ${snowflake_database.database.name}.PUBLIC.js_factorial(FLOAT);
-    EOT
+    statements = "DROP FUNCTION ${snowflake_database.database.name}.PUBLIC.js_factorial(FLOAT);"
   }
 }

examples/resources/exec/complete/provider.tf

@@ -8,7 +8,7 @@ terraform {
     }
     snowsql = {
       source  = "aidanmelen/snowsql"
-      version = ">= 1.3.2"
+      version = ">= 1.3.3"
     }
     random = ">= 2.1"
   }

templates/guides/snowflake-provider-migration-guide.md

@@ -18,7 +18,7 @@ terraform {
   required_providers {
     snowsql = {
       source  = "aidanmelen/snowsql"
-      version = ">= 1.3.2"
+      version = ">= 1.3.3"
     }
   }
 }

templates/resources/exec.md.tmpl

@@ -106,3 +106,7 @@ Import is supported using the following syntax:
 ```shell
 terraform import snowsql_exec.name name
 ```
+
+However, since the default method only imports the object without controlling the read/refresh logic, it is recommended to use snowflake clauses to interact with existing objects, such as:
+
+{{ tffile "examples/resources/exec/basic/.import.tf.docs" }}