Spec for introducing null reports and source registration time

configuration

Author: linnan-github

header-validator/data.trigger.js

@@ -12,6 +12,7 @@ export const testCases = [
         "filters": {"x": []},
         "not_filters": {"y": []}
       }],
+      "aggregatable_source_registration_time": "include",
       "aggregatable_trigger_data": [{
         "filters": {"a": ["b"]},
         "key_piece": "0x1",
@@ -527,4 +528,21 @@ export const testCases = [
       msg: "must match 'aws-cloud' (case-sensitive)",
     }],
   },
+
+  {
+    name: "aggregatable-source-registrationt-time-wrong-type",
+    json: `{"aggregatable_source_registration_time": 1}`,
+    expectedErrors: [{
+      path: ["aggregatable_source_registration_time"],
+      msg: "must be a string",
+    }],
+  },
+  {
+    name: "aggregatable-source-registration-time-unknown-value",
+    json: `{"aggregatable_source_registration_time": "OMIT"}`,
+    expectedErrors: [{
+      path: ["aggregatable_source_registration_time"],
+      msg: "must match 'omit' or 'include' (case-sensitive)",
+    }],
+  },
 ];

header-validator/validate-json.js

@@ -301,6 +301,15 @@ const aggregatableDedupKeys = list(
       not_filters: optional(filters()),
     }))
 
+const aggregatableSourceRegistrationTime = string((state, value) => {
+  const omit = 'omit'
+  const include = 'include'
+  if (value === omit || value === include) {
+    return
+  }
+  state.error(`must match '${omit}' or '${include}' (case-sensitive)`)
+})
+
 export function validateTrigger(trigger) {
   const state = new State()
   state.validate(trigger, {
@@ -313,6 +322,7 @@ export function validateTrigger(trigger) {
     filters: optional(orFilters),
     not_filters: optional(orFilters),
     aggregatable_deduplication_keys: optional(aggregatableDedupKeys),
+    aggregatable_source_registration_time : optional(aggregatableSourceRegistrationTime),
   })
   return state.result()
 }