search

laminas / laminas-ldap

Provides support for LDAP operations including but not limited to binding, searching and modifying entries in an LDAP directory
https://docs.laminas.dev/laminas-ldap/
BSD 3-Clause "New" or "Revised" License
8 stars 18 forks source link

Introduce CI setup for online ldap tests. Fix discovered type errors #49

Closed Xerkus closed 11 months ago

Xerkus commented 11 months ago
Q A
Documentation no
Bugfix yes
BC Break no
New Feature yes
RFC no
QA yes

Description

Majority of tests are integration tests requiring ldap instance. This PR introduces initial CI setup to run those tests.

Locally ran online tests uncovered two type errors and this PR fixes them as well.

Ocramius commented 11 months ago

Thanks, @Xerkus!