CharacterBasedErikaMock is working well + smooth, way better than the old implementation that dumped the whole internal array to the commandline.
MicrostepBasedErikaMock still uses this old implementation, and should also be refactored.
It is used as part of run_manual_tests.sh and spams the output quite a bit.
In addition, this is a good opportunity to get rid of redundancy among AbstractErikaMock, MicrostepBasedErikaMock, CharacterBasedErikaMock.
SHOULD be done after #52 is merged, or based on the feature/deletebranch to avoid merge issues / future rework
Scope
Do not yet remove the whole internal representation of the rendered content as two-dimensional array
meaning CharacterBasedErikaMock.canvas resp MicrostepBasedErikaMock.canvas
CharacterBasedErikaMockis working well + smooth, way better than the old implementation that dumped the whole internal array to the commandline.MicrostepBasedErikaMockstill uses this old implementation, and should also be refactored. It is used as part ofrun_manual_tests.shand spams the output quite a bit.In addition, this is a good opportunity to get rid of redundancy among
AbstractErikaMock,MicrostepBasedErikaMock,CharacterBasedErikaMock.SHOULD be done after #52 is merged, or based on the
feature/deletebranch to avoid merge issues / future reworkScope
CharacterBasedErikaMock.canvasrespMicrostepBasedErikaMock.canvasinstr(y, x [,n])method.