[wip] Kadai2-fujiokayu

kadai1/fujiokayu/converter/converter.go → kadai2/fujiokayu/converter/converter.go

@@ -38,6 +38,8 @@ func dec(filePath string, decodeType string) (image.Image, error) {
 		return jpeg.Decode(reader)
 	case "png":
 		return png.Decode(reader)
+	default:
+		err = fmt.Errorf("undefined decodeType: %s", decodeType)
 	}
 	return nil, err
 }
@@ -57,13 +59,15 @@ func enc(filePath string, encodeType string, m image.Image) error {
 		return jpeg.Encode(writer, m, nil)
 	case "png":
 		return png.Encode(writer, m)
+	default:
+		err = fmt.Errorf("undefined encodeType: %s", encodeType)
 	}
-	return nil
+	return err
 }
 
 //Convert : convert decodeType file to encodeType file
 func Convert(filePath string, decodeType string, encodeType string) error {
-	fmt.Println("Converting", filePath)
+	fmt.Println("Converting: ", filePath)
 
 	m, err := dec(filePath, decodeType)
 	if err != nil {
@@ -75,6 +79,6 @@ func Convert(filePath string, decodeType string, encodeType string) error {
 		return err
 	}
 
-	fmt.Println("Converted", filePath)
+	fmt.Println("Converted : ", strings.TrimSuffix(filePath, path.Ext(filePath))+"."+encodeType)
 	return nil
 }

kadai2/fujiokayu/converter/converter_test.go

@@ -0,0 +1,65 @@
+package converter_test
+
+import (
+	"fmt"
+	"myConverter/converter"
+	"os"
+	"testing"
+)
+
+func fileAssertionHelper(t *testing.T, testCase string, file string) {
+	t.Helper()
+	// 変換後のファイルが存在するかチェック
+	info, err := os.Stat(file)
+	if err != nil {
+		t.Fatal(fmt.Errorf("Case (%s) Failed test: File not generated",
+			testCase))
+	}
+	//　変換後のファイルサイズが0バイトではないかチェック
+	if info.Size() <= 0 {
+		t.Fatal(fmt.Errorf("Case (%s) Failed test: Encoded file is invalid",
+			testCase))
+	}
+}
+
+func Test_Convert(t *testing.T) {
+	cases := []struct {
+		testCase   string
+		isErr      bool
+		decodeFile string
+		encodeFile string
+		decodeType string
+		encodeType string
+	}{
+		{testCase: "J2P", isErr: false, decodeFile: "../testdata/cat.jpg", encodeFile: "../testdata/cat.png", decodeType: "jpg", encodeType: "png"},
+		{testCase: "J2G", isErr: false, decodeFile: "../testdata/cat.jpg", encodeFile: "../testdata/cat.gif", decodeType: "jpg", encodeType: "gif"},
+		{testCase: "P2J", isErr: false, decodeFile: "../testdata/cat.png", encodeFile: "../testdata/cat.jpg", decodeType: "png", encodeType: "jpg"},
+		{testCase: "P2G", isErr: false, decodeFile: "../testdata/cat.png", encodeFile: "../testdata/cat.gif", decodeType: "png", encodeType: "gif"},
+		{testCase: "G2J", isErr: false, decodeFile: "../testdata/cat.gif", encodeFile: "../testdata/cat.jpg", decodeType: "gif", encodeType: "jpg"},
+		{testCase: "G2P", isErr: false, decodeFile: "../testdata/cat.gif", encodeFile: "../testdata/cat.png", decodeType: "gif", encodeType: "png"},
+		{testCase: "decode file not exists", isErr: true, decodeFile: "../testdata/error", encodeFile: "../testdata/cat.png", decodeType: "gif", encodeType: "png"},
+		{testCase: "undefined decode type", isErr: true, decodeFile: "../testdata/cat.jpg", encodeFile: "../testdata/cat.png", decodeType: "tiff", encodeType: "png"},
+		{testCase: "undefined encode type", isErr: true, decodeFile: "../testdata/cat.jpg", encodeFile: "../testdata/cat.png", decodeType: "jpg", encodeType: "tiff"},
+	}
+
+	for _, c := range cases {
+		c := c
+		t.Run(c.testCase, func(t *testing.T) {
+			err := converter.Convert(c.decodeFile, c.decodeType, c.encodeType)
+
+			if !c.isErr && err != nil {
+				t.Errorf(
+					"Case (%s) Failed test: Convert error.",
+					c.testCase)
+			}
+
+			fileAssertionHelper(t, c.testCase, c.encodeFile)
+
+			if c.isErr && err == nil {
+				t.Errorf(
+					"Case (%s) Failed test: Convert error.",
+					c.testCase)
+			}
+		})
+	}
+}