summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMats Wichmann <mats@linux.com>2021-06-08 07:33:44 -0600
committerMats Wichmann <mats@linux.com>2021-06-08 07:33:44 -0600
commit69da0a96542d6e529dbb0eef3cdc4e1a439037ae (patch)
treed9dc729fb3d80bd95bf4bec5602861da5cb79051
parentef0b9aae9604f07104d6daf11e94bd844b7cf39f (diff)
downloadscons-git-69da0a96542d6e529dbb0eef3cdc4e1a439037ae.tar.gz
PR #3942: retstore two checks per PR review
Signed-off-by: Mats Wichmann <mats@linux.com>
-rw-r--r--SCons/EnvironmentTests.py15
1 files changed, 7 insertions, 8 deletions
diff --git a/SCons/EnvironmentTests.py b/SCons/EnvironmentTests.py
index 553b5e85c..78f6ac2aa 100644
--- a/SCons/EnvironmentTests.py
+++ b/SCons/EnvironmentTests.py
@@ -994,19 +994,18 @@ class BaseTestCase(unittest.TestCase,TestEnvironmentFixture):
assert called_it['source'] is None, called_it
def test_BuilderWrapper_attributes(self):
- """Test getting and setting of BuilderWrapper attributes
- """
+ """Test getting and setting of BuilderWrapper attributes."""
b1 = Builder()
b2 = Builder()
e1 = Environment()
e2 = Environment()
- e1.Replace(BUILDERS = {'b' : b1})
+ e1.Replace(BUILDERS={'b': b1})
bw = e1.b
- assert bw.env == e1
+ assert bw.env is e1
bw.env = e2
- assert bw.env == e2
+ assert bw.env is e2
assert bw.builder is b1
bw.builder = b2
@@ -1800,7 +1799,7 @@ def exists(env):
updates and check that the original remains intact
and the copy has the updated values.
"""
- env1 = self.TestEnvironment(XXX = 'x', YYY = 'y')
+ env1 = self.TestEnvironment(XXX='x', YYY='y')
env2 = env1.Clone()
env1copy = env1.Clone()
assert env1copy == env1
@@ -1809,7 +1808,7 @@ def exists(env):
assert env1 != env2
assert env1 == env1copy
- env3 = env1.Clone(XXX = 'x3', ZZZ = 'z3')
+ env3 = env1.Clone(XXX='x3', ZZZ='z3')
assert env3 != env1
assert env3.Dictionary('XXX') == 'x3'
assert env1.Dictionary('XXX') == 'x'
@@ -1832,7 +1831,7 @@ def exists(env):
assert 5 not in env1.Dictionary('ZZZ')
#
- env1 = self.TestEnvironment(BUILDERS = {'b1' : Builder()})
+ env1 = self.TestEnvironment(BUILDERS={'b1': Builder()})
assert hasattr(env1, 'b1'), "env1.b1 was not set"
assert env1.b1.object == env1, "b1.object doesn't point to env1"
env2 = env1.Clone(BUILDERS = {'b2' : Builder()})