Merge drop_py2
#509
Merged
Merge drop_py2
#509
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
(cherry picked from commit 30f497a)
(cherry picked from commit 7a14804)
(cherry picked from commit 80ec49b)
(cherry picked from commit 817b6b7)
(cherry picked from commit 5b0dc8d)
(cherry picked from commit a6fc08b)
(cherry picked from commit 21b6ab6)
(cherry picked from commit 086ec13)
(cherry picked from commit 4e53792)
(cherry picked from commit 14ae44d)
(cherry picked from commit 1398608)
* Improve COM error message * Drop Py2 stuff * Remove dot * Remove empty line (cherry picked from commit 18cafb0)
(cherry picked from commit 12d22a2)
* add `# fmt: ...` for avoiding redundant format * add GHA setting * apply automatic formatter Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> (cherry picked from commit 22bfc50)
(cherry picked from commit 0664400)
(cherry picked from commit 32d6621)
(cherry picked from commit b6a4181)
(cherry picked from commit 0cadf77)
(cherry picked from commit 39d9e00)
(cherry picked from commit 0fecfea)
(cherry picked from commit 7d69dce)
(cherry picked from commit 6b551a4)
(cherry picked from commit 11410ce)
(cherry picked from commit ba1b18f)
(cherry picked from commit 9238cc7)
…le` annotation (enthought#451) (cherry picked from commit 446f52e)
(cherry picked from commit 6732c26)
(cherry picked from commit 7321eaf)
…comtypes/__init__.py`, `comtypes/automation.py` and `comtypes/typeinfo.py` (enthought#454) * update `comtypes/__init__.py` * update `comtypes/automation.py` * update `comtypes/typeinfo.py` * fix `Array[T]` to quoted literal strings (cherry picked from commit 7b38a7a)
…_memberspec` (enthought#455) (cherry picked from commit 5d230aa)
…comtypes/client/...` (enthought#456) * change type hints from comment annotations to inline annotations in `comtypes/client/...` * fix type annotations (cherry picked from commit 18dc2cc)
* update `codegenerator` type annotations * lcid (cherry picked from commit 9e7b900)
(cherry picked from commit 0b4dbca)
) * update `typedesc` type annotations * update `typedesc_base` type annotations (cherry picked from commit 95962e9)
(cherry picked from commit 04037ae)
* change from udf `silence_stdout` to stdlib `contextlib.contextmanager` * remove `sys.version_info` bridges * remove excessive blank lines (cherry picked from commit d470b62)
(cherry picked from commit a76f0b1)
Co-authored-by: jonschz <[email protected]> (cherry picked from commit 1615af5)
* unit test for generated IEnum (enthought#468) * change enum_test to a non-empty IEnum * improved checks and clarity * apply requested changes Co-authored-by: jonschz <[email protected]> (cherry picked from commit e1ee6f0)
…he friendly module (enthought#469) * add `Library` to generated modules' `__all__`. because that symbol is public but not included. * add `typelib_path` to generated modules' `__all__`. because that symbol is public but not included. * make `ModuleGenerator` class that encapsulates `CodeGenerator` instance. * rename to `generate_wrapper_code` from `generate_code` * add `generate_friendly_code` * add type annotations to `generate_wrapper_code` * add docstring * add `get_symbols` methods to `DeclaredNamespaces` and `ImportedNamespaces` * update imporing symbols * add type annotation to return value for `__init__` * change to using f-string in `generate_friendly_code` (cherry picked from commit 532c399)
from `setup.py` (cherry picked from commit 9724dc0)
* change `_MemberSpec` to `NamedTuple` * `_ComMemberSpec` is no more `Generic` for Py<3.11 backward compatibility. * apply `black` style (cherry picked from commit 94b81af)
…_make_dp` method (enthought#485) (cherry picked from commit 62ce303)
* move clear_comtypes_cache to be a callable module This commit modifies the clear_comtypes_cache.py script so that it is inside the main comtypes module (renamed as just clear_cache) so that is can be called more easily as "py -m comtypes.clear_cache". The main function of the script is also exported using the "console_scripts" entry point so that the script also goes into the standard Python "Scripts" folder as before, but now as a .exe instead of a .py script, which makes it easier to run if systems are set to open .py files instead of running them. This version also includes a test case using the 3rd party package pyfakefs. Currently, this is not desired to avoid the requirement of 3rd party packages in comtypes, but is included here for potential use if the position changes. A subsequent commit will modify the tests to use unittest.patch instead, which is an inferior technical solution but avoids a 3rd party package. * modify clear_cache tests to not use pyfakefs This commit updates the test for comtypes.clear_cache to not use any 3rd party packages, instead relying on mocking the shutil.rmtree function which is used to do the actual cache deletion. * change quotes in print string * style changes based on review by @junkmd * Apply suggestions from code review Co-authored-by: Jun Komoda <[email protected]> --------- Co-authored-by: Ben Rowland <[email protected]> Co-authored-by: Jun Komoda <[email protected]> (cherry picked from commit 8954f61)
(cherry picked from commit 6532d32)
(cherry picked from commit 65e3c10)
from `ImportedNamespaces.getvalue` (cherry picked from commit da27aaf)
…cted name, `__wrapper_module__` (enthought#493) * add test * add the `__wrapper_module__` to friendly modules By doing so, it is possible to achieve a combination of friendly-modules functionalities and improved access to the wrapper-modules. (cherry picked from commit b7d5f1f)
* bugfix in `call_with_inout` * minor cleanup * handling the case of no in and no out * Test case for _fix_inout_args * additional cleanup and error handling * code formatting fixed * fix python 3.7 and 3.8 compatibility * Temporary addition of real-world test * code cleanup * intermediate commit, do not review * Refactor of unit test, removing portdevice test * fix global side-effect of other skipped test * Update comtypes/test/test_outparam.py Co-authored-by: Jun Komoda <[email protected]> * work on tests for inout_args and outparam - cleanup for test_outparam.py - improvements to test_inout_args.py - comments on a possible error in _memberspec.py * removing dead code * rename variables and add assertions * pass `MagicMock` instead of `ut.TestCase` * make tests for each argument passing patterns * remove duplicated comments * update test code for readability - remove name from mock - move line breaks to between mock preparations and assertions * split the testcases * add `Test_Error` * minor corrections, remove redundancy, migration - rewrite the permutations test - missing direction and omitted name redundant - migrate autogenerated keywords - TBD: more real life tests * Add tests covering 24 patterns - instead of using `if` statements and `permutations` * update test name * add real world tests, remove old code * formatting issue * Update comtypes/_memberspec.py dict type annotation Co-authored-by: Jun Komoda <[email protected]> * Change missing 'in' or 'out' to be treated as 'in' * Add real-world test: param without 'in' or 'out' * add `contextlib.redirect_stdout` to supress warnings * apply review feedback * update comments * add line breaks to lines longer than 88 characters * Update comtypes/test/test_inout_args.py --------- Co-authored-by: jonschz <[email protected]> Co-authored-by: Jonathan <[email protected]> Co-authored-by: Jun Komoda <[email protected]> (cherry picked from commit 876801f)
…es down" from "Table of contents" (enthought#505) (cherry picked from commit af78614)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For
1.3.0release