Disable Semaphore and fix some flaky tests

[rvazarkar]

fix: temporarily disable semaphore until we can iron out deadlocks
chore: some misc test fixes to fix flakiness
chore: fix an async function that wasn't async

Description

Motivation and Context

The semaphore in LdapConnectionPool appears to be causing a deadlock during heavy enumeration in certain edge cases. Until we can figure out whats causing it, we're disabling the behavior

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

• Chore (a change that does not modify the application functionality)
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• Documentation updates are needed, and have been made accordingly.
• I have added and/or updated tests to cover my changes.
• All new and existing tests passed.
• My changes include a database migration.