Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
G
gitlab-ce
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
Léo-Paul Géneau
gitlab-ce
Commits
861f93e6
Commit
861f93e6
authored
Jan 30, 2019
by
Winnie Hellmann
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Convert noteable_note_spec.js to Vue test utils
parent
14397584
Changes
1
Show whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
70 additions
and
23 deletions
+70
-23
spec/javascripts/notes/components/noteable_note_spec.js
spec/javascripts/notes/components/noteable_note_spec.js
+70
-23
No files found.
spec/javascripts/notes/components/noteable_note_spec.js
View file @
861f93e6
import
$
from
'
jquery
'
;
import
_
from
'
underscore
'
;
import
_
from
'
underscore
'
;
import
Vue
from
'
vue
'
;
import
{
shallowMount
,
createLocalVue
}
from
'
@vue/test-utils
'
;
import
createStore
from
'
~/notes/stores
'
;
import
createStore
from
'
~/notes/stores
'
;
import
issueNote
from
'
~/notes/components/noteable_note.vue
'
;
import
issueNote
from
'
~/notes/components/noteable_note.vue
'
;
import
NoteHeader
from
'
~/notes/components/note_header.vue
'
;
import
UserAvatarLink
from
'
~/vue_shared/components/user_avatar/user_avatar_link.vue
'
;
import
NoteActions
from
'
~/notes/components/note_actions.vue
'
;
import
NoteBody
from
'
~/notes/components/note_body.vue
'
;
import
{
noteableDataMock
,
notesDataMock
,
note
}
from
'
../mock_data
'
;
import
{
noteableDataMock
,
notesDataMock
,
note
}
from
'
../mock_data
'
;
describe
(
'
issue_note
'
,
()
=>
{
describe
(
'
issue_note
'
,
()
=>
{
let
store
;
let
store
;
let
vm
;
let
wrapper
;
beforeEach
(()
=>
{
beforeEach
(()
=>
{
const
Component
=
Vue
.
extend
(
issueNote
);
store
=
createStore
();
store
=
createStore
();
store
.
dispatch
(
'
setNoteableData
'
,
noteableDataMock
);
store
.
dispatch
(
'
setNoteableData
'
,
noteableDataMock
);
store
.
dispatch
(
'
setNotesData
'
,
notesDataMock
);
store
.
dispatch
(
'
setNotesData
'
,
notesDataMock
);
vm
=
new
Component
({
const
localVue
=
createLocalVue
();
wrapper
=
shallowMount
(
issueNote
,
{
store
,
store
,
propsData
:
{
propsData
:
{
note
,
note
,
},
},
}).
$mount
();
sync
:
false
,
localVue
,
});
});
});
afterEach
(()
=>
{
afterEach
(()
=>
{
vm
.
$
destroy
();
wrapper
.
destroy
();
});
});
it
(
'
should render user information
'
,
()
=>
{
it
(
'
should render user information
'
,
()
=>
{
expect
(
vm
.
$el
.
querySelector
(
'
.user-avatar-link img
'
).
getAttribute
(
'
src
'
)).
toEqual
(
const
{
author
}
=
note
;
note
.
author
.
avatar_url
,
const
avatar
=
wrapper
.
find
(
UserAvatarLink
);
);
const
avatarProps
=
avatar
.
props
();
expect
(
avatarProps
.
linkHref
).
toBe
(
author
.
path
);
expect
(
avatarProps
.
imgSrc
).
toBe
(
author
.
avatar_url
);
expect
(
avatarProps
.
imgAlt
).
toBe
(
author
.
name
);
expect
(
avatarProps
.
imgSize
).
toBe
(
40
);
});
});
it
(
'
should render note header content
'
,
()
=>
{
it
(
'
should render note header content
'
,
()
=>
{
const
el
=
vm
.
$el
.
querySelector
(
'
.note-header .note-header-author-name
'
);
const
noteHeader
=
wrapper
.
find
(
NoteHeader
);
const
noteHeaderProps
=
noteHeader
.
props
();
expect
(
el
.
textContent
.
trim
()).
toEqual
(
note
.
author
.
name
);
expect
(
noteHeaderProps
.
author
).
toEqual
(
note
.
author
);
expect
(
noteHeaderProps
.
createdAt
).
toEqual
(
note
.
created_at
);
expect
(
noteHeaderProps
.
noteId
).
toEqual
(
note
.
id
);
});
});
it
(
'
should render note actions
'
,
()
=>
{
it
(
'
should render note actions
'
,
()
=>
{
expect
(
vm
.
$el
.
querySelector
(
'
.note-actions
'
)).
toBeDefined
();
const
{
author
}
=
note
;
const
noteActions
=
wrapper
.
find
(
NoteActions
);
const
noteActionsProps
=
noteActions
.
props
();
expect
(
noteActionsProps
.
authorId
).
toBe
(
author
.
id
);
expect
(
noteActionsProps
.
noteId
).
toBe
(
note
.
id
);
expect
(
noteActionsProps
.
noteUrl
).
toBe
(
note
.
noteable_note_url
);
expect
(
noteActionsProps
.
accessLevel
).
toBe
(
note
.
human_access
);
expect
(
noteActionsProps
.
canEdit
).
toBe
(
note
.
current_user
.
can_edit
);
expect
(
noteActionsProps
.
canAwardEmoji
).
toBe
(
note
.
current_user
.
can_award_emoji
);
expect
(
noteActionsProps
.
canDelete
).
toBe
(
note
.
current_user
.
can_edit
);
expect
(
noteActionsProps
.
canReportAsAbuse
).
toBe
(
true
);
expect
(
noteActionsProps
.
canResolve
).
toBe
(
false
);
expect
(
noteActionsProps
.
reportAbusePath
).
toBe
(
note
.
report_abuse_path
);
expect
(
noteActionsProps
.
resolvable
).
toBe
(
false
);
expect
(
noteActionsProps
.
isResolved
).
toBe
(
false
);
expect
(
noteActionsProps
.
isResolving
).
toBe
(
false
);
expect
(
noteActionsProps
.
resolvedBy
).
toEqual
({});
});
});
it
(
'
should render issue body
'
,
()
=>
{
it
(
'
should render issue body
'
,
()
=>
{
expect
(
vm
.
$el
.
querySelector
(
'
.note-text
'
).
innerHTML
).
toEqual
(
note
.
note_html
);
const
noteBody
=
wrapper
.
find
(
NoteBody
);
const
noteBodyProps
=
noteBody
.
props
();
expect
(
noteBodyProps
.
note
).
toEqual
(
note
);
expect
(
noteBodyProps
.
line
).
toBe
(
null
);
expect
(
noteBodyProps
.
canEdit
).
toBe
(
note
.
current_user
.
can_edit
);
expect
(
noteBodyProps
.
isEditing
).
toBe
(
false
);
expect
(
noteBodyProps
.
helpPagePath
).
toBe
(
''
);
});
});
it
(
'
prevents note preview xss
'
,
done
=>
{
it
(
'
prevents note preview xss
'
,
done
=>
{
const
imgSrc
=
'
data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7
'
;
const
imgSrc
=
'
data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7
'
;
const
noteBody
=
`<img src="
${
imgSrc
}
" onload="alert(1)" />`
;
const
noteBody
=
`<img src="
${
imgSrc
}
" onload="alert(1)" />`
;
const
alertSpy
=
spyOn
(
window
,
'
alert
'
);
const
alertSpy
=
spyOn
(
window
,
'
alert
'
);
vm
.
updateNote
=
()
=>
new
Promise
(
$
.
noop
);
store
.
hotUpdate
({
actions
:
{
updateNote
()
{},
},
});
const
noteBodyComponent
=
wrapper
.
find
(
NoteBody
);
vm
.
formUpdateHandler
(
noteBody
,
null
,
$
.
noop
);
noteBodyComponent
.
vm
.
$emit
(
'
handleFormUpdate
'
,
noteBody
,
null
,
()
=>
{}
);
setTimeout
(()
=>
{
setTimeout
(()
=>
{
expect
(
alertSpy
).
not
.
toHaveBeenCalled
();
expect
(
alertSpy
).
not
.
toHaveBeenCalled
();
expect
(
vm
.
note
.
note_html
).
toEqual
(
_
.
escape
(
noteBody
));
expect
(
wrapper
.
vm
.
note
.
note_html
).
toEqual
(
_
.
escape
(
noteBody
));
done
();
done
();
},
0
);
},
0
);
});
});
...
@@ -66,17 +107,23 @@ describe('issue_note', () => {
...
@@ -66,17 +107,23 @@ describe('issue_note', () => {
describe
(
'
cancel edit
'
,
()
=>
{
describe
(
'
cancel edit
'
,
()
=>
{
it
(
'
restores content of updated note
'
,
done
=>
{
it
(
'
restores content of updated note
'
,
done
=>
{
const
noteBody
=
'
updated note text
'
;
const
noteBody
=
'
updated note text
'
;
vm
.
updateNote
=
()
=>
Promise
.
resolve
();
store
.
hotUpdate
({
actions
:
{
updateNote
()
{},
},
});
const
noteBodyComponent
=
wrapper
.
find
(
NoteBody
);
noteBodyComponent
.
vm
.
resetAutoSave
=
()
=>
{};
vm
.
formUpdateHandler
(
noteBody
,
null
,
$
.
noop
);
noteBodyComponent
.
vm
.
$emit
(
'
handleFormUpdate
'
,
noteBody
,
null
,
()
=>
{}
);
setTimeout
(()
=>
{
setTimeout
(()
=>
{
expect
(
vm
.
note
.
note_html
).
toEqual
(
noteBody
);
expect
(
wrapper
.
vm
.
note
.
note_html
).
toEqual
(
noteBody
);
vm
.
formCancelHandler
(
);
noteBodyComponent
.
vm
.
$emit
(
'
cancelForm
'
);
setTimeout
(()
=>
{
setTimeout
(()
=>
{
expect
(
vm
.
note
.
note_html
).
toEqual
(
noteBody
);
expect
(
wrapper
.
vm
.
note
.
note_html
).
toEqual
(
noteBody
);
done
();
done
();
});
});
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment