fix: profile deletion does not work (#158)

s1n7ax · web-flow · commit cb62d076edf5 · 2024-04-12T13:58:13.000+05:30
diff --git a/lua/java/ui/profile.lua b/lua/java/ui/profile.lua
@@ -1,3 +1,4 @@
+local event = require('nui.utils.autocmd').event
 local Layout = require('nui.layout')
 local Menu = require('nui.menu')
 local Popup = require('nui.popup')
@@ -144,9 +145,6 @@ function ProfileUI:get_menu()
 			close = { '<Esc>', '<C-c>' },
 			submit = { '<CR>', '<Space>' },
 		},
-		on_change = function(item)
-			self.focus_item = item
-		end,
 		on_submit = function(item)
 			if item.text == new_profile then
 				self:_open_profile_editor()
@@ -327,6 +325,10 @@ end
 function ProfileUI:openMenu()
 	self.menu = self:get_menu()
 
+	self.menu:on(event.CursorMoved, function()
+		self.focus_item = self.menu.tree:get_node()
+	end)
+
 	self.menu:mount()
 	-- quit
 	self.menu:map('n', 'q', function()
@@ -352,6 +354,7 @@ local get_error_handler = require('java.handlers.error')
 
 --- @type ProfileUI
 M.ProfileUI = ProfileUI
+
 function M.ui()
 	return async(function()
 			local dap_config = DapSetup(jdtls().client):get_dap_config()
diff --git a/tests/java/ui/profile_spec.lua b/tests/java/ui/profile_spec.lua
@@ -43,6 +43,7 @@ describe('java.ui.profile', function()
 		self.table1 = table1
 		self.table2 = table2
 	end
+	function MockMenu.on() end
 	function MockMenu.unmount() end
 	function MockMenu.map() end
 	function MockMenu.mount() end
@@ -96,7 +97,6 @@ describe('java.ui.profile', function()
 			menu.table1.border.text.bottom,
 			'[a]ctivate [d]elete [b]ack [q]uit'
 		)
-		assert(menu.table2.on_change ~= nil)
 		assert(menu.table2.on_submit ~= nil)
 	end)
 
