From 1621d01992686bb232feb9bf8763c797f705e136 Mon Sep 17 00:00:00 2001 From: Shannon Date: Fri, 17 Jan 2014 13:18:49 +1100 Subject: [PATCH] fixes some unit tests --- .../test/unit/app/media/edit-media-controller.spec.js | 2 +- .../propertyeditors/content-picker-controller.spec.js | 10 ++++++---- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/test/unit/app/media/edit-media-controller.spec.js b/src/Umbraco.Web.UI.Client/test/unit/app/media/edit-media-controller.spec.js index 3b453a7930..64a21b670e 100644 --- a/src/Umbraco.Web.UI.Client/test/unit/app/media/edit-media-controller.spec.js +++ b/src/Umbraco.Web.UI.Client/test/unit/app/media/edit-media-controller.spec.js @@ -47,7 +47,7 @@ describe('edit media controller tests', function () { }); it('it should have a tabs collection', function () { - expect(scope.content.tabs.length).toBe(5); + expect(scope.content.tabs.length).toBe(1); }); it('it should have a properties collection on each tab', function () { diff --git a/src/Umbraco.Web.UI.Client/test/unit/app/propertyeditors/content-picker-controller.spec.js b/src/Umbraco.Web.UI.Client/test/unit/app/propertyeditors/content-picker-controller.spec.js index 2ffd204d24..b42a861e8a 100644 --- a/src/Umbraco.Web.UI.Client/test/unit/app/propertyeditors/content-picker-controller.spec.js +++ b/src/Umbraco.Web.UI.Client/test/unit/app/propertyeditors/content-picker-controller.spec.js @@ -60,7 +60,7 @@ describe('Content picker controller tests', function () { it("Removing an item should update renderModel, ids and model.value", function(){ scope.remove(1); - + scope.$apply(); expect(scope.renderModel.length).toBe(2); expect(scope.ids.length).toBe(2); expect(scope.model.value).toBe("1233,23121"); @@ -69,7 +69,7 @@ describe('Content picker controller tests', function () { it("Adding an item should update renderModel, ids and model.value", function(){ scope.add({name: "meh", id: 666, icon: "woop"}); - + scope.$apply(); expect(scope.renderModel.length).toBe(4); expect(scope.ids.length).toBe(4); expect(scope.model.value).toBe("1233,1231,23121,666"); @@ -77,12 +77,14 @@ describe('Content picker controller tests', function () { it("Adding a dublicate item should note update renderModel, ids and model.value", function(){ - scope.add({name: "meh", id: 666, icon: "woop"}); + scope.add({ name: "meh", id: 666, icon: "woop" }); + scope.$apply(); expect(scope.renderModel.length).toBe(4); expect(scope.ids.length).toBe(4); expect(scope.model.value).toBe("1233,1231,23121,666"); - scope.add({name: "meh 2", id: 666, icon: "woop 2"}); + scope.add({ name: "meh 2", id: 666, icon: "woop 2" }); + scope.$apply(); expect(scope.renderModel.length).toBe(4); expect(scope.ids.length).toBe(4); expect(scope.model.value).toBe("1233,1231,23121,666");