Skip to content

Conversation

@ritro
Copy link
Contributor

@ritro ritro commented Mar 11, 2013

Fix: searching Rows and Cells in Table (Winforms.DataGridView) for localized versions of Windows and VS (the Name property can be in different language rather than English).
Add: controls added to corresponding versions of TestApplications. Tests for these controls added to TestStack.White.UITests project.
Fix: Old test for message box fixed for running in localized versions of Windows and VS (the button's Name property is in current locale language).
Add: menu double click tests (to Issue #49)

UI and Unit tests pass (except for the one that failed before any changes). Can't fully run UITests.Old because on my home computer they partly pass because of localization issues described above - all Table retrieving needed to be localy unrelated and also many tests fail because in Assertions english type names used while i get localized (this however doesn't affect on elements retrieving except for some parts of the WinForms.Tables). Previously I ran tests at work where I don't have these issues with localizations but now I can't do that.

ritro added 3 commits March 11, 2013 22:31
…calized versions of Windows and VS (the Name property can be in different language rather than English).

Add: controls added to corresponding versions of TestApplications. Tests for these controls added to TestStack.White.UITests project.
Fix: Old test for message box fixed for running in localized versions of Windows and VS (the button's Name property is in current locale language).
…ct and in WhiteCore project FIX: DataGrid test divided into two separate files (for wpf and winforms)
Copy link
Member

Choose a reason for hiding this comment

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

I don't think this comment is required. Change is fair enough by itemself

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As you say, should I remove it?

@JakeGinnivan
Copy link
Member

Looks good, if you could just respond to my questions im happy to pull this in

@JakeGinnivan
Copy link
Member

Superseded by #129

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.

2 participants