Skip to content

Commit 1845152

Browse files
committed
fixes r
1 parent de3bb5d commit 1845152

File tree

4 files changed

+18
-17
lines changed

4 files changed

+18
-17
lines changed

js/tests/env_vars/r.test.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ sandboxTest.skipIf(isDebug)('env vars on sandbox (R)', async () => {
1414
language: 'r',
1515
})
1616

17-
expect(result.results[0]?.text.trim()).toEqual('supertest')
17+
expect(result.results[0]?.text.trim()).toEqual(`[1] "supertest"`)
1818
} finally {
1919
await sandbox.kill()
2020
}
@@ -23,6 +23,7 @@ sandboxTest.skipIf(isDebug)('env vars on sandbox (R)', async () => {
2323
sandboxTest('env vars per execution (R)', async ({ sandbox }) => {
2424
const result = await sandbox.runCode('Sys.getenv("FOO")', {
2525
envs: { FOO: 'bar' },
26+
language: 'r',
2627
})
2728

2829
const result_empty = await sandbox.runCode(
@@ -32,8 +33,8 @@ sandboxTest('env vars per execution (R)', async ({ sandbox }) => {
3233
}
3334
)
3435

35-
expect(result.results[0]?.text.trim()).toEqual('bar')
36-
expect(result_empty.results[0]?.text.trim()).toEqual('default')
36+
expect(result.results[0]?.text.trim()).toEqual(`[1] "bar"`)
37+
expect(result_empty.results[0]?.text.trim()).toEqual(`[1] "default"`)
3738
})
3839

3940
sandboxTest.skipIf(isDebug)('env vars overwrite', async () => {
@@ -47,7 +48,7 @@ sandboxTest.skipIf(isDebug)('env vars overwrite', async () => {
4748
envs: { TEST_ENV_VAR: 'overwrite' },
4849
})
4950

50-
expect(result.results[0]?.text.trim()).toEqual('overwrite')
51+
expect(result.results[0]?.text.trim()).toEqual(`[1] "overwrite"`)
5152
} finally {
5253
await sandbox.kill()
5354
}

python/tests/async/env_vars/test_r.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,32 +6,32 @@ async def test_env_vars_on_sandbox():
66
sandbox = await AsyncSandbox.create(envs={"TEST_ENV_VAR": "supertest"})
77
try:
88
result = await sandbox.run_code(
9-
"Sys.getenv('TEST_ENV_VAR')",
9+
'Sys.getenv("TEST_ENV_VAR")',
1010
language="r"
1111
)
1212
assert result.text is not None
13-
assert result.text.strip() == "supertest"
13+
assert result.text.strip() == '[1] "supertest"'
1414
finally:
1515
await sandbox.kill()
1616

1717
async def test_env_vars_per_execution():
1818
sandbox = await AsyncSandbox.create()
1919
try:
2020
result = await sandbox.run_code(
21-
"Sys.getenv('FOO')",
21+
'Sys.getenv("FOO")',
2222
envs={"FOO": "bar"},
2323
language="r"
2424
)
2525

2626
result_empty = await sandbox.run_code(
27-
"Sys.getenv('FOO', unset = 'default')",
27+
'Sys.getenv("FOO", unset = "default")',
2828
language="r"
2929
)
3030

3131
assert result.text is not None
32-
assert result.text.strip() == "bar"
32+
assert result.text.strip() == '[1] "bar"'
3333
assert result_empty.text is not None
34-
assert result_empty.text.strip() == "default"
34+
assert result_empty.text.strip() == '[1] "default"'
3535
finally:
3636
await sandbox.kill()
3737

@@ -40,11 +40,11 @@ async def test_env_vars_overwrite():
4040
sandbox = await AsyncSandbox.create(envs={"TEST_ENV_VAR": "supertest"})
4141
try:
4242
result = await sandbox.run_code(
43-
"Sys.getenv('TEST_ENV_VAR')",
43+
'Sys.getenv("TEST_ENV_VAR")',
4444
language="r",
4545
envs={"TEST_ENV_VAR": "overwrite"}
4646
)
4747
assert result.text is not None
48-
assert result.text.strip() == "overwrite"
48+
assert result.text.strip() == '[1] "overwrite"'
4949
finally:
5050
await sandbox.kill()

python/tests/sync/env_vars/test_r.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ def test_env_vars_on_sandbox():
99
"Sys.getenv('TEST_ENV_VAR')",
1010
language="r"
1111
)
12-
assert result.results[0].text.strip() == "supertest"
12+
assert result.results[0].text.strip() == '[1] "supertest"'
1313
finally:
1414
sandbox.kill()
1515

@@ -27,8 +27,8 @@ def test_env_vars_per_execution():
2727
language="r"
2828
)
2929

30-
assert result.results[0].text.strip() == "bar"
31-
assert result_empty.results[0].text.strip() == "default"
30+
assert result.results[0].text.strip() == '[1] "bar"'
31+
assert result_empty.results[0].text.strip() == '[1] "default"'
3232
finally:
3333
sandbox.kill()
3434

@@ -41,6 +41,6 @@ def test_env_vars_overwrite():
4141
language="r",
4242
envs={"TEST_ENV_VAR": "overwrite"}
4343
)
44-
assert result.results[0].text.strip() == "overwrite"
44+
assert result.results[0].text.strip() == '[1] "overwrite"'
4545
finally:
4646
sandbox.kill()

template/server/messaging.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ async def set_env_vars(self, env_vars: Dict[StrictStr, str]):
148148
elif self.language == "deno":
149149
env_commands.append(f"Deno.env.set('{k}', '{v}')")
150150
elif self.language == "r":
151-
env_commands.append(f"Sys.setenv('{k}' = '{v}')")
151+
env_commands.append(f'Sys.setenv({k} = "{v}")')
152152
elif self.language == "java":
153153
env_commands.append(f'System.setProperty("{k}", "{v}")')
154154
elif self.language == "bash":

0 commit comments

Comments
 (0)