update doc and unit test

Signed-off-by: Anand Kumar Singh <[email protected]>

Author: anandrkskd

controller/appcontroller_test.go

@@ -732,7 +732,7 @@ func TestSkipAutoSync(t *testing.T) {
 	// Verify we skip when auto-sync is disabled
 	t.Run("AutoSyncEnableFieldIsSetFalse", func(t *testing.T) {
 		app := newFakeApp()
-		enable := true
+		enable := false
 		app.Spec.SyncPolicy.Automated = &v1alpha1.SyncPolicyAutomated{Enable: &enable}
 		ctrl := newFakeController(&fakeData{apps: []runtime.Object{app}}, nil)
 		syncStatus := v1alpha1.SyncStatus{

docs/user-guide/auto_sync.md

@@ -18,7 +18,7 @@ spec:
   syncPolicy:
     automated: {}
 ```
-Application CRD now also support explicitly setting automated sync to be turn on or off  by using `spec.syncPolicy.automated.enable` flag to true or false
+Application CRD now also support explicitly setting automated sync to be turn on or off by using `spec.syncPolicy.automated.enable` flag to true or false. When `enable` field is set to true, Automated Sync is active and when set to false controller will skip automated sync even if `prune`, `self-heal` and `allowEmpty` are set.
 ```yaml
 spec:
   syncPolicy:
@@ -27,7 +27,7 @@ spec:
 ```
 
 !!!note 
-  **Note:** Setting `spec.syncPolicy.automated.enable` flag to null will be treated as automated sync as enabled.
+  Setting `spec.syncPolicy.automated.enable` flag to null will be treated as automated sync as enabled. When using `enable` field set to false, fields like `prune`, `self-heal` and `allowEmpty` can be set without enabling them.
 
 ## Temporarily toggling auto-sync for applications managed by ApplicationSets