Details
-
Improvement
-
Status: Resolved
-
Minor
-
Resolution: Fixed
-
None
-
None
Description
Right now in AutomatedTestBase class, readDMLMatrixFromHDFS() method reads a file from OUTPUT_DIR, readDMLScalarFromHDFS() method reads a file from OUTPUT_DIR, readRMatrixFromFS() method reads a file from EXPECTED_DIR, and readRScalarFromFS() method reads a file from EXPECTED_DIR.
Changing method names so that they reflect where the read is coming from (OUTPUT_DIR or EXPECTED_DIR) would be very helpful in terms of visual comprehension of the test code so that the internals of the methods don't need to be looked at to determine functionality. Additionally, the mention of HDFS and FS is not necessarily relevant.
Potential name changes:
readDMLMatrixFromHDFS --> readDMLMatrixFromOutputDir
readDMLScalarFromHDFS --> readDMLScalarFromOutputDir
readRMatrixFromFS --> readRMatrixFromExpectedDir
readRScalarFromFS --> readRScalarFromExpectedDir