Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

Commit 828e80c

Browse files
vedharishcnishina
authored andcommitted
fix(browserstack): mark test suite as failed/passed on BrowserStack (#3409)
closes #3256
1 parent 71532f0 commit 828e80c

File tree

1 file changed

+15
-20
lines changed

1 file changed

+15
-20
lines changed

lib/driverProviders/browserStack.ts

+15-20
Original file line numberDiff line numberDiff line change
@@ -67,26 +67,21 @@ export class BrowserStack extends DriverProvider {
6767
});
6868
let jobStatus = update.passed ? 'completed' : 'error';
6969
options.method = 'PUT';
70-
https
71-
.request(
72-
options,
73-
(res) => {
74-
let responseStr = '';
75-
res.on('data', (data: Buffer) => {
76-
responseStr += data.toString();
77-
});
78-
res.on('end', () => {
79-
logger.info(responseStr);
80-
deferred.resolve();
81-
});
82-
res.on('error', (e: Error) => {
83-
throw new BrowserError(
84-
logger,
85-
'Error updating BrowserStack pass/fail status: ' +
86-
util.inspect(e));
87-
});
88-
})
89-
.write('{\'status\': ' + jobStatus + '}');
70+
let update_req = https.request(options, (res) => {
71+
let responseStr = '';
72+
res.on('data', (data: Buffer) => { responseStr += data.toString(); });
73+
res.on('end', () => {
74+
logger.info(responseStr);
75+
deferred.resolve();
76+
});
77+
res.on('error', (e: Error) => {
78+
throw new BrowserError(
79+
logger, 'Error updating BrowserStack pass/fail status: ' +
80+
util.inspect(e));
81+
});
82+
});
83+
update_req.write('{"status":"' + jobStatus + '"}');
84+
update_req.end();
9085
});
9186
return deferred.promise;
9287
});

0 commit comments

Comments
 (0)