Why the change here? When will @product be nil that we want to pass the read_test?