node-ldapjs
node-ldapjs copied to clipboard
fix #765
Can you add a test for this change?
Can you add a test for this change?
there are already many in test/client.test.js
...
res.on('searchEntry', function (entry) {
t.ok(entry instanceof ldap.SearchEntry)
...
res.on('searchReference', function (referral) {
t.ok(referral instanceof ldap.SearchReference)
...
Those were passing before this change so they don't really show the need for this change.
👋
On February 22, 2023, we released version 3 of this library. As a result, we are closing this issue/pull request.
Please see issue #839 for more information, including how to proceed if you feel this closure is in error.