Удалить ключевое слово async
, потому что вы уже используете Обещание.
Запрос никогда не запускается, потому что вы возвращаете Обещание Обещания (и поскольку никто не слушает внешнее Обещание, он никогда не будет запущен ).
Вы бы поймали эту ошибку, если бы набрали тип возвращаемого значения функции - это хорошая практика, которую вы могли бы рассмотреть.
Кстати, ваш код будет чище, если использовать
this.http.get<...>(...)
.pipe(..., catchError(...))
.toPromise();