fix: simple query

Author: arlac77

src/service-ldap.mjs

@@ -23,7 +23,7 @@ export class ServiceLDAP extends Service {
           attributes: {
             bindDN: {
               type: "string"
-            },    
+            },
             base: {
               type: "string"
             },
@@ -62,9 +62,9 @@ export class ServiceLDAP extends Service {
     await this.start();
 
     try {
-      await this.client.bind(this.bindDN);
-      const { entries } = await this.client.search(query.base, query.options);
-      return entries;
+      await this.client.bind(query.bindDN, query.password);
+      return await this.client.search(query.base, query);
+    //  return entries;
     } finally {
       await this.client.unbind();
     }

tests/fixtures/ldap/base.ldif

@@ -38,3 +38,11 @@ dn: cn=konsum,ou=groups,dc=example,dc=com
 cn: konsum
 objectclass: groupOfUniqueNames
 uniqueMember: uid=user1,ou=accounts,dc=example,dc=com
+
+dn: cn=service1,ou=groups,dc=example,dc=com
+cn: service1
+objectclass: groupOfUniqueNames
+
+dn: cn=service2,ou=groups,dc=example,dc=com
+cn: service2
+objectclass: groupOfUniqueNames

tests/query-test.mjs

@@ -0,0 +1,30 @@
+import test from "ava";
+import { StandaloneServiceProvider } from "@kronos-integration/service";
+import { ServiceLDAP } from "../src/service-ldap.mjs";
+
+const config = {
+  type: ServiceLDAP,
+  url: "ldap://localhost:3389",
+  bindDN: "ou=accounts,dc=example,dc=com"
+};
+
+test("service-ldap search", async t => {
+  const sp = new StandaloneServiceProvider();
+  const ldap = await sp.declareService(config);
+  await ldap.start();
+  t.is(ldap.state, "running");
+
+  t.deepEqual(
+    (
+      await ldap.search({
+        bindDN: "uid=user1,ou=accounts,dc=example,dc=com",
+        password: "test",
+        base: "ou=groups,dc=example,dc=com",
+        scope: "sub",
+        attributes: ["cn"],
+        filter: "(objectclass=groupOfUniqueNames)"
+      })
+    ).searchEntries,
+    [{ cn: "konsum", dn: "cn=konsum,ou=groups,dc=example,dc=com" }]
+  );
+});

tests/start-local-ldap.sh

@@ -17,5 +17,5 @@ case $(uname) in
 esac
 
 
-${SLAPD} -f ${SLAPD_CONF} -h ldap://localhost:3389 -d 9 &
+${SLAPD} -f ${SLAPD_CONF} -h ldap://localhost:3389 -d 1 &
 ldapadd -h localhost:3389 -D cn=Manager,dc=example,dc=com -w test -f tests/fixtures/ldap/base.ldif