summaryrefslogtreecommitdiff
path: root/tests/Codegen/src/com/android/codegentest/SampleDataClassTest.java
diff options
context:
space:
mode:
authorTej Singh <singhtejinder@google.com>2019-09-24 19:12:41 -0700
committerTej Singh <singhtejinder@google.com>2019-09-24 19:19:55 -0700
commit3ce441f63c1bb90f356c801f11dc08e341294ae9 (patch)
tree398792887a53295dccd01124528854b2e0a6b5c4 /tests/Codegen/src/com/android/codegentest/SampleDataClassTest.java
parent4e13461ef96281c2fabfbc29a8a8c609d629c039 (diff)
Statsd unit tests: clear data on disk after tests
A couple of tests had data stored to disk, but were not reading the data. This meant that the data would persist across tests (and between tests runs and forever). This causes issues when conbined with the fact that in tests, we use ConfigKey without initializing it. This means that the uid/id of the config key will just be set to some random stack space. In Christine's case, adding State to the StatsdConfig.proto caused the "random" stack space to be the same as the one in ConfigTttl_e2e_test, which also wrote data to disk. Then, when the duration metric test called onDumpReport, it would grab the data on disk (from ConfigTtl, which used a count metric), in addition to the actual data we cared about, Adding the logic to convert the output to base64 string likely changed which stack space was allocated, meaning the config ttl data would not be pulled, thereby destroying our debugging efforts. Test: unit tests pass, nothing is written to /data/misc/stats-data after running unit tests. Test: unit tests pass with Christine's proto change to add State to StatsdConfig Change-Id: Ica987b7a1c089dae6c45d0500bf6557fa7402191
Diffstat (limited to 'tests/Codegen/src/com/android/codegentest/SampleDataClassTest.java')
0 files changed, 0 insertions, 0 deletions