test for nulls etc