Skip to content

Commit 7529c3b

Browse files
pkratochm-blaha
authored andcommitted
Fix 'already installed' message output.
1 parent fcb4597 commit 7529c3b

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

dnf/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2423,4 +2423,4 @@ def _verification_of_packages(pkg_list, logger_msg):
24232423
def _msg_installed(pkg):
24242424
name = ucd(pkg)
24252425
msg = _('Package %s is already installed, skipping.')
2426-
logger.warning(msg, name)
2426+
logger.info(msg, name)

tests/test_install.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ def test_install_filename(self):
235235
def test_install_installed(self):
236236
"""Test that nothing changes if an installed package matches."""
237237
stdout = dnf.pycomp.StringIO()
238-
with tests.support.wiretap_logs('dnf', logging.WARNING, stdout):
238+
with tests.support.wiretap_logs('dnf', logging.INFO, stdout):
239239
self.base.install('librita')
240240
self.assertEqual(self.base._goal.req_length(), 0)
241241
self.assertIn(
@@ -268,7 +268,7 @@ def test_install_name_choice(self):
268268
dnf.subject.Subject('pepper.src').get_best_query(self.base.sack))
269269

270270
stdout = dnf.pycomp.StringIO()
271-
with tests.support.wiretap_logs('dnf', logging.WARNING, stdout):
271+
with tests.support.wiretap_logs('dnf', logging.INFO, stdout):
272272
self.base.install('pepper')
273273
self.assertEqual(self.base._goal.req_length(), 0)
274274
self.assertIn(
@@ -373,7 +373,7 @@ def test_install_filename(self):
373373
def test_install_installed(self):
374374
"""Test that nothing changes if an installed package matches."""
375375
stdout = dnf.pycomp.StringIO()
376-
with tests.support.wiretap_logs('dnf', logging.WARNING, stdout):
376+
with tests.support.wiretap_logs('dnf', logging.INFO, stdout):
377377
self.base.install('librita')
378378
installed, removed = self.installed_removed(self.base)
379379
self.assertEmpty(installed)
@@ -408,7 +408,7 @@ def test_install_name_choice(self):
408408
dnf.subject.Subject('pepper.src').get_best_query(self.base.sack))
409409

410410
stdout = dnf.pycomp.StringIO()
411-
with tests.support.wiretap_logs('dnf', logging.WARNING, stdout):
411+
with tests.support.wiretap_logs('dnf', logging.INFO, stdout):
412412
self.base.install('pepper')
413413
installed, removed = self.installed_removed(self.base)
414414
self.assertEmpty(installed | removed)
@@ -451,7 +451,7 @@ def test_install_reponame(self):
451451
def test_install_unavailable(self):
452452
"""Test that nothing changes if an unavailable package matches."""
453453
stdout = dnf.pycomp.StringIO()
454-
with tests.support.wiretap_logs('dnf', logging.WARNING, stdout):
454+
with tests.support.wiretap_logs('dnf', logging.INFO, stdout):
455455
cnt = self.base.install('hole')
456456
self.assertEqual(cnt, 1)
457457
installed_pkgs = self.base.sack.query().installed().run()

0 commit comments

Comments
 (0)