Skip to content
This repository was archived by the owner on May 3, 2022. It is now read-only.

Commit 357846e

Browse files
author
Miguel Jacinto
committed
EZP-2612: Expose the event name in the markup generated by UserMenuItemFireEventView
1 parent 3dcc86c commit 357846e

File tree

2 files changed

+34
-2
lines changed

2 files changed

+34
-2
lines changed

Resources/public/js/views/ez-usermenuitemfireeventview.js

+3-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,9 @@ YUI.add('ez-usermenuitemfireeventview', function (Y) {
2727
},
2828

2929
initializer: function () {
30-
this.containerTemplate = '<li class="' + this._generateViewClassName(this._getName()) + '"/>';
30+
var dataEvent = 'data-event-name="' + this.get('eventName') + '"';
31+
32+
this.containerTemplate = '<li class="' + this._generateViewClassName(this._getName()) + '" ' + dataEvent + '/>';
3133

3234
this.on('addedToUserMenu', this._addUserMenuHideOnEvent, this);
3335
},

Tests/js/views/assets/ez-usermenuitemfireeventview-tests.js

+31-1
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* For full copyright and license information view LICENSE file distributed with this source code.
44
*/
55
YUI.add('ez-usermenuitemfireeventview-tests', function (Y) {
6-
var renderTest, eventTest;
6+
var renderTest, eventTest, dataTest;
77

88
renderTest = new Y.Test.Case({
99
name: "eZ User Menu Item Fire Event render test",
@@ -108,7 +108,37 @@ YUI.add('ez-usermenuitemfireeventview-tests', function (Y) {
108108
},
109109
});
110110

111+
dataTest = new Y.Test.Case({
112+
name: "eZ User Menu Item Fire Event data test",
113+
114+
setUp: function () {
115+
this.view = new Y.eZ.UserMenuItemFireEventView({
116+
eventName: 'logOut'
117+
});
118+
},
119+
120+
tearDown: function () {
121+
this.view.destroy();
122+
},
123+
124+
"Should pass the event name to the container": function () {
125+
var container = this.view.get('container');
126+
127+
Y.Assert.isTrue(
128+
container.hasAttribute('data-event-name'),
129+
'Should have a data event name attribute in the container'
130+
);
131+
Y.Assert.areEqual(
132+
this.view.get('eventName'), container.getAttribute('data-event-name'),
133+
'Should have the correct data event name value in the container'
134+
//'Should pass to the container the correct event name'
135+
);
136+
},
137+
138+
});
139+
111140
Y.Test.Runner.setName("eZ User Menu Item Fire Event View tests");
112141
Y.Test.Runner.add(renderTest);
113142
Y.Test.Runner.add(eventTest);
143+
Y.Test.Runner.add(dataTest);
114144
}, '', {requires: ['test', 'node-event-simulate', 'ez-usermenuitemfireeventview']});

0 commit comments

Comments
 (0)