Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Test] Test Coverage for Function class #935

Closed
JckXia opened this issue Mar 16, 2021 · 3 comments
Closed

[Test] Test Coverage for Function class #935

JckXia opened this issue Mar 16, 2021 · 3 comments
Labels

Comments

@JckXia
Copy link
Member

JckXia commented Mar 16, 2021

class methods
Function Function()
See PR #928 (Covered) Value Call(const std::initalizer_list<napi_value>& args)const
See PR #928 (Covered) Value Call(const std::vector<napi_value>&args ) const
See PR #928 (Covered) Value Call(size_t argc, const napi_value* args) const
See PR #928 (Covered) Value Call(napi_value recv, const std::initializer_list<napi_value>& args)
See PR #928 (Covered) Value Call(napi_value recv, size_t argc, const napi_value* args) const
See PR #928 (Covered) Value Call(napi_value recv, const std::vector<napi_value>& args ) const
See PR #928 (Covered) Value MakeCallback(napi_value recv, const std::initializer_list<napi_value>& args) const
See PR #928 (Covered) Value MakeCallback(napi_value recv, const std::vector<napi_value>& args) const
See PR #928 (Covered) Value MakeCallback(napi_value recv, size_t argc, const napi_value* args) const

@NickNaso

@NickNaso NickNaso added the test label Mar 17, 2021
@KevinEady
Copy link
Contributor

Can we close this as #928 has been merged?

@github-actions
Copy link
Contributor

github-actions bot commented Jul 9, 2021

This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made.

@github-actions github-actions bot added the stale label Jul 9, 2021
@NickNaso NickNaso removed the stale label Jul 9, 2021
@NickNaso
Copy link
Member

I'm closing this issue becasue all seems to be covered.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants