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

pin_write() does not always return fully qualified path as documented #647

Closed
juliasilge opened this issue Aug 26, 2022 · 4 comments
Closed
Labels
bug an unexpected problem or unintended behavior

Comments

@juliasilge
Copy link
Member

juliasilge commented Aug 26, 2022

As @jonthegeek points out in #646 object_write() does not always return the path. Sometimes it returns the object.

@juliasilge juliasilge added the bug an unexpected problem or unintended behavior label Aug 26, 2022
@juliasilge
Copy link
Member Author

juliasilge commented Sep 21, 2022

Wait, is this wrong? It does always return the path, doesn't it?

Maybe we should write a couple of tests of what pin_write() returns.

@jonthegeek
Copy link
Contributor

Huh, I think I misread it. I thought it was returning what the sub-function returns, but it ignores that... so I think you're all set!

@juliasilge
Copy link
Member Author

We actually are testing this, via the local_pin() function:

name <- pin_write(board, value, random_pin_name(), ...)

That function is called in all the tests that run across boards, including the metadata tests like test_api_meta(). We are definitely good to go. 👍

@github-actions
Copy link

github-actions bot commented Oct 6, 2022

This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.

@github-actions github-actions bot locked and limited conversation to collaborators Oct 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug an unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

2 participants