Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(54)

Unified Diff: tracing/tracing/extras/importer/trace_event_importer_test.html

Issue 2013553002: [Tracing] Export TraceConfig to model.metadata (Closed) Base URL: https://github.com/catapult-project/catapult@master
Patch Set: Address Fadi's comment Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tracing/tracing/extras/importer/trace_event_importer.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tracing/tracing/extras/importer/trace_event_importer_test.html
diff --git a/tracing/tracing/extras/importer/trace_event_importer_test.html b/tracing/tracing/extras/importer/trace_event_importer_test.html
index 98d1b6e7b4e0b9c2702a4ec67c9a2b4b0a252533..9270fd150ee25ea83259c56be487a8a00661817c 100644
--- a/tracing/tracing/extras/importer/trace_event_importer_test.html
+++ b/tracing/tracing/extras/importer/trace_event_importer_test.html
@@ -5300,6 +5300,28 @@ tr.b.unittest.testSuite(function() {
assert.equal(m.metadata[0].value, metadataValue);
});
+ test('importTraceConfigAsMetadata', function() {
+ var traceConfigData = {
+ enable_argument_filter: false,
+ enable_sampling: false,
+ enable_systrace: false,
+ record_mode: 'record-until-full'
+ };
+
+ var eventData = {
+ traceEvents: [
+ { name: 'TraceConfig', args: { value: traceConfigData },
+ pid: 2, ts: 0, tid: 1, ph: 'M', cat: '__metadata' },
+ ],
+ };
+
+ var m = makeModel(eventData);
+ assert.isTrue(m.metadata instanceof Array);
+ assert.equal(m.metadata.length, 1);
+ assert.equal(m.metadata[0].name, 'TraceConfig');
+ assert.equal(m.metadata[0].value, traceConfigData);
+ });
+
test('importMarks', function() {
var eventData = {
traceEvents: [
« no previous file with comments | « tracing/tracing/extras/importer/trace_event_importer.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698