Skip to content

Commit

Permalink
I suspect codspeed may only need the benchmark marker
Browse files Browse the repository at this point in the history
The fixture may be superfluous
  • Loading branch information
SolarDrew committed May 10, 2024
1 parent 4ef6919 commit 762ff6e
Showing 1 changed file with 8 additions and 10 deletions.
18 changes: 8 additions & 10 deletions dkist/tests/test_benchmarks.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,15 +27,13 @@ def test_pixel_to_world(benchmark, large_visp_dataset):
@pytest.mark.benchmark(min_rounds=2)
@pytest.mark.parametrize("axes", [
["y", "x", None, None],
["y", None, "x", None],
["y", None, None, "x"],
[None, "y", "x", None],
[None, "y", None, "x"],
[None, None, "y", "x"],
# ["y", None, "x", None],
# ["y", None, None, "x"],
# [None, "y", "x", None],
# [None, "y", None, "x"],
# [None, None, "y", "x"],
])
def test_plot_dataset(benchmark, axes, large_visp_dataset):
@benchmark
def plot_and_save_fig(ds=large_visp_dataset, axes=axes):
ds.plot(plot_axes=axes)
plt.savefig("tmpplot")
plt.close()
large_visp_dataset.plot(plot_axes=axes)
plt.savefig("tmpplot")
plt.close()

Check warning on line 39 in dkist/tests/test_benchmarks.py

View check run for this annotation

Codecov / codecov/patch

dkist/tests/test_benchmarks.py#L37-L39

Added lines #L37 - L39 were not covered by tests

0 comments on commit 762ff6e

Please sign in to comment.