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

Add console test #501

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Add console test #501

wants to merge 4 commits into from

Conversation

gjmooney
Copy link
Collaborator

@gjmooney gjmooney commented Feb 26, 2025

Description

Add a UI test for the embedded console.

Checklist

  • PR has a descriptive title and content.
  • PR description contains references to any issues the PR resolves, e.g. Resolves #XXX.
  • PR has one of the labels: documentation, bug, enhancement, feature, maintenance
  • Checks are passing.
    Failing lint checks can be resolved with:
    • pre-commit run --all-files
    • jlpm run lint

📚 Documentation preview: https://jupytergis--501.org.readthedocs.build/en/501/
💡 JupyterLite preview: https://jupytergis--501.org.readthedocs.build/en/501/lite

Copy link
Contributor

Binder 👈 Launch a Binder on branch gjmooney/jupytergis/console_test

Copy link
Contributor

github-actions bot commented Feb 26, 2025

Integration tests report: appsharing.space

@arjxn-py
Copy link
Member

Bot please update snapshots again :)

@arjxn-py arjxn-py closed this Feb 26, 2025
@arjxn-py arjxn-py reopened this Feb 26, 2025
@arjxn-py
Copy link
Member

Triggering CI after snapshot update

Copy link
Member

@arjxn-py arjxn-py left a comment

Choose a reason for hiding this comment

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

Looks pretty good.
Thanks :)

@gjmooney
Copy link
Collaborator Author

please update snapshots

@gjmooney gjmooney closed this Feb 26, 2025
@gjmooney gjmooney reopened this Feb 26, 2025
@martinRenou
Copy link
Member

Mmh. How about juste taking a screenshot of the console, which will be stable 😅

@gjmooney
Copy link
Collaborator Author

Mmh. How about juste taking a screenshot of the console, which will be stable 😅

Making the layer prints the ID of it! Nothing is stable! These screenshots have it out for me!

@arjxn-py
Copy link
Member

Maybe try taking a screenshot without adding the layer?

@martinRenou
Copy link
Member

Making the layer prints the ID of it!

You can fix that by adding a ; at the end of your executed Python code in the console, it will make it stop printing anything

.getByRole('textbox')
.filter({ hasText: /^$/ })
.locator('div')
.fill('doc.add_geojson_layer(path="testDir/eq.json")');
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
.fill('doc.add_geojson_layer(path="testDir/eq.json")');
.fill('doc.add_geojson_layer(path="testDir/eq.json");');

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants