Conversation
…ain the prototype behavior of building the puppet and toolkit anew when the tests are run
|
Building this branch is not 100% necessary if anyone wants to drop by my office instead for a demonstration. I mostly want confirmation that I'm on the right track before I go much further (though WinUI is already done to mostly the same state as WPF here) |
dotMorten
left a comment
There was a problem hiding this comment.
Great start! Let's change the Puppet name though :)
| var dpi = VisualTreeHelper.GetDpi(this); | ||
| ScreenDensityTextBlock.Text = dpi.DpiScaleX.ToString(); | ||
|
|
||
| NetVersionTextBlock.Text = Environment.Version.ToString(); | ||
|
|
||
| var runtimeTypeInfo = typeof(ArcGISRuntimeEnvironment).GetTypeInfo(); | ||
| var runtimeVersion = FileVersionInfo.GetVersionInfo(runtimeTypeInfo.Assembly.Location); | ||
| RuntimeVersionTextBlock.Text = runtimeVersion.FileVersion; | ||
|
|
||
| var toolkitTypeInfo = typeof(Esri.ArcGISRuntime.Toolkit.UI.Controls.ScaleLine).GetTypeInfo(); | ||
| var toolkitVersion = FileVersionInfo.GetVersionInfo(runtimeTypeInfo.Assembly.Location); | ||
| ToolkitVersionTextBlock.Text = toolkitVersion.FileVersion; |
There was a problem hiding this comment.
Screen density is used when making sure widths are correct even if screen dpis change. The others are just a means for users to visual confirm that the correct versions of .NET, Runtime, and the Toolkit are being used, like we do in the Maui runtime UI tests
|
|
||
| driver = MakeWindowsDriver(wpfSamplesApp); | ||
|
|
||
| driver.Manage().Window.Maximize(); |
There was a problem hiding this comment.
Do you also need to ensure it's on the primary display to get consistent screen coordinates?
There was a problem hiding this comment.
Maybe. Using find element means we shouldn't need to use manual coordinates, and the app displays the screen dpi so that we can adjust expected ui element dimensions
This pull request only includes WPF Scale Line tests to make sure I'm on the right track. Converted old tests to MSTest and created a new puppet app separate from the Samples.
Building requires installing Appium: https://appium.io/docs/en/latest/quickstart/install/
And the windows driver here: https://appium.io/docs/en/latest/ecosystem/drivers/#windows
Make sure appium is running before starting the tests.