Make Value.get_text_contents() use child csigs instead of contents

Current implementation of Value.get_text_contents() returns a
concatenation of all child contents. Instead, make the contents of a
Value similar to an Alias, except prepend the stringified 'value'
attribute as well.

Since Value.get_text_contents() will no longer fail (as long as child
nodes implement a working get_csig() which they should), make
get_contents() return get_text_contents() but utf-8 encoded. Also
make get_csig() just return get_text_contents()

Add new tests for testing get_contents() and get_text_contents() only
rely on child node csigs, and not directly on child content.

Author: eugenhu

SCons/Node/Python.py

@@ -138,24 +138,20 @@ def read(self):
     def get_text_contents(self):
         """By the assumption that the node.built_value is a
         deterministic product of the sources, the contents of a Value
-        are the concatenation of all the contents of its sources.  As
-        the value need not be built when get_contents() is called, we
+        are the concatenation of all the content signatures of its sources.
+        As the value need not be built when get_contents() is called, we
         cannot use the actual node.built_value."""
-        ###TODO: something reasonable about universal newlines
         contents = str(self.value)
-        for kid in self.children(None):
-            contents = contents + kid.get_contents().decode()
+        contents += ''.join(n.get_csig() for n in self.children())
         return contents
 
     def get_contents(self):
         """
-        Get contents for signature calculations.
+        Same as get_text_contents() but encode as utf-8 in case other
+        functions rely on get_contents() existing.
         :return: bytes
         """
-        contents = str(self.value).encode('utf-8')
-        for kid in self.children(None):
-            contents = contents + kid.get_contents()
-        return contents
+        return self.get_text_contents().encode('utf-8')
 
     def changed_since_last_build(self, target, prev_ni):
         cur_csig = self.get_csig()
@@ -176,10 +172,8 @@ def get_csig(self, calc=None):
         except AttributeError:
             pass
 
-        csig = str(self.value)
-        csig += ''.join(child.get_csig() for child in self.children(None))
+        csig = self.get_text_contents()
 
-        self.get_ninfo().csig = csig
         return csig
 
 

SCons/Node/PythonTests.py

@@ -122,22 +122,39 @@ def get_csig(self, calc=None):
         csig = v.get_csig(None)
         assert csig == 'aaacsig1csig2', csig
 
-    def test_get_content_with_child_binary_content(self):
+    def test_get_text_contents_with_children(self):
+        """Test calculating text contents with child nodes
+        """
         class DummyNode:
-            def __init__(self, contents):
-                self.contents = contents
-            def get_contents(self):
-                return self.contents
+            def __init__(self, csig):
+                self.csig = csig
+            def get_csig(self):
+                return self.csig
 
-        # Node with binary content that is not valid utf-8.
-        node_with_binary = DummyNode(b'\xff')
+        v = SCons.Node.Python.Value('aaa')
+        c1 = DummyNode(csig='csig1')
+        c2 = DummyNode(csig='csig2')
 
-        v = SCons.Node.Python.Value('v')
-        v.add_dependency([node_with_binary])
+        v.add_dependency([c1, c2])
+        text_contents = v.get_text_contents()
+        assert text_contents == 'aaacsig1csig2', text_contents
 
-        # Just make sure this call doesn't fail. Not sure what to check the
-        # return value against.
-        v.get_contents()
+    def test_get_content_with_children(self):
+        """Test calculating contents with child nodes
+        """
+        class DummyNode:
+            def __init__(self, csig):
+                self.csig = csig
+            def get_csig(self):
+                return self.csig
+
+        v = SCons.Node.Python.Value('aaa')
+        c1 = DummyNode(csig='csig1')
+        c2 = DummyNode(csig='csig2')
+
+        v.add_dependency([c1, c2])
+        contents = v.get_contents()
+        assert contents == v.get_text_contents().encode('utf-8'), contents
 
 
 class ValueNodeInfoTestCase(unittest.TestCase):