Look at values file too to determine if mtls is enabled for the test or not.

gargnupur · gargnupur · commit 577c8d9dc2dd · 2019-01-15T13:42:40.000-08:00
diff --git a/pkg/test/framework/runtime/components/environment/kube/environment.go b/pkg/test/framework/runtime/components/environment/kube/environment.go
@@ -18,13 +18,16 @@ import (
 	"bytes"
 	"fmt"
 	"io"
+	"path/filepath"
 	"strings"
 	"testing"
 	"text/template"
 
 	"github.com/google/uuid"
 	multierror "github.com/hashicorp/go-multierror"
+	yaml2 "gopkg.in/yaml.v2"
 
+	"istio.io/istio/pkg/test"
 	"istio.io/istio/pkg/test/deployment"
 	"istio.io/istio/pkg/test/framework/api/component"
 	"istio.io/istio/pkg/test/framework/api/context"
@@ -92,12 +95,31 @@ func (e *Environment) Scope() lifecycle.Scope {
 	return e.scope
 }
 
-// Is mtls enabled
+// Is mtls enabled. Check in Values flag and Values file.
 func (e *Environment) IsMtlsEnabled() bool {
 	if e.s.Values["global.mtls.enabled"] == "true" {
 		return true
 	}
 
+	data, err := test.ReadConfigFile(filepath.Join(e.s.ChartDir, e.s.ValuesFile))
+	if err != nil {
+		return false
+	}
+	m := make(map[interface{}]interface{})
+	err = yaml2.Unmarshal([]byte(data), &m)
+	if err != nil {
+		return false
+	}
+	if m["global"] != nil {
+		switch globalVal := m["global"].(type) {
+		case map[interface{}]interface{}:
+			switch mtlsVal := globalVal["mtls"].(type) {
+			case map[interface{}]interface{}:
+				return mtlsVal["enabled"].(bool)
+			}
+		}
+	}
+
 	return false
 }
 
diff --git a/tests/integration2/mixer/scenarios_test.go b/tests/integration2/mixer/scenarios_test.go
@@ -146,7 +146,6 @@ func TestTcpMetric(t *testing.T) {
 	visitProductPage(ingress, 30*time.Second, 200, t)
 
 	query := fmt.Sprintf("sum(istio_tcp_sent_bytes_total{destination_app=\"%s\"})", "mongodb")
-
 	validateMetric(t, prometheus, query, "istio_tcp_sent_bytes_total")
 
 	query = fmt.Sprintf("sum(istio_tcp_received_bytes_total{destination_app=\"%s\"})", "mongodb")
