Skip to content

Commit 82a535b

Browse files
committed
Change logging to print
Python logging recommends using print instead of logging for ordinary output of command line program. Also changed the test to reflect this.
1 parent 95924a3 commit 82a535b

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

dvc/command/remote.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ class CmdRemoteDefault(CmdRemoteConfig):
5050
def run(self):
5151
if self.args.name is None and not self.args.unset:
5252
name = self.remote_config.get_default(level=self.args.level)
53-
logger.info("{}".format(name))
53+
print(name)
5454
else:
5555
self.remote_config.set_default(
5656
self.args.name, unset=self.args.unset, level=self.args.level

tests/func/test_remote.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -145,10 +145,11 @@ def test(self):
145145
self.assertEqual(default, None)
146146

147147

148-
def test_show_default(dvc_repo, caplog):
148+
def test_show_default(dvc_repo, capsys):
149149
assert main(["remote", "default", "foo"]) == 0
150150
assert main(["remote", "default"]) == 0
151-
assert "foo" == caplog.record_tuples[0][2]
151+
out, _ = capsys.readouterr()
152+
assert out == "foo\n"
152153

153154

154155
class TestRemoteShouldHandleUppercaseRemoteName(TestDvc):

0 commit comments

Comments
 (0)