From 05e94ab0a8a108e77df134ee4a8c5c91c65c47d6 Mon Sep 17 00:00:00 2001 From: Ned Batchelder Date: Thu, 18 Oct 2018 08:27:13 -0400 Subject: This is literally what setdefault is for --- tests/test_cmdline.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'tests') diff --git a/tests/test_cmdline.py b/tests/test_cmdline.py index 39827ff5..d1b38b98 100644 --- a/tests/test_cmdline.py +++ b/tests/test_cmdline.py @@ -104,8 +104,7 @@ class BaseCmdLineTest(CoverageTest): # the defaults. This lets the tests just mention the interesting ones. for name, _, kwargs in m2.method_calls: for k, v in self.DEFAULT_KWARGS.get(name, {}).items(): - if k not in kwargs: - kwargs[k] = v + kwargs.setdefault(k, v) self.assert_same_method_calls(m1, m2) def cmd_executes_same(self, args1, args2): -- cgit v1.2.1