Skip to content

Commit

Permalink
Updated the FQN of the unparser CLI
Browse files Browse the repository at this point in the history
  • Loading branch information
sfc-gh-azwiegincew committed Jun 4, 2024
1 parent 9c76dc9 commit e508b5d
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
3 changes: 3 additions & 0 deletions src/snowflake/snowpark/mock/_connection.py
Original file line number Diff line number Diff line change
Expand Up @@ -800,6 +800,9 @@ def create_coprocessor(self):
# It's not necessary to mock this call.
pass

def phase1_enabled(self):
return False


def _fix_pandas_df_fixed_type(table_res: TableEmulator) -> "pandas.DataFrame":
pd_df = pandas.DataFrame()
Expand Down
2 changes: 0 additions & 2 deletions tests/ast/run-unparser.sh

This file was deleted.

7 changes: 3 additions & 4 deletions tests/ast/test_ast_driver.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,6 @@

DATA_DIR = TEST_DIR / "data"

UNPARSER_SCRIPT = TEST_DIR / "run-unparser.sh"

@dataclass
class TestCase:
filename: str
Expand Down Expand Up @@ -60,10 +58,11 @@ def render(ast_base64: str) -> str:
"""Uses the unparser to render the AST."""
assert pytest.unparser_jar, "A valid Unparser JAR path must be supplied either via --unparser-jar=<path> or the environment variable SNOWPARK_UNPARSER_JAR"
res = subprocess.run([
UNPARSER_SCRIPT,
"java",
"-cp",
pytest.unparser_jar,
"com.snowflake.snowpark.experimental.unparser.UnparserCli",
ast_base64,
"--lang", "python"
], capture_output=True, text=True)
return res.stdout

Expand Down

0 comments on commit e508b5d

Please sign in to comment.