Commenting out the skipping of tests because I think this might be reason we didn't include test jar in our artifact set published to mvn