unknown 21-74327cdd 21 src/main/java/com/jcabi/ssh/SSH.java 70-72 30 This class has common data, implementation of exec() and constructor validations with SSHByPassword class. Common functionality should be extracted into a separate module. IMP Suseika wlasowegor@gmail.com unknown 21-5047b140 21 src/test/java/com/jcabi/ssh/SSHByPasswordTest.java 63-65 30 SSH server setup in this class is nearly identical to the one in {@link SSHTest}. It should be extracted in a separate reusable module. IMP Suseika wlasowegor@gmail.com unknown 21-40f93421 21 src/test/java/com/jcabi/ssh/SSHTest.java 164-167 30 EchoCommand and EchoCommandCreator are duplicated in SSHTest and SSHByPasswordTest, these duplicates should be extracted into package com.jcabi.ssh.mock under src/main/java, renamed to MkCommandCreator/MkCommand and unit-tested. IMP Suseika wlasowegor@gmail.com 62 30-15484dec 30 src/main/java/com/jcabi/ssh/SSH.java 72-74 30 Refactor this class into smaller ones to avoid null checking of passphrase. There should probably be separate classes for encrypted/unencrypted private key. IMP Martin Macko macko.mar@gmail.com