NO-ISSUE: Take screenshot after each console key press (#1768)

* Take screenshot after each console key press

Add additional screenshots to help with debugging issues with
console interaction in CI tests.

* Add 3s sleep after ESC from network tree view

The bad_dns tests appears to fail when the Tab key press to
move to the <Quit> button is not being registered. At the end
of the test Enter key is pressed but instead of quiting
agent-tui, it enters nmtui because the cursor is still on the
<Configure Network> button.

Adding a 3s sleep after escaping from the the network tree view
should give it more time to properly exit and return to
agent-tui.

* Add an additional ESC key press after exiting nmtui

There appears to be a blank screen in some environments that
that appears after exiting nmtui. The ESC key then exits out of
this blank screen and into the network tree view. Adding
an extra ESC key press to fully exit out of nmtui and tree view.

In environments where the blank screen does not appear, the
extra ESC key press is innocuous.

* Quit agent-tui using the q key

Navigating to the <Quit> button using TAB or LEFT sometimes does
not register when sent through "virsh send-key" in the CI.

openshift/agent-installer-utils#107
introduced an alternative way to exit agent-tui from the checks
page.

Author: rwsu

agent/e2e/agent-tui/test-fix-wrong-dns.sh

@@ -63,10 +63,10 @@ fi
 pressDown "Select Quit" ${numStepsToQuit}
 
 pressEnter "Select 'Quit' menu item"
-pressEsc "Esc from network tree view"
-pressTab "Goto <Quit> button to exit agent-tui" 1
+pressEsc "Esc from network tree view" 2
+sleep 3
 
 # wait for check to update, to visually see the release image
 # check change to passing.
 sleep 10
-pressEnter "Exit agent-tui"
+pressKeys "Exit agent-tui" "qq"

agent/e2e/agent-tui/utils.sh

@@ -14,6 +14,9 @@ function _pressKey() {
   fi
   sudo virsh send-key $node_name $keyCode
 
+  timestamp=$(date +"%Y-%m-%d--%H:%M:%S")
+  sudo virsh screenshot $node_name "${OCP_DIR}/${timestamp}-${node_name}_after_${keyCode}.ppm"
+
   # On some CI instances, the sequence of events appears to be too fast
   # for the console refresh, leading the test in the wrong state.
   # Let's add a small pause between one keypress event and the subsequent
@@ -44,7 +47,7 @@ function pressEnter() {
 
 function pressTab() {
   pressKey "$1" KEY_TAB "$2"
-}	
+}
 
 function pressDown() {
   pressKey "$1" KEY_DOWN "$2" "$3"
@@ -54,7 +57,6 @@ function pressBackspace() {
   pressKey "$1" KEY_BACKSPACE "$2"
 }
 
-
 function pressEsc() {
   pressKey "$1" KEY_ESC "$2"
 }