Good catches, Aaron!
--On Friday, September 2, 2005 7:28 PM -1000 Aaron Kagawa
<[EMAIL PROTECTED]> wrote:
1) I think the test cases download jar files from
http://hackystat.ics.hawaii.edu instead of http://localhost:8080 (or what
ever is set in the properties).
Oooh. That's not good. Should be fixed.
2) I noticed that my sensor.properties
file is "touched" by hackyInstaller testcases. Does that mean it could
potentially change it? Or that it is actually being used in the test
cases? In either case, that is probably bad.
Oooh. That's really not good. The test cases should not touch the user's
actual sensor.properties file. You'll need to construct a 'fake' one for
testing purposes.
Side Note:
I noticed hackyinstaller.xml file, which contains ANT_HOME. It showed an
Ant version I don't use. I upgraded ANT and changed my ANT_HOME a while
ago in my environment variables. So, if I didn't know to check my ANT
HOME in hackyInstaller then my sensors could be broken. Anyway, the
point is that could it be possible to use the ANT_HOME in my ENV instead
of manually setting it.
This we can't really do anything about; the hackyinstaller "ANT_HOME" is
only coincidentally named the same as one's ANT_HOME environment variable.
(And anyway, what are you doing looking hackyinstaller.xml :-)
But I will add something to the documentation that if people move their
installation directory, they have to run hackyInstaller and update the
location.
Cheers,
Philip
