Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
李少辉-开发者
gitlab-foss
提交
e64c9c21
G
gitlab-foss
项目概览
李少辉-开发者
/
gitlab-foss
通知
15
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
G
gitlab-foss
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
未验证
提交
e64c9c21
编写于
12月 14, 2018
作者:
P
Phil Hughes
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Fixed duplicated inline diff discussions
Closes
https://gitlab.com/gitlab-org/gitlab-ce/issues/55388
上级
9714dfdf
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
66 addition
and
1 deletion
+66
-1
app/assets/javascripts/diffs/store/mutations.js
app/assets/javascripts/diffs/store/mutations.js
+1
-1
spec/javascripts/diffs/store/mutations_spec.js
spec/javascripts/diffs/store/mutations_spec.js
+65
-0
未找到文件。
app/assets/javascripts/diffs/store/mutations.js
浏览文件 @
e64c9c21
...
...
@@ -138,7 +138,7 @@ export default {
if
(
file
.
highlighted_diff_lines
)
{
file
.
highlighted_diff_lines
=
file
.
highlighted_diff_lines
.
map
(
line
=>
mapDiscussions
(
line
)
,
lineCheck
(
line
)
?
mapDiscussions
(
line
)
:
line
,
);
}
...
...
spec/javascripts/diffs/store/mutations_spec.js
浏览文件 @
e64c9c21
...
...
@@ -358,6 +358,71 @@ describe('DiffsStoreMutations', () => {
expect
(
state
.
diffFiles
[
0
].
highlighted_diff_lines
[
0
].
discussions
[
0
].
resolved
).
toBe
(
true
);
});
it
(
'
should not duplicate inline diff discussions
'
,
()
=>
{
const
diffPosition
=
{
base_sha
:
'
ed13df29948c41ba367caa757ab3ec4892509910
'
,
head_sha
:
'
b921914f9a834ac47e6fd9420f78db0f83559130
'
,
new_line
:
null
,
new_path
:
'
500-lines-4.txt
'
,
old_line
:
5
,
old_path
:
'
500-lines-4.txt
'
,
start_sha
:
'
ed13df29948c41ba367caa757ab3ec4892509910
'
,
};
const
state
=
{
latestDiff
:
true
,
diffFiles
:
[
{
file_hash
:
'
ABC
'
,
highlighted_diff_lines
:
[
{
line_code
:
'
ABC_1
'
,
discussions
:
[
{
id
:
1
,
line_code
:
'
ABC_1
'
,
diff_discussion
:
true
,
resolvable
:
true
,
original_position
:
diffPosition
,
position
:
diffPosition
,
diff_file
:
{
file_hash
:
'
ABC
'
,
},
},
],
},
{
line_code
:
'
ABC_2
'
,
discussions
:
[],
},
],
},
],
};
const
discussion
=
{
id
:
2
,
line_code
:
'
ABC_2
'
,
diff_discussion
:
true
,
resolvable
:
true
,
original_position
:
diffPosition
,
position
:
diffPosition
,
diff_file
:
{
file_hash
:
state
.
diffFiles
[
0
].
file_hash
,
},
};
const
diffPositionByLineCode
=
{
ABC_2
:
diffPosition
,
};
mutations
[
types
.
SET_LINE_DISCUSSIONS_FOR_FILE
](
state
,
{
discussion
,
diffPositionByLineCode
,
});
expect
(
state
.
diffFiles
[
0
].
highlighted_diff_lines
[
0
].
discussions
.
length
).
toBe
(
1
);
});
it
(
'
should add legacy discussions to the given line
'
,
()
=>
{
const
diffPosition
=
{
base_sha
:
'
ed13df29948c41ba367caa757ab3ec4892509910
'
,
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录