fix Correct if syntax (#167)

* Fix the syntax in if_else.sh

The test script `if_else.sh` has incorrect syntax for an if statement:

    $ bash if_else.sh
    if_else.sh: line 2: syntax error near unexpected token `then'
    if_else.sh: line 2: `if [[ $FOO -eq 1 ]] then'

The correct syntax uses `;` before `then`.

* fix if syntax

* Added from_file to TestBuilder and minor fix in if parser

* Update function name to script_file due to from_* conflict

---------

Co-authored-by: Ondřej Čertík <ondrej@certik.us>

Author: prsabahrami

crates/deno_task_shell/src/grammar.pest

@@ -284,13 +284,13 @@ if_clause = !{
 }
 
 else_part = !{
-    Elif ~ conditional_expression ~ Then ~ complete_command ~ linebreak ~ else_part? |
+    Elif ~ conditional_expression ~ linebreak ~ Then ~ complete_command ~ linebreak ~ else_part? |
     Else ~ linebreak ~ complete_command
 }
 
 conditional_expression = !{
-    ("[[" ~ (unary_conditional_expression | binary_conditional_expression | UNQUOTED_PENDING_WORD) ~ "]]") |
-    ("[" ~ (unary_conditional_expression | binary_conditional_expression | UNQUOTED_PENDING_WORD) ~ "]") | 
+    ("[[" ~ (unary_conditional_expression | binary_conditional_expression | UNQUOTED_PENDING_WORD) ~ "]]" ~ ";"?) |
+    ("[" ~ (unary_conditional_expression | binary_conditional_expression | UNQUOTED_PENDING_WORD) ~ "]" ~ ";"?) | 
     ("test" ~ (unary_conditional_expression | binary_conditional_expression | UNQUOTED_PENDING_WORD))
 }
 

crates/tests/src/lib.rs

@@ -887,6 +887,36 @@ async fn date() {
         .await;
 }
 
+#[tokio::test]
+async fn if_clause() {
+    TestBuilder::new()
+        .command(r#"FOO=2; if [[ $FOO == 1 ]]; then echo "FOO is 1"; elif [[ $FOO -eq 2 ]]; then echo "FOO is 2"; else echo "FOO is not 1 or 2"; fi"#)
+        .assert_stdout("FOO is 2\n")
+        .run()
+        .await;
+    TestBuilder::new()
+        .command(r#"FOO=3; if [[ $FOO == 1 ]]; then echo "FOO is 1"; elif [[ $FOO -eq 2 ]]; then echo "FOO is 2"; else echo "FOO is not 1 or 2"; fi"#)
+        .assert_stdout("FOO is not 1 or 2\n")
+        .run()
+        .await;
+
+    TestBuilder::new()
+        .command(r#"FOO=1; if [[ $FOO == 1 ]]; then echo "FOO is 1"; elif [[ $FOO -eq 2 ]]; then echo "FOO is 2"; else echo "FOO is not 1 or 2"; fi"#)
+        .assert_stdout("FOO is 1\n")
+        .run()
+        .await;
+
+    TestBuilder::new()
+        .script_file("../../scripts/if_else.sh")
+        .assert_exit_code(0)
+        .assert_stdout("FOO is 2\n")
+        .assert_stdout("FOO is 2\n")
+        .assert_stdout("FOO is 2\n")
+        .assert_stdout("FOO is 2\n")
+        .run()
+        .await;
+}
+
 #[cfg(test)]
 fn no_such_file_error_text() -> &'static str {
     if cfg!(windows) {

crates/tests/src/test_builder.rs

@@ -128,6 +128,11 @@ impl TestBuilder {
         self
     }
 
+    pub fn script_file(&mut self, path: &str) -> &mut Self {
+        self.command(fs::read_to_string(path).unwrap().as_str());
+        self
+    }
+
     pub fn stdin(&mut self, stdin: &str) -> &mut Self {
         self.stdin = stdin.as_bytes().to_vec();
         self

scripts/if_else.sh

@@ -1,8 +1,41 @@
 FOO=2
-if [[ $FOO -eq 1 ]] then
+if [[ $FOO -eq 1 ]];
+then
+    echo "FOO is 1";
+elif [[ $FOO -eq 2 ]];
+then
+    echo "FOO is 2";
+else
+    echo "FOO is not 1 or 2";
+fi
+
+FOO=2
+if [[ $FOO -eq 1 ]]; then
     echo "FOO is 1"
-elif [[ $FOO -eq 2 ]] then  
+elif [[ $FOO -eq 2 ]]; then
     echo "FOO is 2"
 else
     echo "FOO is not 1 or 2"
+fi
+
+FOO=2
+if [[ $FOO -eq 1 ]];
+then
+    echo "FOO is 1";
+elif [[ $FOO -eq 2 ]];
+then
+    echo "FOO is 2";
+else
+    echo "FOO is not 1 or 2";
+fi
+
+FOO=2
+if [[ $FOO -eq 1 ]]
+then
+    echo "FOO is 1";
+elif [[ $FOO -eq 2 ]]
+then
+    echo "FOO is 2";
+else
+    echo "FOO is not 1 or 2";
 fi