refactor: [SNOW-1890085] post code review fixes

Author: sfc-gh-jwilkowski

src/snowflake/cli/_plugins/dbt/commands.py

@@ -130,8 +130,9 @@ def _dbt_execute(
         run_async = ctx.parent.params["run_async"]
         execute_args = (dbt_command, name, run_async, *dbt_cli_args)
 
+        result = DBTManager().execute(*execute_args)
+
         if run_async is True:
-            result = DBTManager().execute(*execute_args)
             return MessageResult(
                 f"Command submitted. You can check the result with `snow sql -q \"select execution_status from table(information_schema.query_history_by_user()) where query_id in ('{result.sfqid}');\"`"
             )
@@ -142,4 +143,4 @@ def _dbt_execute(
             transient=True,
         ) as progress:
             progress.add_task(description="Executing dbt command...", total=None)
-            return QueryResult(DBTManager().execute(*execute_args))
+            return QueryResult(result)

src/snowflake/cli/_plugins/dbt/manager.py

@@ -44,7 +44,9 @@ def deploy(
     ) -> SnowflakeCursor:
         dbt_project_path = path / "dbt_project.yml"
         if not dbt_project_path.exists():
-            raise ClickException(f"dbt_project.yml does not exist in provided path.")
+            raise ClickException(
+                f"dbt_project.yml does not exist in directory {path.path.absolute()}."
+            )
 
         if self.exists(name=name) and force is not True:
             raise ClickException(
@@ -69,6 +71,6 @@ def deploy(
 
     def execute(self, dbt_command: str, name: str, run_async: bool, *dbt_cli_args):
         if dbt_cli_args:
-            dbt_command = dbt_command + " " + " ".join([arg for arg in dbt_cli_args])
-        query = f"EXECUTE DBT PROJECT {name} args='{dbt_command.strip()}'"
+            dbt_command = " ".join([dbt_command, *dbt_cli_args]).strip()
+        query = f"EXECUTE DBT PROJECT {name} args='{dbt_command}'"
         return self.execute_query(query, _exec_async=run_async)

tests/dbt/test_dbt_commands.py

@@ -177,7 +177,7 @@ def test_raises_when_dbt_project_is_not_available(
         )
 
         assert result.exit_code == 1, result.output
-        assert "dbt_project.yml does not exist in provided path." in result.output
+        assert f"dbt_project.yml does not exist in directory" in result.output
         assert mock_connect.mocked_ctx.get_query() == ""
 
     def test_raises_when_dbt_project_exists_and_is_not_force(