Remove passing of anonymous visualize function

[serathius]

What would you like to be added?

Before linearizationInfo was a private struct we were not able to return the results up the call stack. So I wrapped the function I needed and passed it like

etcd/tests/robustness/validate/operations.go

Lines 49 to 56 in e4d1da7

	return result, func(path string) error {
	lg.Info("Saving visualization", zap.String("path", path))
	err := porcupine.VisualizePath(model.NonDeterministicModel, info, path)
	if err != nil {
	return fmt.Errorf("failed to visualize, err: %w", err)
	}
	return nil
	}

With https://github.com/etcd-io/etcd/blob/main/tests/robustness/validate/operations.go#L49-L56 making the struct piblic we can simply replace it with passing it linearizationInfo.

Why is this needed?

Cleanup code

[serathius]

As VisualizePath function needs also model as argument, maybe we can wrap it in struct with Visualize method:

type Results struct {
  info pourcupine.LinearizationInfo
  model pourcupine.Model
}

[lhajouji]

Hello @serathius , i'd like to work on this ?
Could you assign it to me, Thanks in advance.

[VidhuSarwal]

Hi, I submitted a PR please review. Thanks.