commit 6cfe30d8947ea362d4ca8a5fe09fcfd28645b3ad
author Aaron Vaage <vaage@google.com> Wed Mar 13 12:14:22 2024 -0700
committer Aaron Vaage <vaage@google.com> Wed Mar 13 12:14:22 2024 -0700
tree 2f58ea09091dcd37cef13a5d72aa59b2a7e25654
parent 6e398c3d3f5005fb39a36b9f711fdbf29acf981c

Trace Redaction - Expand 'scrub ftrace events' test coverage

1. Added integration test that pushes an actual trace through 'scrub ftrace
   events' and makes sure the allow-listed events are not affected.

2. Remove the old serialization tests and replaces them with tests that:

   a: uses TestConfig's DummyFields to communicate and enfore types
      (e.g. field_sfixed32).

   b: cover each values full domain (e.g. min int 32 to max int 64).

Bug: 318576499
Change-Id: Ib1c936289ab769f0d104a487f28c251408696464

Android.bp

diff --git a/Android.bp b/Android.bp
index 8d476ce..3233f92 100644
--- a/Android.bp
+++ b/Android.bp
@@ -12492,6 +12492,7 @@
     srcs: [
         "src/trace_redaction/find_package_uid.cc",
         "src/trace_redaction/populate_allow_lists.cc",
+        "src/trace_redaction/proto_util.cc",
         "src/trace_redaction/prune_package_list.cc",
         "src/trace_redaction/scrub_ftrace_events.cc",
         "src/trace_redaction/scrub_trace_packet.cc",
@@ -12505,6 +12506,7 @@
     name: "perfetto_src_trace_redaction_unittests",
     srcs: [
         "src/trace_redaction/find_package_uid_unittest.cc",
+        "src/trace_redaction/proto_util_unittest.cc",
         "src/trace_redaction/prune_package_list_unittest.cc",
         "src/trace_redaction/scrub_ftrace_events_unittest.cc",
         "src/trace_redaction/scrub_trace_packet_unittest.cc",