From 7c2786535e0cf97edfbb68f16331e3cfbb34d43d Mon Sep 17 00:00:00 2001
From: Bob Mottram <bob@freedombone.net>
Date: Wed, 23 Dec 2020 19:55:51 +0000
Subject: [PATCH] Call graph colors

---
 tests.py | 41 +++++++++++++++++++++++++++++++----------
 1 file changed, 31 insertions(+), 10 deletions(-)

diff --git a/tests.py b/tests.py
index bc3b0d312..32dd0b01a 100644
--- a/tests.py
+++ b/tests.py
@@ -2854,6 +2854,7 @@ def testFunctions():
                     'goldenrod', 'burlywood2', 'bisque1', 'brown1',
                     'chartreuse2', 'cornsilk', 'darksalmon')
     maxModuleCalls = 1
+    maxFunctionCalls = 1
     colorCtr = 0
     for modName, modProperties in modules.items():
         lineCtr = 0
@@ -2868,6 +2869,8 @@ def testFunctions():
                     getFunctionCalls(name, modules[modName]['lines'],
                                      lineCtr, functionProperties)
                 functionProperties[name]['calls'] = callsList.copy()
+                if len(callsList) > maxFunctionCalls:
+                    maxFunctionCalls = len(callsList)
                 # keep track of which module calls which other module
                 for fn in callsList:
                     modCall = functionProperties[fn]['module']
@@ -2889,14 +2892,17 @@ def testFunctions():
     for modName, modProperties in modules.items():
         if not modProperties.get('calls'):
             callGraphStr += '  "' + modName + \
-                '" [fillcolor = yellow style=filled];\n'
+                '" [fillcolor=yellow style=filled];\n'
             continue
-        if len(modProperties['calls']) < int(maxModuleCalls / 4):
+        if len(modProperties['calls']) <= int(maxModuleCalls / 8):
             callGraphStr += '  "' + modName + \
-                '" [fillcolor = orange style=filled];\n'
+                '" [fillcolor=green style=filled];\n'
+        elif len(modProperties['calls']) < int(maxModuleCalls / 4):
+            callGraphStr += '  "' + modName + \
+                '" [fillcolor=orange style=filled];\n'
         else:
             callGraphStr += '  "' + modName + \
-                '" [fillcolor = red style=filled];\n'
+                '" [fillcolor=red style=filled];\n'
     callGraphStr += '\n'
     # connections between modules
     for modName, modProperties in modules.items():
@@ -2920,24 +2926,39 @@ def testFunctions():
     for modName, modProperties in modules.items():
         callGraphStr += '  subgraph cluster_' + modName + ' {\n'
         callGraphStr += '    label = "' + modName + '";\n'
-        callGraphStr += '    node '
-        callGraphStr += '[style=filled fillcolor='
-        callGraphStr += modProperties['color'] + '];\n'
+        callGraphStr += '    node [style=filled];\n'
         moduleFunctionsStr = ''
         for name in modProperties['functions']:
             if name.startswith('test'):
                 continue
             if name not in excludeFuncs:
-                moduleFunctionsStr += '"' + name + '" '
+                if not functionProperties[name]['calls']:
+                    moduleFunctionsStr += \
+                        '  "' + name + '" [fillcolor=yellow style=filled];\n'
+                    continue
+                noOfCalls = len(functionProperties[name]['calls'])
+                if noOfCalls < int(maxFunctionCalls / 4):
+                    moduleFunctionsStr += '  "' + name + \
+                        '" [fillcolor=orange style=filled];\n'
+                else:
+                    moduleFunctionsStr += '  "' + name + \
+                        '" [fillcolor=red style=filled];\n'
+
         if moduleFunctionsStr:
-            callGraphStr += '    ' + moduleFunctionsStr + ';\n'
+            callGraphStr += moduleFunctionsStr + '\n'
         callGraphStr += '    color=blue;\n'
         callGraphStr += '  }\n\n'
 
     for name, properties in functionProperties.items():
         if not properties['calls']:
             continue
-        modColor = modules[properties['module']]['color']
+        noOfCalls = len(properties['calls'])
+        if noOfCalls <= int(maxFunctionCalls / 8):
+            modColor = 'blue'
+        elif noOfCalls < int(maxFunctionCalls / 4):
+            modColor = 'green'
+        else:
+            modColor = 'red'
         for calledFunc in properties['calls']:
             if calledFunc.startswith('test'):
                 continue