Skip to content

Commit 2628bd7

Browse files
authored
Merge pull request #47 from mathworks/metrics_readonly_props
add a test about issue #34
2 parents 66b73f2 + b90544a commit 2628bd7

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

test/tmetrics.m

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,19 +61,23 @@ function testDefaultReader(testCase)
6161
verifyEqual(testCase, string(class(reader.MetricExporter)), ...
6262
"opentelemetry.exporters.otlp.OtlpHttpMetricExporter");
6363
verifyEqual(testCase, reader.Interval, minutes(1));
64+
verifyEqual(testCase, reader.Interval.Format, 'm');
6465
verifyEqual(testCase, reader.Timeout, seconds(30));
66+
verifyEqual(testCase, reader.Timeout.Format, 's');
6567
end
6668

6769

6870
function testReaderBasic(testCase)
6971
exporter = opentelemetry.exporters.otlp.defaultMetricExporter;
70-
interval = seconds(1);
71-
timeout = seconds(0.5);
72+
interval = hours(1);
73+
timeout = minutes(30);
7274
reader = opentelemetry.sdk.metrics.PeriodicExportingMetricReader(exporter, ...
7375
"Interval", interval, ...
7476
"Timeout", timeout);
7577
verifyEqual(testCase, reader.Interval, interval);
78+
verifyEqual(testCase, reader.Interval.Format, 'h'); % should not be converted to other units
7679
verifyEqual(testCase, reader.Timeout, timeout);
80+
verifyEqual(testCase, reader.Timeout.Format, 'm');
7781
end
7882

7983

0 commit comments

Comments
 (0)