fix v_o should update the cursor x position

Close NeoVintageous#974

CHANGELOG.md

@@ -68,6 +68,7 @@ All notable changes are documented in this file using the [Keep a CHANGELOG](htt
 
 ### Fixed
 
+- [#974](https://github.com/NeoVintageous/NeoVintageous/issues/974): `v_o` should update the cursor x position.
 - [#963](https://github.com/NeoVintageous/NeoVintageous/issues/963): Don't populate small delete when a register is specified.
 - [#962](https://github.com/NeoVintageous/NeoVintageous/issues/962): `zt`, `zb`, and `zz` should maintain cursor column.
 - [#962](https://github.com/NeoVintageous/NeoVintageous/issues/962): `z<CR>`, `z-`, and `z.` should position the cursor on the first non-blank.

nv/vi/cmd_defs.py

@@ -77,7 +77,7 @@ def init(self):
 class ViGoToOtherEnd(ViOperatorDef):
     def init(self):
         self.scroll_into_view = True
-        self.updates_xpos = False
+        self.updates_xpos = True
         self.command = 'nv_vi_visual_o'
 
 

tests/functional/test_o.py

@@ -38,6 +38,11 @@ def test_V(self):
         self.eq('r_x\n|fizz\n|x', 'V_o', 'x\n|fizz\n|x')
         self.assertStatusLineIsVisualLine()
 
+    def test_issue_974_xpos_should_be_updated(self):
+        self.eq('fizz\nfi|zz\nfizz buzz\nfizz bu|zz', 'v_o', 'r_fizz\nfi|zz\nfizz buzz\nfizz bu|zz')
+        self.feed('j')
+        self.assertRVisual('fizz\nfizz\nfi|zz buzz\nfizz bu|zz')
+
 
 class Test_o_auto_indent(unittest.FunctionalTestCase):