diff --git a/app/assets/javascripts/boards/models/issue.js.es6 b/app/assets/javascripts/boards/models/issue.js.es6 index e16d434121254652c232ed7618787c48dffe6fc5..1af3ded386b44bbfef862e3e4c65f9e5bb3bc787 100644 --- a/app/assets/javascripts/boards/models/issue.js.es6 +++ b/app/assets/javascripts/boards/models/issue.js.es6 @@ -1,17 +1,17 @@ -class Issue { +class ListIssue { constructor (obj) { this.id = obj.iid; this.title = obj.title; this.confidential = obj.confidential; if (obj.assignee) { - this.assignee = new User(obj.assignee); + this.assignee = new ListUser(obj.assignee); } this.labels = []; _.each(obj.labels, (label) => { - this.labels.push(new Label(label)); + this.labels.push(new ListLabel(label)); }); this.priority = _.reduce(this.labels, (max, label) => { @@ -24,7 +24,7 @@ class Issue { const hasLabel = this.findLabel(label); if (!hasLabel) { - this.labels.push(new Label(label)); + this.labels.push(new ListLabel(label)); } } } diff --git a/app/assets/javascripts/boards/models/label.js.es6 b/app/assets/javascripts/boards/models/label.js.es6 index 974e59d725e2e3d5afd5de1ba929c99a30b5d38a..e81e91fe972f0d9f528cbfe7ea92d40f293d1564 100644 --- a/app/assets/javascripts/boards/models/label.js.es6 +++ b/app/assets/javascripts/boards/models/label.js.es6 @@ -1,4 +1,4 @@ -class Label { +class ListLabel { constructor (obj) { this.id = obj.id; this.title = obj.title; diff --git a/app/assets/javascripts/boards/models/list.js.es6 b/app/assets/javascripts/boards/models/list.js.es6 index 60214e2c79899b96ec02f327d061301079ea096f..662ab7354d481960b24216f4fe617d2830c6fd3d 100644 --- a/app/assets/javascripts/boards/models/list.js.es6 +++ b/app/assets/javascripts/boards/models/list.js.es6 @@ -10,7 +10,7 @@ class List { this.issues = []; if (obj.label) { - this.label = new Label(obj.label); + this.label = new ListLabel(obj.label); } if (this.type !== 'blank' && this.id) { @@ -85,8 +85,8 @@ class List { } createIssues (data) { - _.each(data, (issue) => { - this.issues.push(new Issue(issue)); + _.each(data, (issueObj) => { + this.issues.push(new ListIssue(issueObj)); }); } diff --git a/app/assets/javascripts/boards/models/user.js.es6 b/app/assets/javascripts/boards/models/user.js.es6 index e6b35569f77d0a161b36696fa7a195a13b00cfb1..904b3a68507a3d1e897cc8cb938bfad6001198ea 100644 --- a/app/assets/javascripts/boards/models/user.js.es6 +++ b/app/assets/javascripts/boards/models/user.js.es6 @@ -1,4 +1,4 @@ -class User { +class ListUser { constructor (user) { this.id = user.id; this.name = user.name; diff --git a/spec/javascripts/boards/boards_store_spec.js.es6 b/spec/javascripts/boards/boards_store_spec.js.es6 index c3c979a2a7f121000c4ca365e63a200aab562bf3..ef59bac6e56d4c60fccb36859a130b480953c734 100644 --- a/spec/javascripts/boards/boards_store_spec.js.es6 +++ b/spec/javascripts/boards/boards_store_spec.js.es6 @@ -54,6 +54,7 @@ setTimeout(() => { expect(list.issues.length).toBe(1); + expect(list.issues[0].id).toBe(1); done(); }, 0); }); @@ -135,10 +136,10 @@ expect(BoardsStore.state.lists.length).toBe(2); - setTimeout(() => { - const list = BoardsStore.findList('id', 1); - const listTwo = BoardsStore.findList('id', 2); + const list = BoardsStore.findList('id', 1), + listTwo = BoardsStore.findList('id', 2); + setTimeout(() => { expect(list.issues.length).toBe(1); expect(listTwo.issues.length).toBe(1); @@ -148,7 +149,7 @@ expect(listTwo.issues.length).toBe(1); done(); - }); + }, 0); }); }); }); diff --git a/spec/javascripts/boards/issue_spec.js.es6 b/spec/javascripts/boards/issue_spec.js.es6 index b31bd74302a05615effe0724977eef6831c11943..b53be896872af5c13f365d9dc3b05445f821adc3 100644 --- a/spec/javascripts/boards/issue_spec.js.es6 +++ b/spec/javascripts/boards/issue_spec.js.es6 @@ -19,7 +19,7 @@ describe('Issue model', () => { gl.boardService = new BoardService('/test/issue-boards/board'); BoardsStore.create(); - issue = new Issue({ + issue = new ListIssue({ title: 'Testing', iid: 1, confidential: false,