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.
This commit is contained in:
Kaio Giurizatto Utsch
2019-11-26 15:02:03 -03:00
parent 95924a34e6
commit 82a535b97e
2 changed files with 4 additions and 3 deletions
+1 -1
View File
@@ -50,7 +50,7 @@ class CmdRemoteDefault(CmdRemoteConfig):
def run(self):
if self.args.name is None and not self.args.unset:
name = self.remote_config.get_default(level=self.args.level)
logger.info("{}".format(name))
print(name)
else:
self.remote_config.set_default(
self.args.name, unset=self.args.unset, level=self.args.level
+3 -2
View File
@@ -145,10 +145,11 @@ class TestRemoteDefault(TestDvc):
self.assertEqual(default, None)
def test_show_default(dvc_repo, caplog):
def test_show_default(dvc_repo, capsys):
assert main(["remote", "default", "foo"]) == 0
assert main(["remote", "default"]) == 0
assert "foo" == caplog.record_tuples[0][2]
out, _ = capsys.readouterr()
assert out == "foo\n"
class TestRemoteShouldHandleUppercaseRemoteName(TestDvc):