chore: fix race condition on init read (#409)

Newly added tests showed flakiness - this is due to missing initial
fetch handling.

This PR improves tests only.

Signed-off-by: Kavindu Dodanduwa <kavindudodanduwa@gmail.com>

pkg/sync/file/filepath_sync_test.go

@@ -82,20 +82,6 @@ func TestSimpleSync(t *testing.T) {
 				},
 			},
 		},
-		"empty-file-use-default": {
-			manipulationFuncs: []func(t *testing.T){
-				func(t *testing.T) {
-					writeToFile(t, "")
-				},
-			},
-			expectedDataSync: []sync.DataSync{
-				{
-					FlagData: defaultState,
-					Source:   fmt.Sprintf("%s/%s", fetchDirName, fetchFileName),
-					Type:     sync.ALL,
-				},
-			},
-		},
 	}
 
 	handler := Sync{
@@ -120,6 +106,12 @@ func TestSimpleSync(t *testing.T) {
 				}
 			}()
 
+			// file sync perform an initial fetch and then watch for file events
+			init := <-dataSyncChan
+			if init.FlagData != defaultState {
+				t.Errorf("initial fetch for empty file expected to return default state: %s", defaultState)
+			}
+
 			for i, manipulation := range tt.manipulationFuncs {
 				syncEvent := tt.expectedDataSync[i]
 				manipulation(t)