Quanto ao método à declaração do método listFromUser na service, o mesmo não deveria ser implementado com o retorno explícito ao invés de
listFromUser(userName: string) { // code }
declarar da seguinte forma:
listFromUser(userName: string) : Photo[] { // code }
Fala aí Phael, tudo bem? Depende, não existe nada "certo" sobre o assunto.
Eu Matheus, gosto de deixar todos os retornos explicitos, no meu tslint configuro para dar erro caso o retorno não seja informado.
Mas, é uma questão de boa prática e depende de cada um ou empresa.
Espero ter ajudado.