Slither does not handle multiple contracts with the same name across compilation units

[hacker-DOM]

Consider the codebase https://github.com/hacker-DOM/reference-resolving/tree/ecd8bf0e91a3636104b5ffc805537eaac2eeb8d7

These are the results for inheritance-graph and call-graph, both of which are wrong

[montyly]

Indeed the printers merge contracts with the same name, which they should not. We will update the printer.

[0xalpharush]

We need to make these id's unique across compilation unit

slither/slither/printers/call/call_graph.py

Lines 20 to 25 in e5f2a86

	return f"cluster_{contract.id}_{contract.name}"
	# return unique id for contract function to use as node name
	def _function_node(contract: Contract, function: Union[Function, Variable]) -> str:
	return f"{contract.id}_{function.name}"

slither/slither/printers/inheritance/inheritance_graph.py

Lines 103 to 106 in e5f2a86

	ret += f"{contract.name} -> {contract.immediate_inheritance[0]};\n"
	else:
	for i, immediate_inheritance in enumerate(contract.immediate_inheritance):
	ret += f'{contract.name} -> {immediate_inheritance} [ label="{i + 1}" ];\n'