[yang] Enforce yang full support in full config command (sonic-net#3716)

What I did
Add strict YANG validation for full config command.

How I did it
Fail if found table with no YANG support

How to verify it
UT

Author: wen587

config/main.py

@@ -1402,6 +1402,13 @@ def config_file_yang_validation(filename):
                         fg='magenta')
             raise click.Abort()
 
+        sy.tablesWithOutYang.pop('bgpraw', None)
+        if len(sy.tablesWithOutYang):
+            click.secho("Config tables are missing yang models: {}".format(str(sy.tablesWithOutYang.keys())),
+                        fg='magenta')
+            raise click.Abort()
+
+
 # This is our main entrypoint - the main 'config' command
 @click.group(cls=clicommon.AbbreviationGroup, context_settings=CONTEXT_SETTINGS)
 @click.pass_context

tests/config_override_input/new_feature_config.json

@@ -14,7 +14,6 @@
 DATA_DIR = os.path.join(SCRIPT_DIR, "config_override_input")
 EMPTY_INPUT = os.path.join(DATA_DIR, "empty_input.json")
 PARTIAL_CONFIG_OVERRIDE = os.path.join(DATA_DIR, "partial_config_override.json")
-NEW_FEATURE_CONFIG = os.path.join(DATA_DIR, "new_feature_config.json")
 FULL_CONFIG_OVERRIDE = os.path.join(DATA_DIR, "full_config_override.json")
 PORT_CONFIG_OVERRIDE = os.path.join(DATA_DIR, "port_config_override.json")
 EMPTY_TABLE_REMOVAL = os.path.join(DATA_DIR, "empty_table_removal.json")
@@ -125,15 +124,6 @@ def test_golden_config_db_partial(self):
             db, config, read_data['running_config'], read_data['golden_config'],
             read_data['expected_config'])
 
-    def test_golden_config_db_new_feature(self):
-        """Golden Config append NEW_FEATURE_TABLE"""
-        db = Db()
-        with open(NEW_FEATURE_CONFIG, "r") as f:
-            read_data = json.load(f)
-        self.check_override_config_table(
-            db, config, read_data['running_config'], read_data['golden_config'],
-            read_data['expected_config'])
-
     def test_golden_config_db_full(self):
         """Golden Config makes change to every table in configDB"""
         db = Db()

tests/config_override_test.py

@@ -1192,6 +1192,30 @@ def read_json_file_side_effect(filename):
             assert result.exit_code != 0
             assert "Authentication with 'tacacs+' is not allowed when passkey not exists." in result.output
 
+    @mock.patch('sonic_py_common.device_info.get_paths_to_platform_and_hwsku_dirs',
+                mock.MagicMock(return_value=("dummy_path", None)))
+    def test_load_minigraph_no_yang_failure(self, get_cmd_module):
+        def is_file_side_effect(filename):
+            return True if 'golden_config' in filename else False
+
+        def read_json_file_side_effect(filename):
+            return {
+                "NEW_FEATURE": {
+                    "global": {
+                        "state": "enable"
+                    }
+                }
+            }
+
+        with mock.patch("utilities_common.cli.run_command", mock.MagicMock(side_effect=mock_run_command_side_effect)), \
+                mock.patch('os.path.isfile', mock.MagicMock(side_effect=is_file_side_effect)), \
+                mock.patch('config.main.read_json_file', mock.MagicMock(side_effect=read_json_file_side_effect)):
+            (config, _) = get_cmd_module
+            runner = CliRunner()
+            result = runner.invoke(config.config.commands["load_minigraph"], ["--override_config", "-y"])
+            assert result.exit_code != 0
+            assert "Config tables are missing yang models: dict_keys(['NEW_FEATURE'])" in result.output
+
     @mock.patch('sonic_py_common.device_info.get_paths_to_platform_and_hwsku_dirs', mock.MagicMock(return_value=("dummy_path", None)))
     def test_load_minigraph_with_traffic_shift_away(self, get_cmd_module):
         with mock.patch("utilities_common.cli.run_command", mock.MagicMock(side_effect=mock_run_command_side_effect)) as mock_run_command: