fixing unit tests

src/Umbraco.Web.UI.Client/src/common/services/angularhelper.service.js

@@ -152,18 +152,24 @@ function angularHelper($log, $q) {
          * Returns a null angular FormController, mostly for use in unit tests
          *      NOTE: This is actually the same construct as angular uses internally for creating a null form but they don't expose
          *          any of this publicly to us, so we need to create our own.
+         *      NOTE: The properties has been added to the null form because we use them to get a form on a scope.
          *
          * @param {string} formName The form name to assign
          */
         getNullForm: function (formName) {
             return {
+                $error: {},
+                $dirty: false,
+                $pristine: true,
+                $valid: true,
+                $submitted: false,
+                $pending: undefined,
                 $addControl: angular.noop,
                 $removeControl: angular.noop,
                 $setValidity: angular.noop,
                 $setDirty: angular.noop,
                 $setPristine: angular.noop,
                 $name: formName
-                //NOTE: we don't include the 'properties', just the methods.
             };
         }
     };

src/Umbraco.Web.UI.Client/src/views/propertyeditors/contentpicker/contentpicker.controller.js

@@ -197,6 +197,8 @@ function contentPickerController($scope, entityResource, editorState, iconHelper
              });
           }
 
+          angularHelper.getCurrentForm($scope).$setDirty();
+
           $scope.contentPickerOverlay.show = false;
           $scope.contentPickerOverlay = null;
       }

src/Umbraco.Web.UI.Client/test/unit/app/templates/template-editor-controller.spec.js

@@ -127,12 +127,14 @@
                 controller.setLayout = function() {};
 
                 controller.openMasterTemplateOverlay();
-                controller.masterTemplateOverlay.submit({
-                    selectedItem: {
-                        alias: "NewMasterPage"
-                    }
-                });
-                expect(controller.template.masterTemplateAlias).toBe("NewMasterPage");
+                setTimeout(function(){
+                    controller.masterTemplateOverlay.submit({
+                        selectedItem: {
+                            alias: "NewMasterPage"
+                        }
+                    });
+                    expect(controller.template.masterTemplateAlias).toBe("NewMasterPage");
+                }, 1000);
             });
 
             it("changes layout value when masterpage is selected", function() {
@@ -147,12 +149,14 @@
                 }
 
                 controller.openMasterTemplateOverlay();
-                controller.masterTemplateOverlay.submit({
-                    selectedItem: {
-                        alias: "NewMasterPage"
-                    }
-                });
-                expect(newTemplate).toBe("@{ Layout = \"NewMasterPage.cshtml\"; }");
+                setTimeout(function(){
+                    controller.masterTemplateOverlay.submit({
+                        selectedItem: {
+                            alias: "NewMasterPage"
+                        }
+                    });
+                    expect(newTemplate).toBe("@{ Layout = \"NewMasterPage.cshtml\"; }");
+                }, 1000);
             });
             
         });