Add test for issue 860

This PR adds a new integration test for issue #860.

Author: jsumners

test-integration/client/issue-860.test.js

@@ -0,0 +1,48 @@
+'use strict'
+
+const tap = require('tap')
+const ldapjs = require('../../lib')
+const parseDN = ldapjs.parseDN
+
+const SCHEME = process.env.SCHEME || 'ldap'
+const HOST = process.env.HOST || '127.0.0.1'
+const PORT = process.env.PORT || 389
+const baseURL = `${SCHEME}://${HOST}:${PORT}`
+
+const client = ldapjs.createClient({ url: baseURL })
+
+const opts = {
+  filter: '(&(objectClass=person))',
+  scope: 'sub',
+  paged: true,
+  sizeLimit: 100,
+  attributes: ['cn', 'employeeID']
+}
+
+const baseDN = parseDN('ou=テスト,dc=planetexpress,dc=com')
+
+tap.test('can search OUs with Japanese characters', t => {
+  client.bind('cn=admin,dc=planetexpress,dc=com', 'GoodNewsEveryone', (err) => {
+    t.error(err, 'bind error')
+  })
+
+  client.search(baseDN.toString(), opts, (err, res) => {
+    t.error(err, 'search error')
+    res.on('searchEntry', (entry) => {
+      t.match(entry.pojo, {
+        type: 'SearchResultEntry',
+        objectName: 'cn=jdoe,ou=\\e3\\83\\86\\e3\\82\\b9\\e3\\83\\88,dc=planetexpress,dc=com',
+        attributes: [{
+          type: 'cn',
+          values: ['John', 'jdoe']
+        }]
+      })
+    })
+    res.on('error', (err) => {
+      t.error(err, 'search entry error')
+    })
+    res.on('end', () => {
+      client.unbind(t.end)
+    })
+  })
+})

test-integration/client/issues.test.js

@@ -80,9 +80,9 @@ tap.test('can access large groups (issue #582)', t => {
 
         const memberAttr = results[0].attributes.find(a => a.type === 'member')
         t.ok(memberAttr)
-        t.ok(memberAttr.vals)
-        t.type(memberAttr.vals, Array)
-        t.equal(memberAttr.vals.length, 2000)
+        t.ok(memberAttr.values)
+        t.type(memberAttr.values, Array)
+        t.equal(memberAttr.values.length, 2000)
 
         client.unbind(t.end)
       })