stdlib icon indicating copy to clipboard operation
stdlib copied to clipboard

test: add missing tests to `ndarray/base/every`

Open headlessNode opened this issue 8 months ago • 2 comments


type: pre_commit_static_analysis_report description: Results of running static analysis checks when committing changes. report:

  • task: lint_filenames status: passed
  • task: lint_editorconfig status: passed
  • task: lint_markdown status: na
  • task: lint_package_json status: na
  • task: lint_repl_help status: na
  • task: lint_javascript_src status: na
  • task: lint_javascript_cli status: na
  • task: lint_javascript_examples status: na
  • task: lint_javascript_tests status: passed
  • task: lint_javascript_benchmarks status: na
  • task: lint_python status: na
  • task: lint_r status: na
  • task: lint_c_src status: na
  • task: lint_c_examples status: na
  • task: lint_c_benchmarks status: na
  • task: lint_c_tests_fixtures status: na
  • task: lint_shell status: na
  • task: lint_typescript_declarations status: na
  • task: lint_typescript_tests status: na
  • task: lint_license_headers status: passed ---

Progresses #2656.

Description

What is the purpose of this pull request?

This pull request:

  • add tests for 100% code coverage.

Related Issues

Does this pull request have any related issues?

This pull request:

  • progresses #2656.

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.


@stdlib-js/reviewers

headlessNode avatar Jun 06 '25 09:06 headlessNode

Coverage Report

Package Statements Branches Functions Lines
ndarray/base/every $\color{red}12000/12006$
$\color{green}+99.95\%$
$\color{red}1464/1468$
$\color{green}+99.73\%$
$\color{green}64/64$
$\color{green}+100.00\%$
$\color{red}12000/12006$
$\color{green}+99.95\%$

The above coverage report was generated for the changes in this PR.

stdlib-bot avatar Jun 06 '25 09:06 stdlib-bot

Headsup, working on 100% coverage for accessors.

headlessNode avatar Jun 09 '25 10:06 headlessNode

Heads up! It appears strides for non-contiguous cases without large arrays maybe incorrect.

headlessNode avatar Jun 17 '25 09:06 headlessNode

@kgryte ready for review.

headlessNode avatar Jun 17 '25 21:06 headlessNode