Create parent path of report file

Author: per1234

configuration/configuration.go

@@ -70,7 +70,6 @@ func Initialize(flags *pflag.FlagSet, projectPaths []string) error {
 
 	recursive, _ = flags.GetBool("recursive")
 
-	// TODO: validate path
 	reportFilePathString, _ := flags.GetString("report-file")
 	reportFilePath = paths.New(reportFilePathString)
 

result/result.go

@@ -225,7 +225,19 @@ func (results Type) jsonReportRaw() []byte {
 
 // WriteReport writes a report for all projects to the specified file.
 func (results Type) WriteReport() error {
-	err := configuration.ReportFilePath().WriteFile(results.jsonReportRaw())
+	reportFilePath := configuration.ReportFilePath()
+	reportFilePathParentExists, err := reportFilePath.Parent().ExistCheck()
+	if err != nil {
+		return fmt.Errorf("Problem processing --report-file flag value %v: %v", reportFilePath, err)
+	}
+	if !reportFilePathParentExists {
+		err = reportFilePath.Parent().MkdirAll()
+		if err != nil {
+			return fmt.Errorf("Error while creating report file path (%v): %v", reportFilePath.Parent(), err)
+		}
+	}
+
+	err = reportFilePath.WriteFile(results.jsonReportRaw())
 	if err != nil {
 		return fmt.Errorf("Error while writing report: %v", err)
 	}

result/result_test.go

@@ -38,9 +38,17 @@ func TestWriteReport(t *testing.T) {
 	reportFolderPath := paths.New(reportFolderPathString)
 
 	reportFilePath := reportFolderPath.Join("report-file.json")
+	_, err = reportFilePath.Create() // Create file using the report folder path.
+	require.Nil(t, err)
+
+	flags.Set("report-file", reportFilePath.Join("report-file.json").String())
+	assert.Nil(t, configuration.Initialize(flags, projectPaths))
+	assert.Error(t, Results.WriteReport(), "Parent folder creation should fail due to a collision with an existing file at that path")
+
+	reportFilePath = reportFolderPath.Join("report-file-subfolder", "report-file-subsubfolder", "report-file.json")
 	flags.Set("report-file", reportFilePath.String())
 	assert.Nil(t, configuration.Initialize(flags, projectPaths))
-	assert.NoError(t, Results.WriteReport())
+	assert.NoError(t, Results.WriteReport(), "Creation of multiple levels of parent folders")
 
 	reportFile, err := reportFilePath.Open()
 	require.Nil(t, err)