#type# #carrier#
++ #type# + + + #carrier# + +
diff --git a/apps/communications/contacts/elements/phone_details.html b/apps/communications/contacts/elements/phone_details.html
index 6eb35192a046..9a4c623e4914 100644
--- a/apps/communications/contacts/elements/phone_details.html
+++ b/apps/communications/contacts/elements/phone_details.html
@@ -1,8 +1,14 @@
-
-
+
#type# #carrier#
++ #type# + + + #carrier# + +
diff --git a/apps/communications/contacts/test/unit/contacts_buttons_test.js b/apps/communications/contacts/test/unit/contacts_buttons_test.js
index b40b1871d7f5..426e693383a2 100644
--- a/apps/communications/contacts/test/unit/contacts_buttons_test.js
+++ b/apps/communications/contacts/test/unit/contacts_buttons_test.js
@@ -44,6 +44,11 @@ suite('Render contact', function() {
var getSpy;
+ var assertCarrierWrapperVisibility = function(visibility) {
+ var carrierWrapperElt = listDetails.querySelector('.carrier-wrapper');
+ assert.equal(carrierWrapperElt.hidden, !visibility);
+ };
+
mocksHelperForContactsButtons.attachTestHelpers();
suiteSetup(function() {
@@ -80,7 +85,7 @@ suite('Render contact', function() {
listDetails.querySelector('h2').innerHTML,
mockContact.tel[0].carrier
);
- sinon.assert.calledWith(getSpy, 'separator');
+ assertCarrierWrapperVisibility(true);
});
test('with 1 phone and carrier undefined', function() {
@@ -98,6 +103,7 @@ suite('Render contact', function() {
contactNoCarrier.tel[0].value);
var carrierContent = listDetails.querySelector('.carrier').textContent;
assert.lengthOf(carrierContent, 0);
+ assertCarrierWrapperVisibility(false);
});
@@ -134,6 +140,7 @@ suite('Render contact', function() {
assert.include(listDetails.innerHTML, subject.DEFAULT_TEL_TYPE);
assert.equal(
-1, listDetails.innerHTML.indexOf('phone-details-template-2'));
+ assertCarrierWrapperVisibility(true);
});
test('highlight phone number', function() {
@@ -243,7 +250,7 @@ suite('Render contact', function() {
stubCall.restore();
}
- test(' > Not loading MultiSimActionButton when we are on an activity',
+ test('> Not loading MultiSimActionButton when we are on an activity',
function() {
this.sinon.stub(MmiManager, 'isMMI').returns(true);
ActivityHandler.currentlyHandling = true;
diff --git a/apps/communications/contacts/test/unit/mock_contacts_buttons_dom.js.html b/apps/communications/contacts/test/unit/mock_contacts_buttons_dom.js.html
index e7c845dbc0ec..f1ea9f24d6a5 100644
--- a/apps/communications/contacts/test/unit/mock_contacts_buttons_dom.js.html
+++ b/apps/communications/contacts/test/unit/mock_contacts_buttons_dom.js.html
@@ -3,8 +3,13 @@
var MockContactsButtonsDom = `
-