Rather than catching NPE's I'd rather see something like   

    static File[] listLogFiles() {
        final File logFile = new File("");
        File logDir = logFile.getParentFile();
        Assert.assertNotNull("Log directory not found", logDir);
        return logDir.listFiles();
    }

I'd at least prefer a ```null``` check over catching an NPE.