Frozen Coder

Ну, ИМХО, логика простая. В конце каждого выражения нужно ставить ";".
Это является выражением?

return function() {
    ...
}

да, поэтому ставим. А это является?

function func() {

}

нет, поэтому не ставим. Я думаю, линтеры со мной согласятся :-)

1. Создание tcp-соединения медленное;
   
2. Создание tls-соединения ещё медленнее;
   
3. Установка соединения протокола уровня приложения ещё медленнее;
   
4. Аутентификация подключения на порядок медленнее;
   
5. Создание сессии для соединения системой управления базами данных такая же медленная;
   
6. Завершение сессии и освобождение ресурсов медленное;
   
7. Закрытие соединение относительно быстрое, но тоже не мгновенное.
   

Если 500 раз в секунду соединение создаётся, через него выполняется один простой запрос, и соединение закрывается, то СУБД и клиент часть пропускной способности тратят на выполнение бессмысленных действий. Намного разумнее один раз открыть соединение, прогнать через него два миллиона запросов и только потому может быть закрыть.

> Насколько я понимаю, смысл mock-функций в jest - это заглушки для функций, чтобы не тащить код всего модуля и не замедлять процесс тестирования.

Это не совсем так. Моки подразумевают, что вы прямо проверяете то что мокаете. Что функция была вызвана, что запрос был выполнен. Это и называется мокинг. А просто заглушка это стаб. Ее смысл не в том чтобы не тащить код, а в том чтобы изолировать побочные эффекты и добиться детерминированности. К последнему, например, относятся таймеры и рандомные числа. Если все это используется внутри программы, то вы не сможете просто так ее протестировать.

Подводя итог, мокают для того чтобы проверить сам мок, например вы хотите убедиться что запрос действительно делался (как в примере документации jest). В остальных случаях у вас стаб (даже если либа называет его моком). Стаб используется для того чтобы тестировать свой код, а стаб нужен только для изоляции побочного эффекта.

Ни первое ни второе напрямую с видом тестирования не связано. Моки и стабы могут применяться практически на любом уровне автоматизированного тестирования.

Темы для самостоятельного изучения:

Побочные эффекты
Детерминированность
Чистые функции
https://martinfowler.com/articles/mocksArentStubs.html

this.handleAddNews.bind(this, arguments)

А не кажется ли вам, что тут есть что-то лишнее?

Ну и вообще, не надо так. Вместо

handleAddNews(newPost) {

пишите

handleAddNews = (newPost) => {

и тогда никакого bind использовать будет не надо:

<Add newsadd={this.handleAddNews} />

VanillaJS, очень хороший фреймворк. Перешел на него с jQuery и всем советую.

render() {
  const { someStateKey } = this.props;  

  if (!someStateKey) return null;

  return (
    {/* component JSX code */}
  );
}

НАПРИМЕР вот как событие по клику реализовано в стандарте ES5

ES5 тут не при чем, это синтаксис jQuery
в ES5/ES6 на DOM-ивенты подписываются методом

target.addEventListener(type, listener[, options]);

где type - тип ивента,
target - объект реализующий EventTarget интерфейс (например Element),
listener - коллбэк который вызывается после ивента.

В офисе