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 testing infrastructure for anywidgets, plus test for layer_manager and layer_manager_row #2168

Merged
merged 40 commits into from
Nov 14, 2024

Conversation

sufyanAbbasi
Copy link
Collaborator

@sufyanAbbasi sufyanAbbasi commented Nov 11, 2024

  • Adds testing infrastructure using Karma/Jasmine to run tests for the new anywidgets.
  • These can be invoked using npm run test to build and run all tests, or
  • npm run test:watch to run the tests continuously.
    • Note that you must also run npm run dev in a separate terminal for the dev server to update the files.
  • A new browser Chrome browser window will open and start executing the tests.
  • Adds tests for layer_manager and layer_manager_row components.
  • Rearranges some CSS classes for slightly better testability.
Screenshot 2024-11-11 at 10 59 54 AM

naschmitz and others added 30 commits October 10, 2024 13:17
Copy link
Collaborator

@naschmitz naschmitz left a comment

Choose a reason for hiding this comment

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

Looks great! Just a few minor comments.

karma.conf.cjs Outdated Show resolved Hide resolved
karma.conf.cjs Show resolved Hide resolved
tests/layer_manager.spec.ts Outdated Show resolved Hide resolved
tests/layer_manager.spec.ts Show resolved Hide resolved
@sufyanAbbasi sufyanAbbasi merged commit 53078c9 into gee-community:master Nov 14, 2024
15 checks passed
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.

3 participants