From: Steve Francia <steve.francia@gmail.com>
Date: Tue, 8 Dec 2015 22:38:26 +0000 (-0500)
Subject: Fix some of the "new site" tests
X-Git-Tag: v0.16~532
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=e29ede70;p=brevno-suite%2Fhugo

Fix some of the "new site" tests
---

diff --git a/commands/new.go b/commands/new.go
index 39454e6c..d465e83f 100644
--- a/commands/new.go
+++ b/commands/new.go
@@ -15,12 +15,12 @@ package commands
 
 import (
 	"bytes"
+	"errors"
 	"os"
 	"path/filepath"
 	"strings"
 	"time"
 
-	"errors"
 	"github.com/spf13/cobra"
 	"github.com/spf13/hugo/create"
 	"github.com/spf13/hugo/helpers"
diff --git a/commands/new_test.go b/commands/new_test.go
index 08d50f4c..ceed8262 100644
--- a/commands/new_test.go
+++ b/commands/new_test.go
@@ -1,12 +1,13 @@
 package commands
 
 import (
-	"github.com/spf13/afero"
-	"github.com/spf13/hugo/hugofs"
-	"github.com/stretchr/testify/assert"
 	"os"
 	"path/filepath"
 	"testing"
+
+	"github.com/spf13/afero"
+	"github.com/spf13/hugo/hugofs"
+	"github.com/stretchr/testify/assert"
 )
 
 // Issue #1133
@@ -41,10 +42,20 @@ func TestDoNewSite(t *testing.T) {
 	checkNewSiteInited(basepath, t)
 }
 
+func TestDoNewSite_noerror_base_exists_but_empty(t *testing.T) {
+	basepath := filepath.Join(os.TempDir(), "blog")
+	hugofs.SourceFs = new(afero.MemMapFs)
+	hugofs.SourceFs.MkdirAll(basepath, 777)
+	err := doNewSite(basepath, false)
+	assert.Nil(t, err)
+}
+
 func TestDoNewSite_error_base_exists(t *testing.T) {
 	basepath := filepath.Join(os.TempDir(), "blog")
 	hugofs.SourceFs = new(afero.MemMapFs)
 	hugofs.SourceFs.MkdirAll(basepath, 777)
+	hugofs.SourceFs.Create(filepath.Join(basepath, "foo"))
+	// Since the directory already exists and isn't empty, expect an error
 	err := doNewSite(basepath, false)
 	assert.NotNil(t, err)
 }
@@ -59,8 +70,7 @@ func TestDoNewSite_force_empty_dir(t *testing.T) {
 	checkNewSiteInited(basepath, t)
 }
 
-// TODO(spf13): Fix and re-enable this.
-func _TestDoNewSite_error_force_dir_inside_exists(t *testing.T) {
+func TestDoNewSite_error_force_dir_inside_exists(t *testing.T) {
 	basepath := filepath.Join(os.TempDir(), "blog")
 	contentPath := filepath.Join(basepath, "content")
 	hugofs.SourceFs = new(afero.MemMapFs)