Skip to content

Commit

Permalink
test: relax a property check in system tests (#1133)
Browse files Browse the repository at this point in the history
  • Loading branch information
kjin authored Oct 2, 2019
1 parent d3f7d16 commit a3622d1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions system-test/trace-express.js
Original file line number Diff line number Diff line change
Expand Up @@ -144,9 +144,9 @@ describe('express + datastore', () => {
assert.ok(trace.spans.length >= 2, 'should be at least 2 spans: parent, child');
const parent = trace.spans[0];
const child = trace.spans.find(span => {
const datastoreUrl = `https://datastore.googleapis.com/${EXPECTED_ENDPOINT}`;
const urlLabelValue = span.labels[tracer.labels.HTTP_URL_LABEL_KEY];
return span.name === `grpc:/${EXPECTED_ENDPOINT}` ||
span.labels[tracer.labels.HTTP_URL_LABEL_KEY] === datastoreUrl;
(urlLabelValue && urlLabelValue.endsWith(EXPECTED_ENDPOINT));
});

assert.strictEqual(parent.name, testPath, 'should match unique path');
Expand Down

0 comments on commit a3622d1

Please sign in to comment.