jump(direction): returns true if the jump was successful. Alright, this is correct.

Working on new variable, and documentation.

jump(direction): returns true if the jump was successful. Alright, this is correct.

Working on the new variable and the documentation.

Does setting exit_roots to reverse of history value appear good to you?

When history = true, exit_roots = false which means snippets will not exit at $0.
Similarly, when history = false, exit_roots = true which means snippets exit at $0.

This simulates old behavior, correctly, does it not?

Secondly, do you think it is good idea to leave the outer if statement as it is?

		if session.config.exit_roots then
			if (next_node.pos == 0 and next_node.parent.parent_node == nil) then
				session.current_nodes[vim.api.nvim_get_current_buf()] = nil
				return true
			end
		end

Naturally, can take it into the inner if statement, but it appears harder to read.

In reply to @gokberkgunes:matrix.org

Does setting exit_roots to reverse of history value appear good to you?

When history = true, exit_roots = false which means snippets will not exit at $0.
Similarly, when history = false, exit_roots = true which means snippets exit at $0.

This simulates old behavior, correctly, does it not?

In reply to @gokberkgunes:matrix.org

Secondly, do you think it is good idea to leave the outer if statement as it is?

		if session.config.exit_roots then
			if (next_node.pos == 0 and next_node.parent.parent_node == nil) then
				session.current_nodes[vim.api.nvim_get_current_buf()] = nil
				return true
			end
		end

Naturally, can take it into the inner if statement, but it appears harder to read.

Here is the file

local ls_helpers = require("helpers")
local exec_lua, feed, exec =
    ls_helpers.exec_lua, ls_helpers.feed, ls_helpers.exec
local Screen = require("test.functional.ui.screen")

describe("snippets_basic", function()
	local screen

	before_each(function()
		ls_helpers.clear()
		ls_helpers.session_setup_luasnip()

		screen = Screen.new(50, 3)
		screen:attach()
		screen:set_default_attr_ids({
			[0] = { bold = true, foreground = Screen.colors.Blue },
			[1] = { bold = true, foreground = Screen.colors.Brown },
			[2] = { bold = true },
			[3] = { background = Screen.colors.LightGray },
		})
	end)

	after_each(function()
		screen:detach()
	end)


	it("exit_roots exits when the last node of snippet-root is reached.", function()
		exec_lua([[
			ls.setup({
				exit_roots = true
			})

			ls.add_snippets("all", {
			s("aa", { t{"( "}, i(1, "1"), t{" )"}, i(0, "0") })
			})
		]])

		feed("iaa")
		exec_lua("ls.expand()")
		screen:expect({
			grid = [[
			( ^1 )0                                            |
			{0:~                                                 }|
			{2:-- SELECT --}                                      |]]
		})
		feed("aa")
		exec_lua("ls.expand()")
		screen:expect({
			grid = [[
			( ( ^1 )0 )0                                       |
			{0:~                                                 }|
			{2:-- SELECT --}                                      |]]
		})
		-- verify we do not exit when reaching to a child root
		exec_lua("ls.jump(1) ls.jump(-1)")
		screen:expect({
			grid = [[
			( ( ^1 )0 )0                                       |
			{0:~                                                 }|
			{2:-- SELECT --}                                      |]]
		})
		-- be sure that reaching root $0 exits.
		exec_lua("ls.jump(1) ls.jump(1) ls.jump(-1)")
		screen:expect({
			grid = [[
			( ( 1 )0 )^0                                       |
			{0:~                                                 }|
			{2:-- SELECT --}                                      |]]
		})
	end)
end)

Any preferences to use in a test function below? Additionally, is there any difference in changing setup as given in 2nd case? I am worried if first approach will alter behavior of other tests.

ls_helpers.session_setup_luasnip({ setup_extend = { exit_roots = false}, hl_choiceNode = true})

exec_lua("ls.setup(vim.tbl_extend('force', ls.session.config, { exit_roots = true }))")