pogmommy/zellij
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(cli): measure cwd from cli client rather than the zellij server (#1947)

Browse source 
* fix(cli): measure cwd from cli client rather than the zellij server

* style(fmt): rustfmt
This commit is contained in:
Aram Drevekenin 2022-11-16 16:25:01 +01:00 • committed by GitHub
parent ed64cff9b5
commit cc3ac25c74
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 36 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/commands.rs
View file

@ -238,7 +238,8 @@ pub(crate) fn convert_old_theme_file(old_theme_file: PathBuf) {
fn attach_with_cli_client(cli_action: zellij_utils::cli::CliAction, session_name: &str) {
let os_input = get_os_input(zellij_client::os_input_output::get_cli_client_os_input);
match Action::actions_from_cli(cli_action) {
let get_current_dir = || std::env::current_dir().unwrap_or_else(|_| PathBuf::from("."));
match Action::actions_from_cli(cli_action, Box::new(get_current_dir)) {
Ok(actions) => {
zellij_client::cli_client::start_cli_client(Box::new(os_input), session_name, actions);
std::process::exit(0);

3
zellij-server/src/unit/screen_tests.rs
View file

@ -107,7 +107,8 @@ fn send_cli_action_to_server(
) {
let os_input = Box::new(mock_screen.os_input.clone());
let to_server = mock_screen.to_server.clone();
let actions = Action::actions_from_cli(cli_action).unwrap();
let get_current_dir = || PathBuf::from(".");
let actions = Action::actions_from_cli(cli_action, Box::new(get_current_dir)).unwrap();
for action in actions {
route_action(
action,

20
zellij-server/src/unit/snapshots/zellij_server__screen__screen_tests__send_cli_new_tab_action_with_name_and_layout.snap
View file

@ -1,6 +1,6 @@
---
source: zellij-server/src/./unit/screen_tests.rs
assertion_line: 1989
assertion_line: 2287
expression: "format!(\"{:#?}\", new_tab_instruction)"
---
NewTab(
@ -15,7 +15,11 @@ NewTab(
name: None,
children: [],
split_size: None,
run: None,
run: Some(
Cwd(
".",
),
),
borderless: false,
focus: None,
external_children_index: None,
@ -25,7 +29,11 @@ NewTab(
name: None,
children: [],
split_size: None,
run: None,
run: Some(
Cwd(
".",
),
),
borderless: false,
focus: None,
external_children_index: None,
@ -35,7 +43,11 @@ NewTab(
name: None,
children: [],
split_size: None,
run: None,
run: Some(
Cwd(
".",
),
),
borderless: false,
focus: None,
external_children_index: None,

19
zellij-utils/src/input/actions.rs
View file

@ -228,7 +228,10 @@ pub enum Action {
}
impl Action {
pub fn actions_from_cli(cli_action: CliAction) -> Result<Vec<Action>, String> {
pub fn actions_from_cli(
cli_action: CliAction,
get_current_dir: Box<dyn Fn() -> PathBuf>,
) -> Result<Vec<Action>, String> {
match cli_action {
CliAction::Write { bytes } => Ok(vec![Action::Write(bytes)]),
CliAction::WriteChars { chars } => Ok(vec![Action::WriteChars(chars)]),
@ -265,7 +268,10 @@ impl Action {
if !command.is_empty() {
let mut command = command.clone();
let (command, args) = (PathBuf::from(command.remove(0)), command);
let cwd = cwd.or_else(|| std::env::current_dir().ok());
let current_dir = get_current_dir();
let cwd = cwd
.map(|cwd| current_dir.join(cwd))
.or_else(|| Some(current_dir));
let hold_on_start = start_suspended;
let hold_on_close = !close_on_exit;
let run_command_action = RunCommandAction {
@ -304,7 +310,10 @@ impl Action {
cwd,
} => {
let mut file = file;
let cwd = cwd.or_else(|| std::env::current_dir().ok());
let current_dir = get_current_dir();
let cwd = cwd
.map(|cwd| current_dir.join(cwd))
.or_else(|| Some(current_dir));
if file.is_relative() {
if let Some(cwd) = cwd {
file = cwd.join(file);
@ -338,6 +347,10 @@ impl Action {
]),
CliAction::UndoRenameTab => Ok(vec![Action::UndoRenameTab]),
CliAction::NewTab { name, layout, cwd } => {
let current_dir = get_current_dir();
let cwd = cwd
.map(|cwd| current_dir.join(cwd))
.or_else(|| Some(current_dir));
if let Some(layout_path) = layout {
let (path_to_raw_layout, raw_layout) =
Layout::stringified_from_path_or_default(Some(&layout_path), None)