From 986b89f5ed4efaf7ad38fdb2f43bedd5df952a75 Mon Sep 17 00:00:00 2001 From: Orne Brocaar Date: Thu, 9 Jul 2015 20:46:42 +0200 Subject: [PATCH] Cleanup tests. --- plugins/elasticsearch/elasticsearch_test.go | 49 +++++++-------------- 1 file changed, 15 insertions(+), 34 deletions(-) diff --git a/plugins/elasticsearch/elasticsearch_test.go b/plugins/elasticsearch/elasticsearch_test.go index 6611bca01..63abc6bec 100644 --- a/plugins/elasticsearch/elasticsearch_test.go +++ b/plugins/elasticsearch/elasticsearch_test.go @@ -51,41 +51,22 @@ func TestElasticsearch(t *testing.T) { "node_host": "test", } - for key, val := range indicesExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) + testTables := []map[string]float64{ + indicesExpected, + osExpected, + processExpected, + jvmExpected, + threadPoolExpected, + networkExpected, + fsExpected, + transportExpected, + httpExpected, + breakersExpected, } - for key, val := range osExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) - } - - for key, val := range processExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) - } - - for key, val := range jvmExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) - } - - for key, val := range threadPoolExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) - } - - for key, val := range networkExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) - } - - for key, val := range fsExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) - } - - for key, val := range transportExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) - } - for key, val := range httpExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) - } - for key, val := range breakersExpected { - assert.NoError(t, acc.ValidateTaggedValue(key, val, tags)) + for _, testTable := range testTables { + for k, v := range testTable { + assert.NoError(t, acc.ValidateTaggedValue(k, v, tags)) + } } }