I did some work on evolving the ReviewActivity SDT today, and I thought it might be
helpful for you to hear how it went.
If you want to look at the current state, do an SVN switch to:
svn://hackydev.ics.hawaii.edu/hackystat/branches/HACK-631
Modules changed were: hackySdt_ReviewActivity and hackyApp_Review.
The process I followed was the following:
(a) After doing the branch, I did a freshStart all.junit on all related modules to make
sure I had a working branch.
(b) I then started work on evolving the SDT. In this case, all I needed to do was to
eliminate the "data" field and move everything in it to the pMap. That involved deleting
some methods (getData, getPropertyMap), and editing their callers to use the built-in
pMap accessor (getProperty). I also had to implement the reorganizeData method, which
checked for the presence of the data field and if found, extracted the key-value pairs
and moved them into the pMap. Finally, I had to update the test code to check for pMap
values when 'data' fields were constructed.
(c) I then ran freshStart all.junit to make sure everything compiled and tested OK, and
found a couple of bugs.
(d) I committed the code to the branch.
What remains is to update hackySensor_Jupiter to generate sensor data using the pMap
field. I'll work on that tomorrow. Then this should be ready for a code review, and
following that, merging back into the trunk.
Cheers,
Philip
