Pull JSON schemas at build/test not run time

Change-Id: I4f095a8a80f1c2868319c58bb0e9db916044d067
Issue-ID: DCAEGEN2-2221
Signed-off-by: Andrew Gauld <agauld@att.com>

2 files changed, 6 insertions, 47 deletions

diff --git a/mod/onboardingapi/dcae_cli/util/config.py b/mod/onboardingapi/dcae_cli/util/config.py
index f9936c3..8eb28c8 100644
--- a/mod/onboardingapi/dcae_cli/util/config.py
+++ b/mod/onboardingapi/dcae_cli/util/config.py
@@ -1,7 +1,7 @@
 # ============LICENSE_START=======================================================
 # org.onap.dcae
 # ================================================================================
-# Copyright (c) 2017-2018 AT&T Intellectual Property. All rights reserved.
+# Copyright (c) 2017-2020 AT&T Intellectual Property. All rights reserved.
 # ================================================================================
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -40,16 +40,6 @@ def get_config_path():
     return os.path.join(get_app_dir(), 'config.json')
 
 
-def _init_config_user():
-    while True:
-        user = click.prompt('Please enter your user id', type=str).strip()
-
-        # There should be no special characters
-        if re.match("(?:\w*)\Z", user):
-            return user
-        else:
-            click.echo("Invalid user id. Please try again.")
-
 def _init_config_server_url():
     return click.prompt("Please enter the remote server url", type=str).strip()
 
@@ -78,8 +68,6 @@ def _init_config():
 
     # UPDATE: Keeping the server url even though the config was not found there.
     new_config["server_url"] = server_url
-    new_config["user"] = _init_config_user()
-    new_config["cli_version"] = _version.__version__
 
     if "db_url" not in new_config or not new_config["db_url"]:
         # The seed configuration was not provided so manually set up the db
@@ -94,11 +82,6 @@ def _init_config():
     return new_config
 
 
-def should_force_reinit(config):
-    """Configs older than 2.0.0 should be replaced"""
-    ver = config.get("cli_version", "0.0.0")
-    return int(ver.split(".")[0]) < 2
-
 def get_config():
     '''Returns the configuration dictionary'''
     return get_pref(get_config_path(), _init_config)
diff --git a/mod/onboardingapi/dcae_cli/util/tests/test_config.py b/mod/onboardingapi/dcae_cli/util/tests/test_config.py
index 3b4cd6e..047cdf8 100644
--- a/mod/onboardingapi/dcae_cli/util/tests/test_config.py
+++ b/mod/onboardingapi/dcae_cli/util/tests/test_config.py
@@ -1,7 +1,7 @@
 # ============LICENSE_START=======================================================
 # org.onap.dcae
 # ================================================================================
-# Copyright (c) 2017 AT&T Intellectual Property. All rights reserved.
+# Copyright (c) 2017-2020 AT&T Intellectual Property. All rights reserved.
 # ================================================================================
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -38,7 +38,7 @@ def test_no_config(monkeypatch, tmpdir):
     '''Tests the creation and initialization of a config on a clean install'''
     monkeypatch.setattr(click, "get_app_dir", lambda app: str(tmpdir.realpath()))
 
-    mock_config = {'user': 'mock-user'}
+    mock_config = {}
 
     config_file = tmpdir.join("config.json")
     config_file.write(json.dumps(mock_config))
@@ -46,26 +46,13 @@ def test_no_config(monkeypatch, tmpdir):
     assert get_config() == mock_config
 
 
-def test_init_config_user(monkeypatch):
-    good_case = "abc123"
-    values = [ good_case, "d-e-f", "g*h*i", "j k l" ]
-
-    def fake_input(values, message, type="red"):
-        return values.pop()
-
-    monkeypatch.setattr(click, 'prompt', partial(fake_input, values))
-    assert config._init_config_user() == good_case
-
-
 def test_init_config(monkeypatch):
-    monkeypatch.setattr(config, '_init_config_user', lambda: "bigmama")
     monkeypatch.setattr(config, '_init_config_server_url',
             lambda: "http://some-nexus-in-the-sky.com")
     monkeypatch.setattr(dcae_cli.util, 'fetch_file_from_web',
             lambda server_url, path: { "db_url": "conn" })
-    monkeypatch.setattr("dcae_cli._version.__version__", "2.X.X")
 
-    expected = {'cli_version': '2.X.X', 'user': 'bigmama', 'db_url': 'conn',
+    expected = {'db_url': 'conn',
             'server_url': 'http://some-nexus-in-the-sky.com',
             'active_profile': 'default' }
     assert expected == config._init_config()
@@ -104,28 +91,17 @@ def test_init_config(monkeypatch):
         config._init_config()
 
 
-def test_should_force_reinit():
-    bad_config = {}
-    assert config.should_force_reinit(bad_config) == True
-
-    old_config = { "cli_version": "1.0.0" }
-    assert config.should_force_reinit(old_config) == True
-
-    uptodate_config = { "cli_version": "2.0.0" }
-    assert config.should_force_reinit(uptodate_config) == False
-
-
 def test_reinit_config(monkeypatch, tmpdir):
     monkeypatch.setattr(click, "get_app_dir", lambda app: str(tmpdir.realpath()))
 
-    new_config = { "user": "ninny", "db_url": "some-db" }
+    new_config = { "db_url": "some-db" }
 
     def init():
         return new_config
 
     assert config._reinit_config(init) == new_config
 
-    old_config = { "user": "super", "db_url": "other-db", "hidden": "yo" }
+    old_config = { "db_url": "other-db", "hidden": "yo" }
     write_pref(old_config, get_config_path())
 
     new_config["hidden"] = "yo"