test: add unit tests for seurat_clustering and cluster_metrics

[bianjh-cloud]

Changes

test-cluster_metrics.R

• imports BRCA fixture data, clustered by seurat_clustering
• verifies output shape for silhouette parameter
• checks numeric results, consistency across metadata variables
• expect errors for missing cluster columns, invalid reduction

test-seurat_clustering.R

• verifies default parameters
• run function on BRCA fixture
• checks that low / high resolution cluster(s) numbers are plotted correctly
• checks errors for invalid npcs_in, resolution, and algorithm inputs

Issues

PR Checklist

(Strikethrough any points that are not applicable.)

• This comment contains a description of changes with justifications, with any relevant issues linked.
• Write unit tests for any new features, bug fixes, or other code changes.
• Update the docs if there are any API changes (roxygen2 comments, vignettes, readme, etc.).
• Update NEWS.md with a short description of any user-facing changes and reference the PR number. Follow the style described in https://style.tidyverse.org/news.html
• Run devtools::check() locally and fix all notes, warnings, and errors.
• R-CMD-check succeeds on the most recent user commit.

[kelly-sovacool]

Take a look at my review of #49 and apply those concepts here as well

[kelly-sovacool]

@wong-nw do you have suggestions for new names to seurat_clustering and cluster_metrics so they are verbs per the style guide? https://style.tidyverse.org/functions.html