From 9eae55f03c7245232017d8463d4ddbbccaa691ba Mon Sep 17 00:00:00 2001 From: Ivan Kravets Date: Mon, 14 Dec 2015 11:30:39 +0200 Subject: [PATCH] Add link to telemetry setting --- docs/userguide/cmd_settings.rst | 2 +- platformio/app.py | 6 ++---- tests/commands/test_settings.py | 3 +-- 3 files changed, 4 insertions(+), 7 deletions(-) diff --git a/docs/userguide/cmd_settings.rst b/docs/userguide/cmd_settings.rst index 6ea784da..95a2187e 100644 --- a/docs/userguide/cmd_settings.rst +++ b/docs/userguide/cmd_settings.rst @@ -142,7 +142,7 @@ Examples check_platformio_interval 3 Check for the new PlatformIO interval (days) check_platforms_interval 7 Check for the platform updates interval (days) enable_prompts Yes Can PlatformIO communicate with you via prompts: propose to install platforms which aren't installed yet, paginate over library search results and etc.)? ATTENTION!!! If you call PlatformIO like subprocess, please disable prompts to avoid blocking (Yes/No) - enable_telemetry Yes Share diagnostics and usage information (PlatformIO fatal errors/exceptions, platforms, boards, frameworks, commands) to help us make PlatformIO better (Yes/No) + enable_telemetry Yes Telemetry service (Yes/No) 2. Show specified setting diff --git a/platformio/app.py b/platformio/app.py index 0c52cff1..2e522aa6 100644 --- a/platformio/app.py +++ b/platformio/app.py @@ -47,10 +47,8 @@ DEFAULT_SETTINGS = { }, "enable_telemetry": { "description": ( - "Share diagnostics and usage information (PlatformIO fatal " - "errors/exceptions, platforms, boards, frameworks, commands) " - "to help us make PlatformIO better. We DO NOT share PRIVATE " - "information (Yes/No)"), + "Telemetry service (Yes/No)"), "value": True }, "enable_prompts": { diff --git a/tests/commands/test_settings.py b/tests/commands/test_settings.py index b12d398e..312b0e80 100644 --- a/tests/commands/test_settings.py +++ b/tests/commands/test_settings.py @@ -18,8 +18,7 @@ from platformio import app def test_settings_check(clirunner, validate_cliresult): result = clirunner.invoke(cli, ["get"]) - assert result.exit_code == 0 - assert not result.exception + validate_cliresult(result) assert len(result.output) for item in app.DEFAULT_SETTINGS.items(): assert item[0] in result.output