Skip to content

Commit

Permalink
E2E tests: remove web variables workarounds post 2452 fix
Browse files Browse the repository at this point in the history
  • Loading branch information
testlabauto committed Dec 18, 2024
1 parent f7a30ec commit 40e1afc
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 20 deletions.
14 changes: 0 additions & 14 deletions test/e2e/areas/data-explorer/data-explorer-python-pandas.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -138,12 +138,6 @@ df2 = pd.DataFrame(data)`;
await app.workbench.notebook.focusFirstCell();
await app.workbench.notebook.executeActiveCell();

// temporary workaround for fact that variables group
// not properly autoselected on web
if (app.web) {
await app.workbench.positronVariables.selectVariablesGroup(filename);
}

await expect(async () => {
await app.workbench.positronVariables.doubleClickVariableRow('df');
await app.code.driver.getLocator('.label-name:has-text("Data: df")').innerText();
Expand Down Expand Up @@ -190,10 +184,6 @@ Data_Frame = data('mtcars')`;
await app.workbench.positronConsole.executeCode('Python', script, '>>>');
await app.workbench.quickaccess.runCommand('workbench.panel.positronVariables.focus');

if (app.web) {
await app.workbench.positronVariables.selectVariablesGroup(`${process.env.POSITRON_PY_VER_SEL!}`);
}

await expect(async () => {
await app.workbench.positronVariables.doubleClickVariableRow('Data_Frame');
await app.code.driver.getLocator('.label-name:has-text("Data: Data_Frame")').innerText();
Expand All @@ -218,10 +208,6 @@ Data_Frame = data('mtcars')`;
df = pd.DataFrame({'x': ["a ", "a", " ", ""]})`;
await app.workbench.positronConsole.executeCode('Python', script, '>>>');

if (app.web) {
await app.workbench.positronVariables.selectVariablesGroup(`${process.env.POSITRON_PY_VER_SEL!}`);
}

await expect(async () => {
await app.workbench.positronVariables.doubleClickVariableRow('df');
await app.code.driver.getLocator('.label-name:has-text("Data: df")').innerText();
Expand Down
6 changes: 0 additions & 6 deletions test/e2e/areas/variables/variables-notebook.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,6 @@ test.describe('Variables Pane - Notebook', {

const filename = 'Untitled-1.ipynb';

// temporary workaround for fact that variables group
// not properly autoselected on web
if (app.web) {
await app.workbench.positronVariables.selectVariablesGroup(filename);
}

const interpreter = app.workbench.positronVariables.interpreterLocator;
await expect(interpreter).toBeVisible();
await expect(interpreter).toHaveText(filename);
Expand Down

0 comments on commit 40e1afc

Please sign in to comment.