code looks good, test shows that it doesnt work