Merge pull request #21 from bruin-data/feature/remove-bruin-section

fix empty materialization and add test

pkg/pipeline/materializer.go

@@ -17,7 +17,7 @@ type Materializer struct {
 func (m *Materializer) Render(asset *Asset, query string) (string, error) {
 	mat := asset.Materialization
 	if mat.Type == MaterializationTypeNone {
-		return query, nil
+		return removeComments(query), nil
 	}
 
 	if matFunc, ok := m.MaterializationMap[mat.Type][mat.Strategy]; ok {
@@ -35,6 +35,6 @@ func (m *Materializer) Render(asset *Asset, query string) (string, error) {
 func removeComments(query string) string {
 	bytes := []byte(query)
 	re := regexp.MustCompile("(?s)/\\*.*?\\*/")
-	newBytes := re.ReplaceAll(bytes, nil)
+	newBytes := re.ReplaceAll(bytes, []byte(""))
 	return string(newBytes)
 }

pkg/pipeline/materializer_test.go

@@ -20,7 +20,7 @@ func TestMaterializer_Render(t *testing.T) {
 		},
 	}
 
-	query := "SELECT * FROM table"
+	query := "/* @bruin some yaml @bruin*/SELECT * FROM table"
 	expected := "SELECT * FROM table"
 
 	result, err := materializer.Render(asset, query)