diff --git a/app/components/message-list/message-list.directive.html b/app/components/message-list/message-list.directive.html
index 2a56180..05c7120 100644
--- a/app/components/message-list/message-list.directive.html
+++ b/app/components/message-list/message-list.directive.html
@@ -1,6 +1,5 @@
{{::vm.header}}
-
-
{{::message.text}}
diff --git a/tests/components/message-list.directive.spec.js b/tests/components/message-list.directive.spec.js
index 0d3af06..d69e813 100644
--- a/tests/components/message-list.directive.spec.js
+++ b/tests/components/message-list.directive.spec.js
@@ -1,16 +1,13 @@
describe('MessageListDirective', ()=> {
let element, scope, compile, rootScope,
- validTemplate = '',
+ validTemplate = '',
defaultData = [
{
id: 0,
text: 'Hi, Loretta Fitzgerald'
}
- ],
- expectedEmptyDataDirectiveOutput = 'Message L-DR
',
- expectedInvalidDirectiveOutput = ''
-
+ ];
beforeEach(angular.mock.module('espackApp'));
@@ -19,18 +16,16 @@ describe('MessageListDirective', ()=> {
});
describe('when created', function () {
- it('should throw error when ngModel attribute not defined', function () {
- //expect(()=> {
+ it('should not render list when messages property is empty', function () {
element = createDirective(null, '');
- var expected = element.html().replace(/(\r|\n)/g, '');
- expect(expected).toBe(expectedInvalidDirectiveOutput);
- //}).toThrow();
+ var expected = element.find('li').size();
+ expect(expected).toBe(0);
});
it('should render the expected output', function () {
element = createDirective();
- var expected = element.html().replace(/(\r|\n)/g, '');
- expect(expected).toBe(expectedEmptyDataDirectiveOutput);
+ var expected = element.find('li').first().text().trim();
+ expect(expected).toBe(defaultData[0].text);
});
});
@@ -47,7 +42,7 @@ describe('MessageListDirective', ()=> {
let elm;
//setup scope state
- scope.data = data || defaultData;
+ scope.vm = {messages: data || defaultData};
//create directive
elm = compile(template || validTemplate)(scope);