update solution, still not passing rspec