Skip to content

Commit

Permalink
WIP #1186 refactored JobResults
Browse files Browse the repository at this point in the history
  • Loading branch information
brollb committed Sep 17, 2019
1 parent e6ec471 commit e17beb8
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 7 deletions.
10 changes: 10 additions & 0 deletions src/common/execution/backends/JobResults.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
const BaseExecutor = require('./BaseExecutor');

class JobResults {
constructor(status=BaseExecutor.prototype.CREATED) {
this.status = status;
this.resultHashes = [];
}
}

module.exports = JobResults;
9 changes: 2 additions & 7 deletions src/common/execution/backends/local/Client.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
// TODO: Show an error if not running on the server...
define([
'../BaseExecutor',
'../JobResults',
'blob/BlobClient',
'child_process',
'minimatch',
Expand All @@ -12,6 +13,7 @@ define([
'path',
], function(
BaseExecutor,
JobResults,
BlobClient,
childProcess,
minimatch,
Expand Down Expand Up @@ -375,12 +377,5 @@ define([
}
// - [ ] emit updates on stdout...

class JobResults {
constructor(status) {
this.status = status || LocalExecutor.prototype.CREATED;
this.resultHashes = [];
}
}

return LocalExecutor;
});

0 comments on commit e17beb8

Please sign in to comment.