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
1
Merge Requests
1
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
nexedi
gitlab-ce
Commits
aedaef2b
Commit
aedaef2b
authored
Dec 17, 2018
by
Paul Slaughter
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Update callout component to receive slot
**Why?** - This is needed to add buttons to the callout
parent
bc69b934
Changes
2
Show whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
46 additions
and
24 deletions
+46
-24
app/assets/javascripts/vue_shared/components/callout.vue
app/assets/javascripts/vue_shared/components/callout.vue
+3
-2
spec/javascripts/vue_shared/components/callout_spec.js
spec/javascripts/vue_shared/components/callout_spec.js
+43
-22
No files found.
app/assets/javascripts/vue_shared/components/callout.vue
View file @
aedaef2b
...
...
@@ -11,13 +11,14 @@ export default {
},
message
:
{
type
:
String
,
required
:
true
,
required
:
false
,
default
:
''
,
},
},
};
</
script
>
<
template
>
<div
:class=
"`bs-callout bs-callout-$
{category}`" role="alert" aria-live="assertive">
{{
message
}}
{{
message
}}
<slot></slot>
</div>
</
template
>
spec/javascripts/vue_shared/components/callout_spec.js
View file @
aedaef2b
import
Vue
from
'
vue
'
;
import
callout
from
'
~/vue_shared/components/callout.vue
'
;
import
createComponent
from
'
spec/helpers/vue_mount_component_helper
'
;
import
{
createLocalVue
,
shallowMount
}
from
'
@vue/test-utils
'
;
import
Callout
from
'
~/vue_shared/components/callout.vue
'
;
const
TEST_MESSAGE
=
'
This is a callout message!
'
;
const
TEST_SLOT
=
'
<button>This is a callout slot!</button>
'
;
const
localVue
=
createLocalVue
();
describe
(
'
Callout Component
'
,
()
=>
{
let
CalloutComponent
;
let
vm
;
const
exampleMessage
=
'
This is a callout message!
'
;
let
wrapper
;
beforeEach
(()
=>
{
CalloutComponent
=
Vue
.
extend
(
callout
);
const
factory
=
options
=>
{
wrapper
=
shallowMount
(
localVue
.
extend
(
Callout
),
{
localVue
,
...
options
,
});
};
afterEach
(()
=>
{
vm
.
$
destroy
();
wrapper
.
destroy
();
});
it
(
'
should render the appropriate variant of callout
'
,
()
=>
{
vm
=
createComponent
(
CalloutComponent
,
{
factory
({
propsData
:
{
category
:
'
info
'
,
message
:
exampleMessage
,
message
:
TEST_MESSAGE
,
},
});
expect
(
vm
.
$el
.
getAttribute
(
'
class
'
)).
toEqual
(
'
bs-callout bs-callout-info
'
);
expect
(
wrapper
.
classes
()).
toEqual
([
'
bs-callout
'
,
'
bs-callout-info
'
]
);
expect
(
vm
.
$el
.
tagName
).
toEqual
(
'
DIV
'
);
expect
(
wrapper
.
element
.
tagName
).
toEqual
(
'
DIV
'
);
});
it
(
'
should render accessibility attributes
'
,
()
=>
{
vm
=
createComponent
(
CalloutComponent
,
{
message
:
exampleMessage
,
factory
({
propsData
:
{
message
:
TEST_MESSAGE
,
},
});
expect
(
vm
.
$el
.
getAttribute
(
'
role
'
)).
toEqual
(
'
alert
'
);
expect
(
vm
.
$el
.
getAttribute
(
'
aria-live
'
)).
toEqual
(
'
assertive
'
);
expect
(
wrapper
.
attributes
(
'
role
'
)).
toEqual
(
'
alert
'
);
expect
(
wrapper
.
attributes
(
'
aria-live
'
)).
toEqual
(
'
assertive
'
);
});
it
(
'
should render the provided message
'
,
()
=>
{
vm
=
createComponent
(
CalloutComponent
,
{
message
:
exampleMessage
,
factory
({
propsData
:
{
message
:
TEST_MESSAGE
,
},
});
expect
(
wrapper
.
element
.
innerHTML
.
trim
()).
toEqual
(
TEST_MESSAGE
);
});
it
(
'
should render the provided slot
'
,
()
=>
{
factory
({
slots
:
{
default
:
TEST_SLOT
,
},
});
expect
(
vm
.
$el
.
innerHTML
.
trim
()).
toEqual
(
exampleMessage
);
expect
(
wrapper
.
element
.
innerHTML
.
trim
()).
toEqual
(
TEST_SLOT
);
});
});
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