getOptions renamed in osdfapp.py and tests 45/34645/4
authorSastry Isukapalli <sastry@research.att.com>
Thu, 8 Mar 2018 03:00:20 +0000 (22:00 -0500)
committerSastry Isukapalli <sastry@research.att.com>
Thu, 8 Mar 2018 03:15:02 +0000 (22:15 -0500)
Issue-ID: OPTFRA-102
Change-Id: Idf0a61ff837ecd26577febaca5f0538971759792
Signed-off-by: Sastry Isukapalli <sastry@research.att.com>
osdfapp.py
test/mainapp/test_osdfapp.py

index 4230e01..a2af0bf 100755 (executable)
@@ -128,7 +128,7 @@ def internal_failure(error):
     return response
 
 
-def getOptions(argv):
+def get_options(argv):
     program_version_string = '%%prog %s' % ("v1.0")
     program_longdesc = ""
     program_license = ""
@@ -152,11 +152,11 @@ if __name__ == "__main__":
     ports = sys_conf['osdf_ports']
     internal_port, external_port = ports['internal'], ports['external']
     ssl_context = tuple(sys_conf['ssl_context'])
-    local_host = "0.0.0.0"
+    local_host = "0.0.0.0"  # NOSONAR
 
     common_app_opts = dict(host=local_host, threaded=True, use_reloader=False)
 
-    opts = getOptions(sys.argv)
+    opts = get_options(sys.argv)
     if (not opts.local and not opts.devtest):  # normal deployment
         app.run(port=internal_port, ssl_context=ssl_context, debug=False, **common_app_opts)
     else:
index c162ac0..7fbe707 100644 (file)
@@ -65,8 +65,8 @@ class TestOSDFApp(TestCase):
         resp = osdfapp.internal_failure(e)
         assert resp.status_code == 500
 
-    def test_getOptions_default(self):
-        opts = osdfapp.getOptions(["PROG"])  # ensure nothing breaks
+    def test_get_options_default(self):
+        opts = osdfapp.get_options(["PROG"])  # ensure nothing breaks
 
 
 if __name__ == "__main__":