O problema é que upper
retorna outra string. Ele não altera a original (aliás, isso vale para todos os métodos que "modificam" a string - em Python, strings são imutáveis, e estes métodos na verdade sempre retornam outra string modificada).
Sendo assim, você tem que pegar o retorno e ir acumulando em outra string:
def consoantes_maiusculas(frase):
s = ''
for caractere in frase:
if caractere in 'bcdfghjklmnpqrstvxwyz':
s += caractere.upper() # transforma em maiúscula
else: # não é consoante minúscula, mantém como no original
s += caractere
return s
print(consoantes_maiusculas('abcdef')) # aBCDeF
Outra alternativa é usar join
juntamente com uma generator expression:
def consoantes_maiusculas(frase):
return ''.join(caractere.upper() if caractere in 'bcdfghjklmnpqrstvxwyz' else caractere for caractere in frase)
Inclusive, usar join
é o mais recomendado neste caso, por ser bem mais eficiente do que a primeira alternativa.