Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

test: add test for config set with number #470

Merged
merged 1 commit into from
May 15, 2019

Conversation

alanshaw
Copy link
Contributor

@alanshaw alanshaw commented May 15, 2019

Anything that isn't a string needs to be passed with --json to the config set HTTP API. This PR tests that this is happening by virtue of setting a non string value.

License: MIT
Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
Copy link
Contributor

@dirkmc dirkmc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@alanshaw alanshaw merged commit 6d339a2 into master May 15, 2019
@alanshaw alanshaw deleted the test/add-config-set-number-test branch May 15, 2019 15:10
alanshaw pushed a commit to ipfs-inactive/js-ipfs-http-client that referenced this pull request May 15, 2019
Anything that isn't a string needs to be passed with `--json`.

fixes #881

Depends on:

* [x] ipfs-inactive/interface-js-ipfs-core#470

License: MIT
Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants