Про Object.assign знаюи чем он не угодил? Уж точно лучше тормознутого for-in, который у Вас кстати без проверки на собственные свойства...
Теперь, когда я тебе это все расписал, тебе стало понятней что я имел ввиду?
return result;
- это return из функции обработчика события this.openRequest.onsuccess
, а не из метода getActiveUser, из метода getActiveUser не получится вернуть result, так как когда он у Вас появится, метод getActiveUser будет давно завершен.А не нарушит ли этот метод принцип единственной ответственности - поиск юзеров, для которых нужно создать + непосредственно создание...