Add __str__ methods for actions

[leonardt]

Convenience methods for debugging.

For example, with this change

diff --git a/fault/magma_simulator_target.py b/fault/magma_simulator_target.py
index 8bfc66b..2b09d84 100644
--- a/fault/magma_simulator_target.py
+++ b/fault/magma_simulator_target.py
@@ -24,6 +24,7 @@ class MagmaSimulatorTarget(Target):
     def run(self):
         simulator = self.backend_cls(self.circuit, self.clock)
         for action in self.actions:
+            print(action)
             if isinstance(action, actions.Poke):
                 value = action.value
                 # Python simulator does not support setting Bit with

You get a nice printout, such as

Poke(PISO10.PI, 956)
Poke(PISO10.SI, 0)
Poke(PISO10.LOAD, 1)
Step(2, PISO10.CLK)
Poke(PISO10.PI, 255)
Poke(PISO10.SI, 0)
Poke(PISO10.LOAD, 0)
Step(2, PISO10.CLK)
Expect(PISO10.O, 1)

[rsetaluri]

maybe Step({self.clock.debug_name}, steps={self.steps}) to make it more clear? Idk how standard the 'steps' argument is.

[leonardt]

Sure, I was having it match the __init__ call, so if we go with this, I'd want to update the __init__ to match

[rsetaluri]

Fair enough. If it's not too much refactoring that seems more uniform. Otherwise, it is fine as is.

[leonardt]

Done

[coveralls]

Pull Request Test Coverage Report for Build 188

• 11 of 11 (100.0%) changed or added relevant lines in 2 files are covered.
• 3 unchanged lines in 1 file lost coverage.
• Overall coverage increased (+0.5%) to 79.529%

---

Files with Coverage Reduction	New Missed Lines	%
fault/magma_simulator_target.py	3	84.21%

Totals
Change from base Build 167:	0.5%
Covered Lines:	439
Relevant Lines:	552

---

💛 - Coveralls

[rsetaluri]

oops :D

[leonardt]

heh, that's what I get for no tests. Fixed, and pushed a test for the string methods.

[leonardt]

Oh, forgot to update the Step references in the tests directory