Merge pull request #20 from kaypee90/add-config-override-check

feat: add prompt before overriding existing configuration

cmd/depbot/main.go

@@ -6,7 +6,7 @@ import (
 	"os"
 )
 
-const version = "0.2.3"
+const version = "0.3.0"
 
 func displayAppVersion() {
 	fmt.Printf("Depbot %s\n", version)
@@ -21,6 +21,12 @@ func launchApplicaton() {
 
 	// get dependabot configuration details
 	for {
+		if dependabotFileExists() {
+			if getConfigurationOverrideConfirmation() == NO {
+				os.Exit(0)
+			}
+		}
+
 		packageEcosystem := getPackageEcosystem()
 		directory := getDirectory()
 		interval := getInterval()

cmd/depbot/prompts.go

@@ -9,6 +9,21 @@ import (
 const YES = "yes"
 const NO = "no"
 
+func getConfigurationOverrideConfirmation() string {
+	prompt := promptui.Select{
+		Label: "Dependabot is currently configured for this project. Would you like to override the current configuration?",
+		Items: []string{YES, NO},
+	}
+
+	_, result, err := prompt.Run()
+
+	if err != nil {
+		panic(err)
+	}
+
+	return result
+}
+
 func getPackageEcosystem() string {
 	prompt := promptui.Select{
 		Label: "Select Package Ecosystem",

cmd/depbot/utils.go

@@ -10,6 +10,7 @@ import (
 const (
 	dependabotFileName = "dependabot.yml"
 	githubDirectory    = ".github"
+	pathSeperator      = "/"
 )
 
 func printIntroductoryText() {
@@ -33,6 +34,16 @@ func writeBytesToFile(fileName string, data []byte) {
 	log.Printf("Dependabot config created successfully!")
 }
 
+func fileExists(filename string) bool {
+	_, err := os.Stat(filename)
+	if err != nil {
+		if os.IsNotExist(err) {
+			return false
+		}
+	}
+	return true
+}
+
 func createDirectoryIfItDoesNotExist(dir string) (hasCreatedNewDir bool, err error) {
 	if _, err := os.Stat(dir); os.IsNotExist(err) {
 		err := os.Mkdir(dir, 0755)
@@ -48,12 +59,9 @@ func createDirectoryIfItDoesNotExist(dir string) (hasCreatedNewDir bool, err err
 	return false, nil
 }
 
-func createConfigurationFile(fileName string, destinationDir string, data []byte, skipCreatingDir bool) {
-	fullFilePath := fileName
-
+func createConfigurationFile(fullFilePath string, destinationDir string, data []byte, skipCreatingDir bool) {
 	if !skipCreatingDir {
 		createDirectoryIfItDoesNotExist(destinationDir)
-		fullFilePath = destinationDir + "/" + fileName
 	}
 
 	writeBytesToFile(fullFilePath, data)
@@ -68,7 +76,27 @@ func getWorkingDirectory() string {
 	return filepath.Base(currentDirectory)
 }
 
+func workingDirectoryIsGithub() bool {
+	return getWorkingDirectory() == githubDirectory
+}
+
+func getDependabotYamlFilePath(fileName string, destinationDir string) string {
+	fullFilePath := fileName
+
+	if !workingDirectoryIsGithub() {
+		fullFilePath = destinationDir + pathSeperator + fileName
+	}
+
+	return fullFilePath
+}
+
 func createDependabotYamlFile(data []byte) {
-	skipCreatingDir := getWorkingDirectory() == githubDirectory
-	createConfigurationFile(dependabotFileName, githubDirectory, data, skipCreatingDir)
+	skipCreatingDir := workingDirectoryIsGithub()
+	fullFilePath := getDependabotYamlFilePath(dependabotFileName, githubDirectory)
+	createConfigurationFile(fullFilePath, githubDirectory, data, skipCreatingDir)
+}
+
+func dependabotFileExists() bool {
+	dependabotYmlFilePath := getDependabotYamlFilePath(dependabotFileName, githubDirectory)
+	return fileExists(dependabotYmlFilePath)
 }

cmd/depbot/utils_test.go

@@ -31,16 +31,18 @@ func TestCreateDirectoryIfItDoesNotExistIfDirectoryAlreadyExist(t *testing.T) {
 
 func TestCreateConfigurationFile(t *testing.T) {
 	const fileName = "dependabot.yml"
+	const destinationDir = ".github"
 
-	defer os.RemoveAll(testDir)
+	defer os.RemoveAll(destinationDir)
 
-	_ = os.Mkdir(testDir, 0755)
+	_ = os.Mkdir(destinationDir, 0755)
 
 	data := []byte("version: 2\nupdates:\n  - package-ecosystem: \"npm\"")
+	filePath := getDependabotYamlFilePath(fileName, destinationDir)
 
-	createConfigurationFile(fileName, testDir, data, false)
+	createConfigurationFile(filePath, destinationDir, data, false)
 
-	_, err := os.Stat(testDir + "/" + fileName)
+	_, err := os.Stat(destinationDir + "/" + fileName)
 
 	// Exists
 	assert.NilError(t, err)
@@ -51,3 +53,27 @@ func TestGetWorkingDirectory(t *testing.T) {
 
 	assert.Equal(t, dirName, "depbot")
 }
+
+func TestFileExistsWhenFileIsNotCreated(t *testing.T) {
+	defer os.RemoveAll(testDir)
+
+	hasFile := fileExists(testDir + "/dependabot.yml")
+
+	assert.Equal(t, hasFile, false)
+}
+
+func TestFileExistsWhenFileAlrieadyExists(t *testing.T) {
+	filePath := testDir + "/dependabot.yml"
+
+	// Create the file
+	file, err := os.Create(filePath)
+	if err != nil {
+		return
+	}
+	defer file.Close()
+	defer os.RemoveAll(testDir)
+
+	hasFile := fileExists(filePath)
+
+	assert.Equal(t, hasFile, true)
+}