add: test scripts and documentation for PATH directory order testing in minishell

Author: cuajarsaki

docs/dir1/test

@@ -0,0 +1,2 @@
+#!/bin/bash
+echo "This is from dir1"

docs/dir2/test

@@ -0,0 +1,2 @@
+#!/bin/bash
+echo "This is from dir2"

docs/path-testing-en.md

@@ -0,0 +1,79 @@
+# Guide: Testing PATH Directory Order in Minishell
+This document explains how to test whether a minishell implementation correctly searches for executables in PATH directories in left-to-right order.
+
+## Test Setup (Already in Repository)
+The following test environment already exists in the repository:
+1. Test directory structure:
+   ```
+   docs/dir1
+   docs/dir2
+   ```
+
+2. Test executable in the first directory:
+   ```
+   docs/dir1/test (executable, outputs "This is from dir1")
+   ```
+
+3. Test executable with the same name in the second directory:
+   ```
+   docs/dir2/test (executable, outputs "This is from dir2")
+   ```
+
+4. Verify both scripts work correctly by running them directly:
+   ```bash
+   ./docs/dir1/test    # Should print "This is from dir1"
+   ./docs/dir2/test    # Should print "This is from dir2"
+   ```
+
+## Testing in Minishell
+1. Start minishell implementation:
+   ```bash
+   ./minishell
+   ```
+
+2. Set the PATH environment variable with dir1 before dir2:
+   ```bash
+   export PATH=/full_path_to/docs/dir1:/full_path_to/docs/dir2:$PATH
+   ```
+   (Replace `/full_path_to/` with the actual path, e.g., `$(pwd)/`)
+
+3. Verify the PATH is set correctly:
+   ```bash
+   echo $PATH
+   ```
+
+4. Run the test command:
+   ```bash
+   test
+   ```
+   If PATH ordering is correct, this should execute dir1/test and print "This is from dir1"
+
+5. Reverse the order to verify behavior changes:
+   ```bash
+   export PATH=/full_path_to/docs/dir2:/full_path_to/docs/dir1:$PATH
+   test
+   ```
+   This should now print "This is from dir2"
+
+## Troubleshooting
+If you get "command not found":
+- Ensure the paths in PATH are absolute, not relative
+- Check that the directories exist and are accessible
+- Verify the command name doesn't conflict with built-ins
+
+If you get "Exec format error":
+- Ensure the script has the correct line endings (LF, not CRLF)
+- Verify the shebang interpreter exists (#!/bin/bash)
+- Check file permissions are correct (chmod +x)
+
+If the wrong script executes:
+- Use `which test` to see which path is being found
+- Check if a system command with the same name exists
+- Try with a unique command name that doesn't conflict with system commands
+
+## Implementation Notes
+For minishell implementation, ensure:
+1. PATH is split by colons into an array of directories
+2. Directories are searched in order from left to right
+3. The first executable found with the requested name is executed
+4. If no executable is found in any directory, "command not found" is returned

docs/path-testing-ja.md

@@ -0,0 +1,81 @@
+こちらのガイドは、Minishellでの PATH ディレクトリの順序テストに関する説明ですね。すでにリポジトリにテスト環境があることを明記したいとのことですので、以下のように修正しました：
+
+# Minishellにおける PATH ディレクトリ順序テストガイド
+このドキュメントでは、Minishell実装が PATH 環境変数内のディレクトリを左から右への順序で正しく検索するかをテストする方法を説明します。
+
+## テスト環境（すでにリポジトリに存在）
+以下のテスト環境はすでにリポジトリに存在しています：
+1. テストディレクトリ構造:
+   ```
+   docs/dir1
+   docs/dir2
+   ```
+
+2. 最初のディレクトリ内のテスト実行ファイル:
+   ```
+   docs/dir1/test (実行可能、"This is from dir1"を出力)
+   ```
+
+3. 2番目のディレクトリ内の同名テスト実行ファイル:
+   ```
+   docs/dir2/test (実行可能、"This is from dir2"を出力)
+   ```
+
+4. 各スクリプトが正常に動作することを確認:
+   ```bash
+   ./docs/dir1/test    # "This is from dir1"と表示されるはず
+   ./docs/dir2/test    # "This is from dir2"と表示されるはず
+   ```
+
+## Minishellでのテスト手順
+1. Minishell実装を起動:
+   ```bash
+   ./minishell
+   ```
+
+2. PATH環境変数をdir1がdir2よりも前になるように設定:
+   ```bash
+   export PATH=/full_path_to/docs/dir1:/full_path_to/docs/dir2:$PATH
+   ```
+   (`/full_path_to/`は実際のパスに置き換え、例えば`$(pwd)/`など)
+
+3. PATHが正しく設定されていることを確認:
+   ```bash
+   echo $PATH
+   ```
+
+4. テストコマンドを実行:
+   ```bash
+   test
+   ```
+   PATH順序が正しければ、dir1/testが実行され「This is from dir1」と表示されるはず
+
+5. 順序を逆にして動作の変化を確認:
+   ```bash
+   export PATH=/full_path_to/docs/dir2:/full_path_to/docs/dir1:$PATH
+   test
+   ```
+   これで「This is from dir2」と表示されるはず
+
+## トラブルシューティング
+「command not found」が表示される場合:
+- PATHのパスが相対パスではなく絶対パスであることを確認
+- ディレクトリが存在しアクセス可能であることを確認
+- コマンド名がビルトインと競合していないか確認
+
+「Exec format error」が表示される場合:
+- スクリプトの改行コードが正しいこと（CRLF ではなく LF）を確認
+- シェバンインタプリタ（#!/bin/bash）が存在することを確認
+- ファイルのパーミッションが正しいこと（chmod +x）を確認
+
+間違ったスクリプトが実行される場合:
+- `which test` を使用してどのパスが見つかっているかを確認
+- 同じ名前のシステムコマンドが存在するか確認
+- システムコマンドと競合しない一意のコマンド名を試す
+
+## 実装に関する注意点
+Minishell実装では、以下を確認してください:
+1. PATHはコロンで区切られてディレクトリの配列に分割される
+2. ディレクトリは左から右の順序で検索される
+3. 要求された名前の最初に見つかった実行可能ファイルが実行される
+4. どのディレクトリにも実行可能ファイルが見つからない場合、「command not found」が返される

src/executor/executor_path.c

@@ -6,7 +6,7 @@
 /*   By: pchung <[email protected]>              +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2025/03/08 23:24:32 by pchung            #+#    #+#             */
-/*   Updated: 2025/03/08 23:32:07 by pchung           ###   ########.fr       */
+/*   Updated: 2025/03/11 12:14:16 by pchung           ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -16,21 +16,15 @@ static char	*search_executable_in_paths(const char *cmd, char *paths)
 {
 	char	*dir;
 	char	full_path[1024];
-	size_t	dir_len;
-	size_t	cmd_len;
 
 	dir = ft_strtok(paths, ":");
 	while (dir)
 	{
 		ft_snprintf(full_path, sizeof(full_path), "%s/%s", dir, cmd);
-		dir_len = ft_strlen(dir);
-		cmd_len = ft_strlen(cmd);
-		memcpy(full_path, dir, dir_len);
-		full_path[dir_len] = '/';
-		memcpy(full_path + dir_len + 1, cmd, cmd_len);
-		full_path[dir_len + cmd_len + 1] = '\0';
 		if (access(full_path, X_OK) == 0)
+		{
 			return (ft_strdup(full_path));
+		}
 		dir = ft_strtok(NULL, ":");
 	}
 	return (NULL);