From 4ea585dc826a6b6aa0f0ab93f30405ad2c9fca37 Mon Sep 17 00:00:00 2001
From: David Pilnik <davidpil@nvidia.com>
Date: Mon, 25 Dec 2023 16:02:43 +0200
Subject: [PATCH] ldap: Add YANG tests

---
 src/sonic-yang-models/setup.py                |  1 +
 .../tests/files/sample_config_db.json         | 16 +++++
 .../tests/yang_model_tests/tests/ldap.json    | 16 +++++
 .../yang_model_tests/tests_config/ldap.json   | 61 +++++++++++++++++++
 4 files changed, 94 insertions(+)
 create mode 100644 src/sonic-yang-models/tests/yang_model_tests/tests/ldap.json
 create mode 100644 src/sonic-yang-models/tests/yang_model_tests/tests_config/ldap.json

diff --git a/src/sonic-yang-models/setup.py b/src/sonic-yang-models/setup.py
index ff0160ee37f3..9168adae0649 100644
--- a/src/sonic-yang-models/setup.py
+++ b/src/sonic-yang-models/setup.py
@@ -162,6 +162,7 @@ def run(self):
                          './yang-models/sonic-system-aaa.yang',
                          './yang-models/sonic-system-tacacs.yang',
                          './yang-models/sonic-system-radius.yang',
+                         './yang-models/sonic-system-ldap.yang',
                          './yang-models/sonic-telemetry.yang',
                          './yang-models/sonic-telemetry_client.yang',
                          './yang-models/sonic-gnmi.yang',
diff --git a/src/sonic-yang-models/tests/files/sample_config_db.json b/src/sonic-yang-models/tests/files/sample_config_db.json
index 30b6de917b9e..86772196f54e 100644
--- a/src/sonic-yang-models/tests/files/sample_config_db.json
+++ b/src/sonic-yang-models/tests/files/sample_config_db.json
@@ -1565,6 +1565,22 @@
                 "timeout": "5"
             }
         },
+        "LDAP": {
+            "global": {
+                "bind_dn": "test_bind",
+                "bind_password": "secret",
+                "bind_timeout": "5",
+                "version": "3",
+                "base_dn": "test_base",
+                "port": "389",
+                "timeout": "5"
+            }
+        },
+        "LDAP_SERVER": {
+            "192.168.1.1": {
+                "priority": "5"
+            }
+        },
         "NAT_BINDINGS": {
             "bind1": {
                 "nat_pool": "pool1",
diff --git a/src/sonic-yang-models/tests/yang_model_tests/tests/ldap.json b/src/sonic-yang-models/tests/yang_model_tests/tests/ldap.json
new file mode 100644
index 000000000000..93a6032bcf1b
--- /dev/null
+++ b/src/sonic-yang-models/tests/yang_model_tests/tests/ldap.json
@@ -0,0 +1,16 @@
+{
+    "LDAP_TABLE": {
+        "desc": "Configure LDAP global fields."
+    },
+    "LDAP_INVALID_TIMEOUT_TEST": {
+        "desc": "LDAP global configuration with invalid timeout value in LDAP table.",
+        "eStr": "LDAP timeout must be 1..60"
+    },
+    "LDAP_SERVER_TEST" : {
+        "desc": "LDAP server configuration in LDAP_SERVER table."
+    },
+    "LDAP_SERVER_INVALID_PRIORITY_TEST": {
+        "desc": "LDAP server configuration with invalid priority value in LDAP_SERVER table.",
+        "eStr": "LDAP server priority must be 1..8"
+    }
+}
\ No newline at end of file
diff --git a/src/sonic-yang-models/tests/yang_model_tests/tests_config/ldap.json b/src/sonic-yang-models/tests/yang_model_tests/tests_config/ldap.json
new file mode 100644
index 000000000000..39ff556cb2a8
--- /dev/null
+++ b/src/sonic-yang-models/tests/yang_model_tests/tests_config/ldap.json
@@ -0,0 +1,61 @@
+{
+    "LDAP_TABLE": {
+        "sonic-system-ldap:sonic-system-ldap": {
+            "sonic-system-ldap:LDAP": {
+                "global":{
+                    "bind_dn": "test_bind",
+                    "bind_password": "secret",
+                    "bind_timeout": "5",
+                    "version": "3",
+                    "base_dn": "test_base",
+                    "port": "389",
+                    "timeout": "5"
+                }
+            }
+        }
+    },
+    "LDAP_INVALID_TIMEOUT_TEST": {
+        "sonic-system-ldap:sonic-system-ldap": {
+            "sonic-system-ldap:LDAP": {
+                "global": {
+                    "bind_dn": "test_bind",
+                    "bind_password": "secret",
+                    "bind_timeout": "5",
+                    "version": "3",
+                    "base_dn": "test_base",
+                    "port": "389",
+                    "timeout": 150
+                }
+            }
+        }
+    },
+    "LDAP_SERVER_TEST": {
+        "sonic-system-ldap:sonic-system-ldap": {
+            "sonic-system-ldap:LDAP_SERVER": {
+                "LDAP_SERVER_LIST": [
+                {
+                        "hostname": "192.168.1.1",
+                        "priority": 1
+                },
+                {
+                        "hostname": "ldap_server_1",
+                        "priority": 2
+                }
+                ]
+            }
+        }
+    },
+
+    "LDAP_SERVER_INVALID_PRIORITY_TEST": {
+        "sonic-system-ldap:sonic-system-ldap": {
+            "sonic-system-ldap:LDAP_SERVER": {
+                "LDAP_SERVER_LIST": [
+                {
+                    "hostname": "192.168.1.1",
+                    "priority": 70
+                }
+                ]
+            }
+        }
+    }
+}
\ No newline at end of file