[Asterisk-code-review] 'escape-pattern' should default to 'False' if not specified (testsuite[master])

[Kevin Harwell]

Kevin Harwell has uploaded a new change for review.

  https://gerrit.asterisk.org/607

Change subject: 'escape-pattern' should default to 'False' if not specified
......................................................................

'escape-pattern' should default to 'False' if not specified

If not specified in the yaml config the 'escape-pattern' option was defaulting
to 'True'. This caused some tests to fail due to the pattern being escaped
before being matched. This patch defaults the option to 'False'.

Reported by: jbigelow

Change-Id: I836b30c867f5a36580e5dc525121a58aba4c87db
---
M lib/python/asterisk/ami.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.asterisk.org:29418/testsuite refs/changes/07/607/1

diff --git a/lib/python/asterisk/ami.py b/lib/python/asterisk/ami.py
index f3a8e00..8958c8b 100644
--- a/lib/python/asterisk/ami.py
+++ b/lib/python/asterisk/ami.py
@@ -46,7 +46,7 @@
         self._event_observers = []
         self.count = {}
 
-        if instance_config.get('escape-pattern', True):
+        if instance_config.get('escape-pattern', False):
             self.match = (lambda pattern, s, flags=0:
                           re.match(re.escape(pattern), s, flags))
         else:

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I836b30c867f5a36580e5dc525121a58aba4c87db
Gerrit-PatchSet: 1
Gerrit-Project: testsuite
Gerrit-Branch: master
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>