src/main/java/nl/esciencecenter/xenon/adaptors/filesystems/local/LocalFileSystem.java

Use explicit scoping instead of the default package private level
67
Set<PosixFilePermission> xenonPermissions(Set<java.nio.file.attribute.PosixFilePermission> permissions) {

src/main/java/nl/esciencecenter/xenon/adaptors/filesystems/sftp/SftpFileSystem.java

Use explicit scoping instead of the default package private level
403
static XenonException sftpExceptionToXenonException(IOException e, String message) {

src/main/java/nl/esciencecenter/xenon/adaptors/schedulers/ScriptingParser.java

Avoid unnecessary constructors - the compiler will generate these for you
41
public ScriptingParser() {

src/main/java/nl/esciencecenter/xenon/adaptors/schedulers/ssh/SshInteractiveProcess.java

Avoid unused private fields such as 'session'.
43
private final ClientSession session;

src/main/java/nl/esciencecenter/xenon/adaptors/shared/ssh/SSHUtil.java

Avoid unnecessary constructors - the compiler will generate these for you
149
public SSHUtil() {

src/testApi/java/nl/esciencecenter/xenon/adaptors/filesystems/FileSystemTestParent.java

Method names should not contain underscores
108
public void test_createDirectory_existingFile_throw() throws Exception {
Method names should not contain underscores
192
public void test_createFile_withData_sizeKnown() throws Exception {
Method names should not contain underscores
231
public void test_exists_null_throwsException() throws Exception {
Method names should not contain underscores
254
public void test_exists_existingRelativeFile_returnTrue() throws Exception {
Method names should not contain underscores
362
public void test_delete_existingEmptyDir() throws Exception {

src/testApi/java/nl/esciencecenter/xenon/adaptors/schedulers/SchedulerTestParent.java

Method names should not contain underscores
86
public void test_close() throws XenonException {
These nested if statements could be combined
208
if (!stats[i].isDone()) {
Method names should not contain underscores
243
public void test_getDefaultQueueNames() throws XenonException {
Unnecessary use of fully qualified name 'Assert.assertEquals' due to existing static import 'org.junit.Assert.assertEquals'
430
Assert.assertEquals(jobID, status.getJobIdentifier());
Method names should not contain underscores
438
public void test_getJobStatus_finishedJob() throws XenonException {
Unnecessary use of fully qualified name 'Assert.assertEquals' due to existing static import 'org.junit.Assert.assertEquals'
490
Assert.assertEquals("aap", result[0].getJobIdentifier());
Method names should not contain underscores
556
public void test_getQueueStatus_unknownQueue() throws XenonException {
Method names should not contain underscores
581
public void test_getQueueStatuses_empty() throws XenonException {
Unnecessary use of fully qualified name 'Assert.assertEquals' due to existing static import 'org.junit.Assert.assertEquals'
640
Assert.assertEquals(queueNames[0], result[0].getQueueName());
Method names should not contain underscores
724
public void test_interactiveJob_windows() throws Exception {