Использование forEach и async/await ведет себя по-разному для узла и Jest

Асинхронные функции работают даже в контекстах, которые не вызывают их await или не вызывают .then(), то есть я точно могу это сделать:

async function foo() {
  // async calls here
}

foo(); // no await or .then().

Это означает, что вы не можете дождаться завершения операции, вы не можете использовать значение и, что хуже всего, вы не можете поймать или исправить любые асинхронные ошибки, которые могут быть вызваны (или отклонены, если быть точным).

Основное отличие состоит в том, что .forEach() не заботится о завершении операций или ожидает их завершения перед вызовом следующей (поскольку асинхронные функции return выполняются немедленно), тогда как ваш вызов for..of использует await для ожидания завершения каждой операции, прежде чем перейти к следующей. .

Ваш первый пример .forEach был бы примерно эквивалентен нижнему, если бы вы удалили await из вызова внутри цикла.

В результате ваш первый пример возвращает обещание, которое разрешается немедленно, а не после завершения всех ваших вызовов БД, поэтому тест ожидает завершения операций, но это не так. Во втором примере вы должны правильно await завершить все вызовы до завершения асинхронной функции, поэтому return Promise будет разрешаться только после того, как все вызовы разрешатся сами собой.

В этой заметке два примера не эквивалентны, потому что первый будет вызывать insertMany один за другим, не дожидаясь их завершения, что приводит к параллельному выполнению вызовов БД.

Если вы хотите сохранить это поведение, но по-прежнему возвращать правильное обещание, которое ожидает завершения всего, вам следует использовать [].map() вместо [].forEach:

const writeToDB = async (db, data) => {
  const dataKeys = Object.keys(data)
  const allPromises = dataKeys.map(async key =>
    await db.collection(key).insertMany(data[key])) // run all calls in parallel
  return await Promise.all(allPromises); // wait for them all to finish
}

Существующий ответ уже подробно объясняет, почему forEach не следует использовать с обещаниями так, как он используется. forEach обратный вызов не учитывает возвращенные обещания и разрывает цепочку обещаний. async..await необходимо использовать с for..of для последовательной оценки обещаний или с Promise.all и map для параллельной оценки.

Jest поддерживает обещания и ожидает, что обещание, возвращаемое асинхронной функцией (it и т. д.), означает, что асинхронный процесс, который произошел в этой функции, завершен.

Как только Jest завершает тест, он проверяет, есть ли открытые дескрипторы, препятствующие выходу Node. Поскольку обещания не возвращались и не связывались Jest, процессы, которые они представляли, не позволяли Jest завершить процесс тестирования.

Эта проблема представлена ​​указанным сообщением об ошибке:

Jest did not exit one second after the test run has completed.

This usually means that there are asynchronous operations that weren't stopped in your tests. Consider running Jest with --detectOpenHandles to troubleshoot this issue.