Minor corrections

aalhossary · aalhossary · commit 06cde1b62115 · 2022-12-24T08:18:03.000-05:00
Typos, wording, and version correction
diff --git a/biojava-core/src/main/java/org/biojava/nbio/core/util/FileDownloadUtils.java b/biojava-core/src/main/java/org/biojava/nbio/core/util/FileDownloadUtils.java
@@ -195,7 +195,7 @@ public static void createValidationFiles(URL url, File localDestination, URL has
 	 * @param resourceUrlConnection the remote file URLConnection to download
 	 * @param localDestination the local file to download into
 	 * @param hashURL the URL of the hash file to download. Can be <code>null</code>.
-	 * @since 6.0.6
+	 * @since 6.1.1
 	 */
 	public static void createValidationFiles(URLConnection resourceUrlConnection, File localDestination, URL hashURL){
 		long size = resourceUrlConnection.getContentLengthLong();
@@ -234,8 +234,8 @@ public static void createValidationFiles(URLConnection resourceUrlConnection, Fi
 	 * <p>
 	 * This function does not implement hash code verification yet.
 	 * @param localFile The file to validate
-	 * @return <code>false</code> if any of the size or hash code metadata files exists but its contents does not match the expected value in the file, <code>true'</code> otherwise.
-	 * @since 6.0.6
+	 * @return <code>false</code> if any of the size or hash code metadata files exists but its contents does not match the expected value in the file, <code>true</code> otherwise.
+	 * @since 6.1.1
 	 */
 	public static boolean validateFile(File localFile) {
 		File sizeFile = new File(localFile.getParentFile(), localFile.getName()+".size");
diff --git a/biojava-core/src/test/java/org/biojava/nbio/core/util/FileDownloadUtilsTest.java b/biojava-core/src/test/java/org/biojava/nbio/core/util/FileDownloadUtilsTest.java
@@ -214,7 +214,7 @@ void testValidationFiles() throws IOException{
     		FileDownloadUtils.downloadFile(sourceUrl, destFile);
     		assertTrue(destFile.exists(), "couldn't create dest file");
 
-    		assertTrue(FileDownloadUtils.validateFile(destFile), "file not detected to be invalid although there are no validation files");
+    		assertTrue(FileDownloadUtils.validateFile(destFile), "file detected to be invalid although there are no validation files");
 
     		PrintStream temp1 = new PrintStream(sizeFile);
     		temp1.print(15); // some wrong size value
@@ -232,7 +232,7 @@ void testValidationFiles() throws IOException{
     		//This is not yet implemented. I am using this test for documentation purpose.
     		assertThrows(UnsupportedOperationException.class, 
     				() -> FileDownloadUtils.validateFile(destFile), 
-    				"file not detected to be invalid although size value is wrong.");
+    				"file not detected to be invalid although hash value is wrong.");
     		
     		destFile.delete();
     		sizeFile.delete();
diff --git a/biojava-structure/src/main/java/org/biojava/nbio/structure/ecod/EcodFactory.java b/biojava-structure/src/main/java/org/biojava/nbio/structure/ecod/EcodFactory.java
@@ -89,8 +89,8 @@ public static EcodDatabase getEcodDatabase(String version) {
 					}
 				} catch (IOException e) {
 					// For parsing errors, just use the requested version
-					// What about corrupted downloading errors?? Amr
-					logger.warn("Cound not get Ecod version, or file is corrupted", e);
+					// TODO What about corrupted downloading errors?? Amr
+					logger.warn("Could not get Ecod version, or file is corrupted", e);
 					return null;
 				}
 			}

Original file line number	Diff line number	Diff line change
`@@ -89,8 +89,8 @@ public static EcodDatabase getEcodDatabase(String version) {`
`89`	`89`	`}`
`90`	`90`	`} catch (IOException e) {`
`91`	`91`	`// For parsing errors, just use the requested version`
`92`		`- // What about corrupted downloading errors?? Amr`
`93`		`- logger.warn("Cound not get Ecod version, or file is corrupted", e);`
	`92`	`+ // TODO What about corrupted downloading errors?? Amr`
	`93`	`+ logger.warn("Could not get Ecod version, or file is corrupted", e);`
`94`	`94`	`return null;`
`95`	`95`	`}`
`96`	`96`	`}`