Tweak debug handler for unconfigured users

Author: kytrinyx

config/config.go

@@ -109,6 +109,13 @@ func Read(file string) (*Config, error) {
 		return nil, err
 	}
 
+	if _, err = os.Stat(file); err != nil {
+		if os.IsNotExist(err) {
+			return New("", "", "")
+		}
+		return nil, err
+	}
+
 	f, err := os.Open(file)
 	if err != nil {
 		return nil, err

config/config_test.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"fmt"
 	"io/ioutil"
+	"strings"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -68,6 +69,16 @@ func TestExpandsTildeInExercismDirectory(t *testing.T) {
 	assert.NotContains(t, "~", expandedDir)
 }
 
+func TestReadNonexistantConfig(t *testing.T) {
+	c, err := Read("/no/such/config.json")
+	assert.NoError(t, err)
+	assert.Equal(t, c.APIKey, "")
+	assert.Equal(t, c.Hostname, "http://exercism.io")
+	if !strings.HasSuffix(c.Dir, "/exercism") {
+		t.Fatal("Default unconfigured config should use home dir")
+	}
+}
+
 func TestReadingWritingConfig(t *testing.T) {
 	tmpDir, err := ioutil.TempDir("", "")
 	filename := fmt.Sprintf("%s/%s", tmpDir, File)

handlers/debug.go

@@ -3,16 +3,18 @@ package handlers
 import (
 	"fmt"
 	"log"
+	"os"
 	"runtime"
 
 	"github.com/codegangsta/cli"
 	"github.com/exercism/cli/config"
 )
 
 func Debug(ctx *cli.Context) {
+	defer fmt.Printf("\nIf you are having any issues, please contact kytrinyx@exercism.io with this information.\n")
+
 	bail := func(err error) {
 		if err != nil {
-			fmt.Printf("\nIf you are having any issues, please contact kytrinyx@exercism.io with this information.\n")
 			log.Fatal(err)
 		}
 	}
@@ -26,14 +28,25 @@ func Debug(ctx *cli.Context) {
 	fmt.Printf("Home Dir: %s\n", dir)
 
 	file, err := config.FilePath(ctx.GlobalString("config"))
-	bail(err)
+	configured := true
+	if _, err = os.Stat(file); err != nil {
+		if os.IsNotExist(err) {
+			configured = false
+		} else {
+			bail(err)
+		}
+	}
 
 	c, err := config.Read(file)
 	bail(err)
 
-	fmt.Printf("Config file: %s\n", c.File())
+	if configured {
+		fmt.Printf("Config file: %s\n", c.File())
+		fmt.Printf("API Key: %s\n", c.APIKey)
+	} else {
+		fmt.Println("Config file: <not configured>")
+		fmt.Println("API Key: <not configured>")
+	}
 	fmt.Printf("API: %s\n", c.Hostname)
-	fmt.Printf("API Key: %s\n", c.APIKey)
 	fmt.Printf("Exercises Directory: %s\n", c.Dir)
-	fmt.Println()
 }

handlers/info.go

@@ -83,12 +83,6 @@ func main() {
 			},
 			Action: handlers.Configure,
 		},
-		{
-			Name:      "info",
-			ShortName: "i",
-			Usage:     "output config info",
-			Action:    handlers.Info,
-		},
 		{
 			Name:      "current",
 			ShortName: "c",