stdlib icon indicating copy to clipboard operation
stdlib copied to clipboard

test: add tests to `@stdlib/utils/compact-adjacency-matrix`

Open Pratik772846 opened this issue 1 year ago • 3 comments

added tests to @stdlib/utils/compact-adjacency-matrix

Resolves #1330

Description

What is the purpose of this pull request?

This pull request:

  • add tests to @stdlib/utils/compact-adjacency-matrix

Related Issues

Does this pull request have any related issues?

This pull request:

  • resolves #1330

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

Pratik772846 avatar Apr 06 '24 17:04 Pratik772846

image Hi @kgryte @Planeshifter ,These 2 tests to check property and method are failing for "fromEdge" method .But they are passing for all other methods.What can be the reason behind it?

Pratik772846 avatar Apr 07 '24 03:04 Pratik772846

@Pratik772846 The method is not a prototype method, but a static method. You should be checking for a method on the constructor.

kgryte avatar Apr 07 '24 05:04 kgryte

@kgryte Sorry for the oversight. I have updated the tests. Please review it at your convenience.

Pratik772846 avatar Apr 07 '24 08:04 Pratik772846