Refactor

acuminous · Sep 16, 2023 · 58276d4 · 58276d4
1 parent 24b9171
commit 58276d4
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/lib/ManagedResources.js b/lib/ManagedResources.js
@@ -36,10 +36,9 @@ module.exports = class ManagedResources {
  }
 
  async destroyIdleResources() {
- const destroys = this.#list.map((managedResource, index) => {
- if (!managedResource.isIdle()) return;
- return this.destroy(managedResource.resource).then(() => this.#removeManagedResource(managedResource, index));
- });
+ const destroys = this.#list.map((managedResource, index) => (managedResource.isIdle()
+ ? managedResource.destroy().then(() => this.#removeManagedResource(managedResource, index))
+ : Promise.resolve()));
  return Promise.all(destroys);
  }
 
@@ -62,15 +61,15 @@ module.exports = class ManagedResources {
  }
 
  hasIdleResources() {
- return Boolean(this.getIdleManagedResource());
+ return this.#list.some((managedResource) => managedResource.isIdle());
  }
 
  getIdleManagedResource() {
  return this.#list.find((managedResource) => managedResource.isIdle());
  }
 
  hasAcquiredResources() {
- return Boolean(this.#list.find((managedResource) => managedResource.isAcquired()));
+ return this.#list.some((managedResource) => managedResource.isAcquired());
  }
 
  hasSpareCapacity() {