Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use format SQL settings when creating new notebook #115

Merged
merged 3 commits into from
Jun 14, 2024
Merged

Conversation

neelasha23
Copy link
Contributor

Describe your changes

Issue number

Closes #114

Checklist before requesting a review

@neelasha23 neelasha23 marked this pull request as draft June 12, 2024 09:25
remove snowflake tests

rm redshift snowflake

added load_extension

remove tests

serverapp

revert

jl version

removed newlines

insert btn

revert

revert

revert

test

original code

revert test

timeout
@neelasha23 neelasha23 force-pushed the issue114 branch 2 times, most recently from afcd0b2 to 4bd8bd7 Compare June 14, 2024 10:25
newlines

newlines

newlines
@neelasha23 neelasha23 marked this pull request as ready for review June 14, 2024 10:34
@@ -6,7 +6,7 @@ test('test format SQL', async ({ page }) => {
await page.notebook.openByPath("sample.ipynb");
await page.notebook.activate("sample.ipynb");
await page.notebook.addCell("code", "%%sql\nselect * from table")
await page.getByTestId('format-btn').locator('button').click();
await page.getByTestId('format-btn').locator('button').click({ force: true });
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I faced this issue in the CI on the main branch as well as the current branch:

waiting 500ms
      -   waiting for element to be visible, enabled and stable
      -   element is visible, enabled and stable
      -   scrolling into view if needed
      -   done scrolling
      -   <span class="jp-ToolbarButtonComponent-label">Format SQL</span> intercepts pointer events
      - retrying click action, attempt #93 

Fixed using this solution

@edublancas edublancas merged commit 3a602cf into main Jun 14, 2024
7 checks passed
@edublancas edublancas deleted the issue114 branch June 14, 2024 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

hide format SQL button
2 participants