Porque estás usando upper()
en lugar de isupper()
upper()
convierte a mayúsculas, no es un chequeo de datos.
isupper()
chequea si algo está en mayúsculas o no.
Actualización: La forma eficiente de hacer eso no es comparar la letra con otra cosa, si no chequear si la letra es simplemente mayúscula.
def eva_cadena():
contador = 0
cadena = input("ingrese una cadena: ")
for i in cadena:
if i.isupper(): # No hace falta convertir nada.
contador += 1
print(contador)
Actualización 2: Por si es más fácil de entender sin utilizar una función, esto sería similar a:
print(sum([i.isupper() for i in input("ingrese una cadena: ")]))
upper()
devuelve el mismo caracter en esos casos, y tu código los estaría contando como mayúsculas.