Regression in itw from Tue Mar 26

Adam Domurad adomurad at redhat.com
Wed Apr 24 09:34:23 PDT 2013


> +    @Test
> +    public void testNotNullJnlpFile(){
> +        SecurityDesc securityDesc = new SecurityDesc(new 
> DummyJNLPFile(), SecurityDesc.SANDBOX_PERMISSIONS, "hey!");
> +        Assert.assertNotNull("securityDesc should not be ",securityDesc);
> +
> +    }

Huh. Why is this still in ? Sorry, I can't support superstitious coding.

At least drop the assert if you want a constructor sanity check.

Push without this please. Otherwise good.

Happy hacking,
-Adam



More information about the distro-pkg-dev mailing list