Skip to content

Commit

Permalink
fix UI defect RAC-4416
Browse files Browse the repository at this point in the history
  • Loading branch information
root committed Nov 8, 2017
1 parent b197709 commit baf0fcb
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 3 deletions.
19 changes: 16 additions & 3 deletions lib/task-graph.js
Original file line number Diff line number Diff line change
Expand Up @@ -208,10 +208,15 @@ function taskGraphFactory(
waitingOn: taskData.waitOn,
ignoreFailure: taskData.ignoreFailure
};

if (taskData.taskName) {
var taskName = taskData.taskName;
var contains = taskName.indexOf("\/");
if(contains !== -1){
var nameArray = taskName.split("\/");
taskName = nameArray[nameArray.length - 1];
}
return self.constructTaskObject(
taskData.taskName,
taskName,
taskOverrides,
taskData.optionOverrides,
taskData.label
Expand Down Expand Up @@ -360,7 +365,15 @@ function taskGraphFactory(
assert.arrayOfObject(self.definition.tasks, 'Graph.tasks');
return Promise.map(self.definition.tasks, function(taskData) {
if (!_.has(taskData, 'taskDefinition')) {
return store.getTaskDefinition(taskData.taskName)
var taskName = taskData.taskName;
if(taskName){
var contains = taskName.indexOf("\/");
if(contains !== -1){
var nameArray = taskName.split("\/");
taskName = nameArray[nameArray.length - 1];
}
}
return store.getTaskDefinition(taskName)
.then(function(definition) {
return {
taskDefinition: definition,
Expand Down
7 changes: 7 additions & 0 deletions spec/lib/task-graph-spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -130,6 +130,13 @@ describe('Task Graph', function () {
expect(graph).to.be.an.instanceof(TaskGraph);
});
});
it('should return this', function() {
definitions.graphDefinition.tasks[0].taskName = 'api/workflows/tasks/Task.test';
return TaskGraph.create('domain', { definition: definitions.graphDefinition })
.then(function(graph) {
expect(graph).to.be.an.instanceof(TaskGraph);
});
});

it('should fail on an invalid task definition', function() {
this.sandbox.stub(TaskGraph.prototype, '_validateTaskDefinition')
Expand Down

0 comments on commit baf0fcb

Please sign in to comment.