[Asterisk-code-review] tests/apps/statsd/nominal: Add prefix. (testsuite[master])

[Anonymous Coward]

Anonymous Coward #1000019 has submitted this change and it was merged.

Change subject: tests/apps/statsd/nominal: Add prefix.
......................................................................


tests/apps/statsd/nominal: Add prefix.

As StatsD is now seeing more usage some additional packets may
be received outside the scope of this test. As a result a prefix has
been added to limit what it sees to only the test packets.

Change-Id: I50617f6f104e47a447b908132be3a610b8a62305
---
M tests/apps/statsd/nominal/configs/ast1/extensions.conf
M tests/apps/statsd/nominal/test-config.yaml
2 files changed, 9 insertions(+), 8 deletions(-)

Approvals:
  Anonymous Coward #1000019: Verified
  Matt Jordan: Looks good to me, approved



diff --git a/tests/apps/statsd/nominal/configs/ast1/extensions.conf b/tests/apps/statsd/nominal/configs/ast1/extensions.conf
index ceebb56..da1c3cf 100644
--- a/tests/apps/statsd/nominal/configs/ast1/extensions.conf
+++ b/tests/apps/statsd/nominal/configs/ast1/extensions.conf
@@ -1,7 +1,7 @@
 [default]
 
 exten => start,1,NoOp()
-	same => n,StatsD(g,foo,1,1)
-	same => n,StatsD(s,bar,2)
-	same => n,StatsD(ms,baz,3,1)
-	same => n,StatsD(c,last,4)
\ No newline at end of file
+	same => n,StatsD(g,nominal.foo,1,1)
+	same => n,StatsD(s,nominal.bar,2)
+	same => n,StatsD(ms,nominal.baz,3,1)
+	same => n,StatsD(c,nominal.last,4)
diff --git a/tests/apps/statsd/nominal/test-config.yaml b/tests/apps/statsd/nominal/test-config.yaml
index 2486528..cec4f57 100644
--- a/tests/apps/statsd/nominal/test-config.yaml
+++ b/tests/apps/statsd/nominal/test-config.yaml
@@ -24,15 +24,16 @@
     reactor-timeout: 15
 
 statsd-config:
+    prefix: nominal
     packets:
         -
-            'foo:1|g'
+            'nominal.foo:1|g'
         -
-            'bar:2|s'
+            'nominal.bar:2|s'
         -
-            'baz:3|ms'
+            'nominal.baz:3|ms'
         -
-            'last:4|c'
+            'nominal.last:4|c'
 
 originator-config:
     channel: 'Local/start at default'

-- 
To view, visit https://gerrit.asterisk.org/1722
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I50617f6f104e47a447b908132be3a610b8a62305
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>