Skip to content

Commit d276ed6

Browse files
committed
asim: rename generateAllPlots
1 parent 5cdcd13 commit d276ed6

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

pkg/kv/kvserver/asim/tests/datadriven_simulation_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -567,7 +567,7 @@ func TestDataDriven(t *testing.T) {
567567
// parameter to the `eval` command.
568568
testName := name + "_" + mv
569569
for sample, h := range run.hs {
570-
generateAllPlots(t, &buf, h, testName, sample+1, plotDir, hasher, rewrite,
570+
printStatsAndGenerateJSON(t, &buf, h, testName, sample+1, plotDir, hasher, rewrite,
571571
settingsGen.Settings.TickInterval, metricsMap)
572572
generateTopology(t, h,
573573
filepath.Join(plotDir, fmt.Sprintf("%s_%d_topology.txt", testName, sample+1)),

pkg/kv/kvserver/asim/tests/plotting_test.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,13 @@ import (
2121
"github.com/stretchr/testify/require"
2222
)
2323

24-
// generateAllPlots creates plots for all available metrics from simulation history.
24+
// printStatsAndGenerateJSON creates plots for all available metrics from simulation history.
2525
// All plot files are hashed directly to the provided hasher.
2626
// If rewrite is false, plots are generated but not saved to disk.
2727
// Returns a slice of filenames for all generated plots.
2828
//
2929
// TODO(tbg): introduce a SimulationEnv and make this a method on it.
30-
// TODO(tbg): rename to generateArtifacts.
31-
func generateAllPlots(
30+
func printStatsAndGenerateJSON(
3231
t *testing.T,
3332
buf *strings.Builder,
3433
h history.History,

0 commit comments

Comments
 (0)