no sense in testing empty values here