现在是我编写第一个测试的时候了。我很兴奋,但是我被困住了。我有方法,应该验证已安装的Java版本
public static boolean verifyJavaVersion() throws UserException {
if (System.getProperty("java.version") != null) {
String[] javaProperty = System.getProperty("java.version").substring(2, System.getProperty("java.version").length()).split("\\.");
int installedJavaVersion = Integer.parseInt(javaProperty[0]);
int installedJavaReleaseVersion = Integer.parseInt(javaProperty[1].substring(2, javaProperty[1].length()));
if (installedJavaVersion != REQUIRED_JAVA_VERSION || installedJavaReleaseVersion < MINIMUM_JAVA_RELEASE_VERSION)
throw new UserException("Java version is not correct. Required Java version "
+ REQUIRED_JAVA_VERSION + ", release " + MINIMUM_JAVA_RELEASE_VERSION + " or higher");
return true;
}
throw new UserException("Java version not found");
}
我有2个变量,它们都是private static final
,所以,正如你所看到的,这个方法中没有参数。我如何测试这种方法,我应该测试任何期望改变这2个变量的东西吗?
我有一个想法是创建公共getter和setter来测试这些变量,但这是一个好习惯吗?
答案 0 :(得分:3)
认识到编写测试很难,通常意味着可以改进代码。对我而言,这是编写测试的最大原因之一:它们迫使您编写更好的代码!
如果你看一下你的代码片段,它实际上做了几件事:
当考虑到这一点时,您会发现这可以分成几个功能。 他们可以更容易地进行测试。
public static boolean verifyJavaVersion() throws UserException {
String javaVersion = ...
int installedJavaVersion = extractJavaVersion(javaVersion);
int installedJavaReleaseVersion = extractJavaReleaseVersion(javaVersion);
checkVersion(installedJavaVersion, REQUIRED_JAVA_VERSION, installedJavaReleaseVersion, MINIMUM_JAVA_RELEASE_VERSION);
}
每个都应该易于测试。
为了简单起见,我删除了您的空检查,但您看到了这个想法。