refactor: change cleanup for TestAddFileResolvedFields

Signed-off-by: Dominik Rosiek <[email protected]>

operator/builtin/input/file/file_test.go

@@ -99,17 +99,16 @@ func TestAddFileResolvedFields(t *testing.T) {
 	require.NoError(t, err)
 
 	require.NoError(t, operator.Start(testutil.NewMockPersister("test")))
-	defer operator.Stop()
 
 	e := waitForOne(t, logReceived)
 	require.Equal(t, filepath.Base(symLinkPath), e.Attributes["file_name"])
 	require.Equal(t, symLinkPath, e.Attributes["file_path"])
 	require.Equal(t, filepath.Base(resolved), e.Attributes["file_name_resolved"])
 	require.Equal(t, resolved, e.Attributes["file_path_resolved"])
 
-	// Clean up (linux based host)
-	// Ignore error on windows host (The process cannot access the file because it is being used by another process.)
-	os.RemoveAll(dir)
+	operator.Stop()
+	err = os.RemoveAll(dir)
+	require.NoError(t, err)
 }
 
 // AddFileResolvedFields tests that the `file_name_resolved` and `file_path_resolved` fields are included