You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2015/06/18 17:59:02 UTC
svn commit: r1686261 -
/commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/FTPClientConfigTest.java
Author: sebb
Date: Thu Jun 18 15:59:02 2015
New Revision: 1686261
URL: http://svn.apache.org/r1686261
Log:
No need for fields to be mutable or package-visible
Modified:
commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/FTPClientConfigTest.java
Modified: commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/FTPClientConfigTest.java
URL: http://svn.apache.org/viewvc/commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/FTPClientConfigTest.java?rev=1686261&r1=1686260&r2=1686261&view=diff
==============================================================================
--- commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/FTPClientConfigTest.java (original)
+++ commons/proper/net/trunk/src/test/java/org/apache/commons/net/ftp/FTPClientConfigTest.java Thu Jun 18 15:59:02 2015
@@ -39,12 +39,12 @@ public class FTPClientConfigTest extends
assertNull(config.getServerLanguageCode());
}
- String A = "A";
- String B = "B";
- String C = "C";
- String D = "D";
- String E = "E";
- String F = "F";
+ private static final String A = "A";
+ private static final String B = "B";
+ private static final String C = "C";
+ private static final String D = "D";
+ private static final String E = "E";
+ private static final String F = "F";
/*
* Class under test for void FTPClientConfig(String, String, String, String, String, String)
@@ -61,10 +61,10 @@ public class FTPClientConfigTest extends
}
- String badDelim = "jan,feb,mar,apr,may,jun,jul,aug.sep,oct,nov,dec";
- String tooLong = "jan|feb|mar|apr|may|jun|jul|aug|sep|oct|nov|dec|jan";
- String tooShort = "jan|feb|mar|apr|may|jun|jul|aug|sep|oct|nov";
- String fakeLang = "abc|def|ghi|jkl|mno|pqr|stu|vwx|yza|bcd|efg|hij";
+ private static final String badDelim = "jan,feb,mar,apr,may,jun,jul,aug.sep,oct,nov,dec";
+ private static final String tooLong = "jan|feb|mar|apr|may|jun|jul|aug|sep|oct|nov|dec|jan";
+ private static final String tooShort = "jan|feb|mar|apr|may|jun|jul|aug|sep|oct|nov";
+ private static final String fakeLang = "abc|def|ghi|jkl|mno|pqr|stu|vwx|yza|bcd|efg|hij";
public void testSetShortMonthNames() {
}