this could be easily tested but I did not see any unit tests on this PR