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

small fix in uia_defines #1322

Open
wants to merge 2 commits into
base: atspi
Choose a base branch
from

Conversation

junkmd
Copy link
Contributor

@junkmd junkmd commented Aug 14, 2023

delint and rename variables within get_elem_interface

In reality, what is passed to QueryInterface is not the "class name" but the "pattern class", so I changed the variable names to be appropriate.

fix base name typo

Previously, it was 'MulipleView' instead of 'MultipleView', which is why IUIAutomationMultipleViewPattern or UIA_MultipleViewPatternId could not be found in comtypes.gen.UIAutomationClient.

In reality, what is passed to `QueryInterface` is not the "class name"
but the "pattern class", so I changed the variable names to be
appropriate.
Previously, it was 'MulipleView' instead of 'MultipleView', which is why
`IUIAutomationMultipleViewPattern` or `UIA_MultipleViewPatternId`
could not be found in `comtypes.gen.UIAutomationClient`.
@codecov
Copy link

codecov bot commented Aug 14, 2023

Codecov Report

Merging #1322 (b9d6fe3) into atspi (bf7f789) will decrease coverage by 6.31%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##            atspi    #1322      +/-   ##
==========================================
- Coverage   94.15%   87.84%   -6.31%     
==========================================
  Files          60       60              
  Lines       23036    23034       -2     
==========================================
- Hits        21689    20235    -1454     
- Misses       1347     2799    +1452     

@junkmd
Copy link
Contributor Author

junkmd commented Aug 26, 2023

If this PR gets merged, I'm planning to work for refactoring uia_defines.

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.

None yet

2 participants