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

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: 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..20eb115d42e2dc4ad9d22727c5741c9274a56fe2 100644
--- a/tracing/tracing/extras/importer/trace_event_importer_test.html
+++ b/tracing/tracing/extras/importer/trace_event_importer_test.html
@@ -5300,6 +5300,26 @@ tr.b.unittest.testSuite(function() {
assert.equal(m.metadata[0].value, metadataValue);
});
+ test('importTraceConfigAsMetadata', function() {
+ var traceConfigData = {
+ enable_argument_filter: false,
+ enable_sampling: false
+ };
+
+ var eventData = {
+ traceEvents: [
+ { name: 'TraceConfig', args: {value: traceConfigData },
fmeawad 2016/05/24 21:03:51 Can you add a real example of a current TraceConfi
nednguyen 2016/05/24 21:11:51 This is from a real example: "{"pid":16662,"tid":1
+ pid: 2, ts: 0, tid: 1, ph: 'M' },
+ ],
+ };
+
+ 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