Skip to content
This repository has been archived by the owner on Oct 28, 2019. It is now read-only.

Modify mechanism to read JSON file in unit tests #30

Closed
andrie opened this issue Nov 2, 2015 · 1 comment
Closed

Modify mechanism to read JSON file in unit tests #30

andrie opened this issue Nov 2, 2015 · 1 comment
Assignees
@andrie
Labels
tests

Comments

@andrie
Copy link
Contributor

andrie commented Nov 2, 2015

The current mechanism is:

keyfile <- system.file("tests/testthat/config.json", package = "AzureML")

This is flawed since this json file can't be found during package build.

The fix might be to put this json file in /inst/...
@andrie andrie added the tests label Nov 2, 2015
@andrie andrie self-assigned this Nov 2, 2015
@andrie
Copy link
Contributor Author

andrie commented Nov 7, 2015

Closing the issue, since @bwlewis added the fix in the pull request #33

@andrie andrie closed this as completed Nov 7, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@andrie andrie

Labels
tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andrie