fix(github_repository_file): delete error handling and codesigning

github/resource_github_repository_file.go

@@ -460,16 +460,14 @@
 
 	var branch string
 
-	message := fmt.Sprintf("Delete %s", file)
-
-	if commitMessage, hasCommitMessage := d.GetOk("commit_message"); hasCommitMessage {
-		message = commitMessage.(string)
+	opts, err := resourceGithubRepositoryFileOptions(d)
+	if err != nil {
+		return err
 	}
 
-	sha := d.Get("sha").(string)
-	opts := &github.RepositoryContentFileOptions{
-		Message: &message,
-		SHA:     &sha,
+	if *opts.Message == fmt.Sprintf("Add %s", file) {
+		m := fmt.Sprintf("Delete %s", file)
+		opts.Message = &m
 	}
 
 	if b, ok := d.GetOk("branch"); ok {
@@ -503,7 +501,7 @@
 		opts.Branch = &branch
 	}
 
 	_, _, err := client.Repositories.DeleteFile(ctx, owner, repo, file, opts)
 	return handleArchivedRepoDelete(err, "repository file", file, owner, repo)
 }
 

github/resource_github_repository_file_test.go

@@ -12,8 +12,6 @@
 
 func TestAccGithubRepositoryFile(t *testing.T) {
 
-	randomID := acctest.RandStringFromCharSet(5, acctest.CharSetAlphaNum)
-
 	t.Run("creates and manages files", func(t *testing.T) {
 
 		config := fmt.Sprintf(`
@@ -33,7 +31,7 @@
 				commit_author  = "Terraform User"
 				commit_email   = "terraform@example.com"
 			}
-		`, randomID)
+		`, acctest.RandStringFromCharSet(5, acctest.CharSetAlphaNum))
 		check := resource.ComposeTestCheckFunc(
 			resource.TestCheckResourceAttr(
 				"github_repository_file.test", "content",
@@ -111,7 +109,7 @@
 				commit_email        = "terraform@example.com"
 			}
 
-		`, randomID)
+		`, acctest.RandStringFromCharSet(5, acctest.CharSetAlphaNum))
 
 		check := resource.ComposeTestCheckFunc(
 			resource.TestCheckResourceAttr(
@@ -202,7 +200,7 @@
 				commit_author  = "Terraform User"
 				commit_email   = "terraform@example.com"
 			}
-		`, randomID)
+		`, acctest.RandStringFromCharSet(5, acctest.CharSetAlphaNum))
 
 		check := resource.ComposeTestCheckFunc(
 			resource.TestCheckResourceAttr(
@@ -280,7 +278,7 @@
 				commit_email      = "terraform@example.com"
 				autocreate_branch = false
 			}
-		`, randomID)
+		`, acctest.RandStringFromCharSet(5, acctest.CharSetAlphaNum))
 
 		check := resource.ComposeTestCheckFunc(
 			resource.TestCheckResourceAttr(
@@ -362,7 +360,7 @@
 				commit_author = "Terraform User"
 				commit_email = "terraform@example.com"
 			}
 		`, randomID)
 
 		archivedConfig := strings.Replace(config,
 			`auto_init = true`,
@@ -401,7 +399,7 @@
 								auto_init = true
 								archived = true
 							}
 						`, randomID),
 					},
 				},
 			})