[Asterisk-code-review] Remove YAMLLoadWarning: calling yaml.load() (II) (...testsuite[16])

Friendly Automation asteriskteam at digium.com
Fri May 3 08:41:31 CDT 2019


Friendly Automation has submitted this change and it was merged. ( https://gerrit.asterisk.org/c/testsuite/+/11341 )

Change subject: Remove YAMLLoadWarning: calling yaml.load() (II)
......................................................................

Remove YAMLLoadWarning: calling yaml.load() (II)

As commit cac597f96670b994727f6fe98f17c70eefa5f635 remove use of plan
for yaml.load without Loader deprecated from PyYAML 5.1

Change-Id: Ie29b2709c6110a721dc93a824a9ae4bc2adc0d46
---
M lib/python/asterisk/test_runner.py
M usage.py
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  George Joseph: Looks good to me, approved
  Friendly Automation: Approved for Submit



diff --git a/lib/python/asterisk/test_runner.py b/lib/python/asterisk/test_runner.py
index d278e02..443b712 100644
--- a/lib/python/asterisk/test_runner.py
+++ b/lib/python/asterisk/test_runner.py
@@ -233,7 +233,7 @@
         return test_config
 
     with open(test_config_path) as file_stream:
-        test_config = yaml.load(file_stream, )
+        test_config = yaml.load(file_stream, Loader=yaml.SafeLoader)
 
     return test_config
 
diff --git a/usage.py b/usage.py
index 9e7c33f..5b12bdd 100755
--- a/usage.py
+++ b/usage.py
@@ -101,7 +101,7 @@
         print("Unexpected error: %s" % sys.exc_info()[0])
         return None
 
-    config = yaml.load(f)
+    config = yaml.load(f, Loader=yaml.SafeLoader)
     f.close()
 
     return config

-- 
To view, visit https://gerrit.asterisk.org/c/testsuite/+/11341
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: 16
Gerrit-Change-Id: Ie29b2709c6110a721dc93a824a9ae4bc2adc0d46
Gerrit-Change-Number: 11341
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Rodrigo Ramirez Norambuena <a at rodrigoramirez.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190503/a25e0912/attachment.html>


More information about the asterisk-code-review mailing list