-
Notifications
You must be signed in to change notification settings - Fork 1
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
Merge "End to end tests" branch including react test harness and end-to-end playwright harness #50
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some import fixes in module.ts, withThemeWrapper, and MapPanel are needed. Most other fixes that could use attention are inline documentation/comments.
test/module.spec.js
Outdated
@@ -1454,7 +1454,7 @@ describe( "Class MapCanvas", () => { | |||
] | |||
// this appears to be a "load-bearing" reassignment... removing it causes tests to fail? | |||
var topo = newTopology; | |||
/*expect(function(){ | |||
/*expect(function(){ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this commented code be removed?
- added compileOnSave flag to tsconfig - corrected imports for GrafanaTheme2 and locationService from @grafana packages @grafana/data and @grafana/runtime/services/LocationService - exported MapPanelProps from MapPanel - exported MapPanel - removed wrapping calls of withTheme HOC out of MapPanel
- added testables code into dist
…workmap-panel into end-to-end-tests
- restored import of locationServices from @grafana/runtme
- restored import of StringFieldConfigSettings out of @grafana/data instead of deeper introspective import source
No description provided.