comtypes 1.4.5__zip → 1.4.7__zip

{comtypes-1.4.5 → comtypes-1.4.7}/comtypes/hints.pyi

@@ -3,27 +3,17 @@
 # - utilities for type hints.
 import ctypes
 import sys
-from typing import (
-    Any,
-    Callable,
-    ClassVar,
-    Generic,
-    Iterator,
-    List,
-    NoReturn,
-    Sequence,
-    Tuple,
-    Type,
-    TypeVar,
-    Optional,
-    overload,
-    Union as _UnionT,
-)
+from typing import Any as Any, ClassVar, Generic, NoReturn, TypeVar, overload
+from typing import Optional, Union as _UnionT
+from typing import List, Tuple as Tuple, Type
+from typing import Callable, Iterator, Sequence
 
 if sys.version_info >= (3, 8):
     from typing import Protocol
+    from typing import Literal as Literal
 else:
     from typing_extensions import Protocol
+    from typing_extensions import Literal as Literal
 if sys.version_info >= (3, 9):
     from typing import Annotated as Annotated
 else:
@@ -35,9 +25,9 @@ else:
     from typing_extensions import Concatenate, ParamSpec, TypeAlias
     from typing_extensions import TypeGuard as TypeGuard
 if sys.version_info >= (3, 11):
-    from typing import Self
+    from typing import Self as Self
 else:
-    from typing_extensions import Self
+    from typing_extensions import Self as Self
 
 import comtypes
 from comtypes import IUnknown as IUnknown, GUID as GUID

{comtypes-1.4.5 → comtypes-1.4.7}/comtypes/persist.py

@@ -9,12 +9,19 @@
 The 'DictPropertyBag' class is a class implementing the IPropertyBag
 interface, useful in client code.
 """
-from ctypes import *
+from ctypes import c_int, c_ulong, c_ushort, c_wchar_p
+from ctypes import POINTER, Structure
 from ctypes.wintypes import WORD, DWORD, BOOL
-from comtypes import GUID, IUnknown, COMMETHOD, HRESULT, dispid
+from typing import TYPE_CHECKING
+
+from comtypes import GUID, IUnknown, COMMETHOD, HRESULT
 from comtypes import IPersist
 from comtypes.automation import VARIANT, tagEXCEPINFO
 
+if TYPE_CHECKING:
+    from comtypes import hints  # type: ignore
+
+
 # XXX Replace by canonical solution!!!
 WSTRING = c_wchar_p
 
@@ -236,9 +243,18 @@ class IPersistFile(IPersist):
         ),
     ]
 
+    if TYPE_CHECKING:
+        # fmt: off
+        def IsDirty(self) -> hints.Hresult: ...  # noqa
+        def Load(self, pszFileName: str, dwMode: int) -> hints.Hresult: ...  # noqa
+        def Save(self, pszFileName: str, fRemember: bool) -> hints.Hresult: ...  # noqa
+        def SaveCompleted(self, pszFileName: str) -> hints.Hresult: ...  # noqa
+        def GetCurFile(self) -> str: ...  # noqa
+        # fmt: on
+
 
 from comtypes import COMObject
-from comtypes.hresult import *
+from comtypes.hresult import E_INVALIDARG, S_OK
 
 
 class DictPropertyBag(COMObject):

{comtypes-1.4.5 → comtypes-1.4.7}/comtypes/shelllink.py

@@ -1,8 +1,17 @@
 from __future__ import print_function
-from ctypes import *
+from ctypes import c_char_p, c_int, c_short, c_wchar_p
+from ctypes import POINTER
+from ctypes import byref, create_string_buffer, create_unicode_buffer
 from ctypes.wintypes import DWORD, WIN32_FIND_DATAA, WIN32_FIND_DATAW, MAX_PATH
+from typing import Tuple, TYPE_CHECKING
+
 from comtypes import IUnknown, GUID, COMMETHOD, HRESULT, CoClass
 
+
+if TYPE_CHECKING:
+    from comtypes import hints  # type: ignore
+
+
 # for GetPath
 SLGP_SHORTPATH = 0x1
 SLGP_UNCPRIORITY = 0x2
@@ -113,31 +122,52 @@ class IShellLinkA(IUnknown):
         COMMETHOD([], HRESULT, "SetPath", (["in"], c_char_p, "pszFile")),
     ]
 
-    def GetPath(self, flags=SLGP_SHORTPATH):
+    if TYPE_CHECKING:
+        # fmt: off
+        def GetIDList(self) -> hints.Incomplete: ...  # noqa
+        def SetIDList(self, pidl: hints.Incomplete) -> hints.Incomplete: ...  # noqa
+        def SetDescription(self, pszName: bytes) -> hints.Incomplete: ...  # noqa
+        def SetWorkingDirectory(self, pszDir: bytes) -> hints.Hresult: ...   # noqa
+        def SetArguments(self, pszArgs: bytes) -> hints.Hresult: ...   # noqa
+        @property
+        def Hotkey(self) -> int: ...   # noqa
+        @Hotkey.setter
+        def Hotkey(self, pwHotkey: int) -> None: ...   # noqa
+        @property
+        def ShowCmd(self) -> int: ...   # noqa
+        @ShowCmd.setter
+        def ShowCmd(self, piShowCmd: int) -> None: ...   # noqa
+        def SetIconLocation(self, pszIconPath: bytes, iIcon: int) -> hints.Hresult: ...   # noqa
+        def SetRelativePath(self, pszPathRel: bytes, dwReserved: hints.Literal[0]) -> hints.Hresult: ...   # noqa
+        def Resolve(self, hwnd: int, fFlags: int) -> hints.Hresult: ...   # noqa
+        def SetPath(self, pszFile: bytes) -> hints.Hresult: ...   # noqa
+        # fmt: on
+
+    def GetPath(self, flags: int = SLGP_SHORTPATH) -> bytes:
         buf = create_string_buffer(MAX_PATH)
         # We're not interested in WIN32_FIND_DATA
-        self.__com_GetPath(buf, MAX_PATH, None, flags)
+        self.__com_GetPath(buf, MAX_PATH, None, flags)  # type: ignore
         return buf.value
 
-    def GetDescription(self):
+    def GetDescription(self) -> bytes:
         buf = create_string_buffer(1024)
-        self.__com_GetDescription(buf, 1024)
+        self.__com_GetDescription(buf, 1024)  # type: ignore
         return buf.value
 
-    def GetWorkingDirectory(self):
+    def GetWorkingDirectory(self) -> bytes:
         buf = create_string_buffer(MAX_PATH)
-        self.__com_GetWorkingDirectory(buf, MAX_PATH)
+        self.__com_GetWorkingDirectory(buf, MAX_PATH)  # type: ignore
         return buf.value
 
-    def GetArguments(self):
+    def GetArguments(self) -> bytes:
         buf = create_string_buffer(1024)
-        self.__com_GetArguments(buf, 1024)
+        self.__com_GetArguments(buf, 1024)  # type: ignore
         return buf.value
 
-    def GetIconLocation(self):
+    def GetIconLocation(self) -> Tuple[bytes, int]:
         iIcon = c_int()
         buf = create_string_buffer(MAX_PATH)
-        self.__com_GetIconLocation(buf, MAX_PATH, byref(iIcon))
+        self.__com_GetIconLocation(buf, MAX_PATH, byref(iIcon))  # type: ignore
         return buf.value, iIcon.value
 
 
@@ -226,31 +256,52 @@ class IShellLinkW(IUnknown):
         COMMETHOD([], HRESULT, "SetPath", (["in"], c_wchar_p, "pszFile")),
     ]
 
-    def GetPath(self, flags=SLGP_SHORTPATH):
+    if TYPE_CHECKING:
+        # fmt: off
+        def GetIDList(self) -> hints.Incomplete: ...  # noqa
+        def SetIDList(self, pidl: hints.Incomplete) -> hints.Incomplete: ...  # noqa
+        def SetDescription(self, pszName: str) -> hints.Incomplete: ...  # noqa
+        def SetWorkingDirectory(self, pszDir: str) -> hints.Hresult: ...   # noqa
+        def SetArguments(self, pszArgs: str) -> hints.Hresult: ...   # noqa
+        @property
+        def Hotkey(self) -> int: ...   # noqa
+        @Hotkey.setter
+        def Hotkey(self, pwHotkey: int) -> None: ...   # noqa
+        @property
+        def ShowCmd(self) -> int: ...   # noqa
+        @ShowCmd.setter
+        def ShowCmd(self, piShowCmd: int) -> None: ...   # noqa
+        def SetIconLocation(self, pszIconPath: str, iIcon: int) -> hints.Hresult: ...   # noqa
+        def SetRelativePath(self, pszPathRel: str, dwReserved: hints.Literal[0]) -> hints.Hresult: ...   # noqa
+        def Resolve(self, hwnd: int, fFlags: int) -> hints.Hresult: ...   # noqa
+        def SetPath(self, pszFile: str) -> hints.Hresult: ...   # noqa
+        # fmt: on
+
+    def GetPath(self, flags: int = SLGP_SHORTPATH) -> str:
         buf = create_unicode_buffer(MAX_PATH)
         # We're not interested in WIN32_FIND_DATA
-        self.__com_GetPath(buf, MAX_PATH, None, flags)
+        self.__com_GetPath(buf, MAX_PATH, None, flags)  # type: ignore
         return buf.value
 
-    def GetDescription(self):
+    def GetDescription(self) -> str:
         buf = create_unicode_buffer(1024)
-        self.__com_GetDescription(buf, 1024)
+        self.__com_GetDescription(buf, 1024)  # type: ignore
         return buf.value
 
-    def GetWorkingDirectory(self):
+    def GetWorkingDirectory(self) -> str:
         buf = create_unicode_buffer(MAX_PATH)
-        self.__com_GetWorkingDirectory(buf, MAX_PATH)
+        self.__com_GetWorkingDirectory(buf, MAX_PATH)  # type: ignore
         return buf.value
 
-    def GetArguments(self):
+    def GetArguments(self) -> str:
         buf = create_unicode_buffer(1024)
-        self.__com_GetArguments(buf, 1024)
+        self.__com_GetArguments(buf, 1024)  # type: ignore
         return buf.value
 
-    def GetIconLocation(self):
+    def GetIconLocation(self) -> Tuple[str, int]:
         iIcon = c_int()
         buf = create_unicode_buffer(MAX_PATH)
-        self.__com_GetIconLocation(buf, MAX_PATH, byref(iIcon))
+        self.__com_GetIconLocation(buf, MAX_PATH, byref(iIcon))  # type: ignore
         return buf.value, iIcon.value
 
 

{comtypes-1.4.5 → comtypes-1.4.7}/comtypes/test/test_client.py

@@ -77,6 +77,15 @@ class Test_GetModule(ut.TestCase):
 
         self.assertTrue(issubclass(mod.IStream, ISequentialStream))
 
+    def test_msvidctl(self):
+        with contextlib.redirect_stdout(None):  # supress warnings
+            mod = comtypes.client.GetModule("msvidctl.dll")
+        from comtypes.persist import IPersist
+        from comtypes.typeinfo import IRecordInfo
+
+        self.assertIs(mod.IPersist, IPersist)
+        self.assertIs(mod.IRecordInfo, IRecordInfo)
+
     def test_no_replacing_Patch_namespace(self):
         # NOTE: An object named `Patch` is defined in some dll.
         # Depending on how the namespace is defined in the static module,

comtypes-1.4.7/comtypes/test/test_monikers.py

@@ -0,0 +1,70 @@
+from _ctypes import COMError
+from ctypes import byref, POINTER
+import contextlib
+import unittest
+
+import comtypes
+from comtypes.client import GetModule, CreateObject
+from comtypes import hresult, GUID
+
+with contextlib.redirect_stdout(None):  # supress warnings
+    GetModule("msvidctl.dll")
+from comtypes.gen import MSVidCtlLib as msvidctl
+from comtypes.gen.MSVidCtlLib import IBindCtx, IMoniker, IRunningObjectTable
+
+
+MKSYS_ITEMMONIKER = 4
+ROTFLAGS_ALLOWANYCLIENT = 1
+
+
+def _create_item_moniker(delim: str, item: str) -> IMoniker:
+    mon = POINTER(IMoniker)()
+    comtypes._ole32.CreateItemMoniker(delim, item, byref(mon))
+    return mon  # type: ignore
+
+
+def _create_bctx() -> IBindCtx:
+    bctx = POINTER(IBindCtx)()
+    # The first parameter is reserved and must be 0.
+    comtypes._ole32.CreateBindCtx(0, byref(bctx))
+    return bctx  # type: ignore
+
+
+def _create_rot() -> IRunningObjectTable:
+    rot = POINTER(IRunningObjectTable)()
+    # The first parameter is reserved and must be 0.
+    comtypes._ole32.GetRunningObjectTable(0, byref(rot))
+    return rot  # type: ignore
+
+
+class Test_IMoniker(unittest.TestCase):
+    def test_IsSystemMoniker(self):
+        item_id = str(GUID.create_new())
+        mon = _create_item_moniker("!", item_id)
+        self.assertEqual(mon.IsSystemMoniker(), MKSYS_ITEMMONIKER)
+
+
+class Test_IBindCtx(unittest.TestCase):
+    def test_EnumObjectParam(self):
+        bctx = _create_bctx()
+        with self.assertRaises(COMError) as err_ctx:
+            # calling `EnumObjectParam` results in a return value of E_NOTIMPL.
+            # https://learn.microsoft.com/en-us/windows/win32/api/objidl/nf-objidl-ibindctx-enumobjectparam#notes-to-callers
+            bctx.EnumObjectParam()
+        self.assertEqual(err_ctx.exception.hresult, hresult.E_NOTIMPL)
+
+
+class Test_IRunningObjectTable(unittest.TestCase):
+    def test_register_and_revoke_item_moniker(self):
+        vidctl = CreateObject(msvidctl.MSVidCtl, interface=msvidctl.IMSVidCtl)
+        item_id = str(GUID.create_new())
+        mon = _create_item_moniker("!", item_id)
+        rot = _create_rot()
+        bctx = _create_bctx()
+        self.assertEqual(mon.IsRunning(bctx, None, None), hresult.S_FALSE)
+        dw_reg = rot.Register(ROTFLAGS_ALLOWANYCLIENT, vidctl, mon)
+        self.assertEqual(mon.IsRunning(bctx, None, None), hresult.S_OK)
+        self.assertEqual(f"!{item_id}", mon.GetDisplayName(bctx, None))
+        self.assertEqual(rot.GetObject(mon).QueryInterface(msvidctl.IMSVidCtl), vidctl)
+        rot.Revoke(dw_reg)
+        self.assertEqual(mon.IsRunning(bctx, None, None), hresult.S_FALSE)

comtypes-1.4.7/comtypes/test/test_persist.py

@@ -0,0 +1,40 @@
+from pathlib import Path
+import tempfile
+import unittest as ut
+
+from comtypes import CoCreateInstance, GUID, IPersist
+from comtypes import persist
+
+
+CLSID_ShellLink = GUID("{00021401-0000-0000-C000-000000000046}")
+
+
+class Test_IPersist(ut.TestCase):
+    def test_GetClassID(self):
+        p = CoCreateInstance(CLSID_ShellLink).QueryInterface(IPersist)
+        self.assertEqual(p.GetClassID(), CLSID_ShellLink)
+
+
+class Test_IPersistFile(ut.TestCase):
+    def setUp(self):
+        td = tempfile.TemporaryDirectory()
+        self.addCleanup(td.cleanup)
+        self.tmp_dir = Path(td.name)
+
+    def _create_pf(self) -> persist.IPersistFile:
+        return CoCreateInstance(CLSID_ShellLink).QueryInterface(persist.IPersistFile)
+
+    def test_load(self):
+        pf = self._create_pf()
+        tgt_file = (self.tmp_dir / "tgt.txt").resolve()
+        tgt_file.touch()
+        pf.Load(str(tgt_file), persist.STGM_DIRECT)
+        self.assertEqual(pf.GetCurFile(), str(tgt_file))
+
+    def test_save(self):
+        pf = self._create_pf()
+        tgt_file = self.tmp_dir / "tgt.txt"
+        self.assertFalse(tgt_file.exists())
+        pf.Save(str(tgt_file), True)
+        self.assertEqual(pf.GetCurFile(), str(tgt_file))
+        self.assertTrue(tgt_file.exists())

comtypes-1.4.7/comtypes/test/test_recordinfo.py

@@ -0,0 +1,79 @@
+# coding: utf-8
+
+import unittest
+
+from comtypes import typeinfo
+from comtypes.client import GetModule
+from ctypes import byref, pointer, sizeof
+
+ComtypesCppTestSrvLib_GUID = "{07D2AEE5-1DF8-4D2C-953A-554ADFD25F99}"
+
+try:
+    GetModule((ComtypesCppTestSrvLib_GUID, 1, 0, 0))
+    from comtypes.gen.ComtypesCppTestSrvLib import StructRecordParamTest
+
+    IMPORT_FAILED = False
+except (ImportError, OSError):
+    IMPORT_FAILED = True
+
+
+def _create_recordinfo() -> typeinfo.IRecordInfo:
+    return typeinfo.GetRecordInfoFromGuids(*StructRecordParamTest._recordinfo_)
+
+
+def _create_record(
+    question: str, answer: int, needs_clarification: bool
+) -> "StructRecordParamTest":
+    record = StructRecordParamTest()
+    record.question = question
+    record.answer = answer
+    record.needs_clarification = needs_clarification
+    return record
+
+
+@unittest.skipIf(IMPORT_FAILED, "This depends on the out of process COM-server.")
+class Test_IRecordInfo(unittest.TestCase):
+    def test_RecordCopy(self):
+        dst_rec = StructRecordParamTest()
+        ri = _create_recordinfo()
+        ri.RecordCopy(pointer(_create_record("foo", 3, True)), byref(dst_rec))
+        self.assertEqual(dst_rec.question, "foo")
+        self.assertEqual(dst_rec.answer, 3)
+        self.assertEqual(dst_rec.needs_clarification, True)
+
+    def test_GetGuid(self):
+        *_, expected_guid = StructRecordParamTest._recordinfo_
+        self.assertEqual(str(_create_recordinfo().GetGuid()), expected_guid)
+
+    def test_GetName(self):
+        self.assertEqual(
+            _create_recordinfo().GetName(),
+            StructRecordParamTest.__qualname__,
+        )
+
+    def test_GetSize(self):
+        self.assertEqual(
+            _create_recordinfo().GetSize(),
+            sizeof(StructRecordParamTest),
+        )
+
+    def test_GetTypeInfo(self):
+        ta = _create_recordinfo().GetTypeInfo().GetTypeAttr()
+        *_, expected_guid = StructRecordParamTest._recordinfo_
+        self.assertEqual(str(ta.guid), expected_guid)
+
+    def test_IsMatchingType(self):
+        ri = _create_recordinfo()
+        ti = ri.GetTypeInfo()
+        self.assertTrue(typeinfo.GetRecordInfoFromTypeInfo(ti).IsMatchingType(ri))
+
+    def test_RecordCreateCopy(self):
+        ri = _create_recordinfo()
+        actual = ri.RecordCreateCopy(byref(_create_record("foo", 3, True)))
+        self.assertIsInstance(actual, int)
+        dst_rec = StructRecordParamTest()
+        ri.RecordCopy(actual, byref(dst_rec))
+        ri.RecordDestroy(actual)
+        self.assertEqual(dst_rec.question, "foo")
+        self.assertEqual(dst_rec.answer, 3)
+        self.assertEqual(dst_rec.needs_clarification, True)

comtypes-1.4.7/comtypes/test/test_shelllink.py

@@ -0,0 +1,133 @@
+from pathlib import Path
+import tempfile
+import unittest as ut
+
+from comtypes import CoCreateInstance, GUID, shelllink
+import comtypes.hresult
+from comtypes.persist import IPersistFile
+
+
+CLSID_ShellLink = GUID("{00021401-0000-0000-C000-000000000046}")
+
+
+class Test_IShellLinkA(ut.TestCase):
+    def setUp(self):
+        td = tempfile.TemporaryDirectory()
+        self.addCleanup(td.cleanup)
+        self.tmp_dir = Path(td.name)
+        self.src_file = (self.tmp_dir / "src.txt").resolve()
+        self.src_file.touch()
+
+    def _create_shortcut(self) -> shelllink.IShellLinkA:
+        return CoCreateInstance(CLSID_ShellLink, interface=shelllink.IShellLinkA)
+
+    def test_set_and_get_path(self):
+        shortcut = self._create_shortcut()
+        pf = shortcut.QueryInterface(IPersistFile)
+        self.assertEqual(pf.IsDirty(), comtypes.hresult.S_FALSE)
+        shortcut.SetPath(str(self.src_file).encode("utf-8"))
+        self.assertEqual(pf.IsDirty(), comtypes.hresult.S_OK)
+        self.assertEqual(
+            shortcut.GetPath(shelllink.SLGP_UNCPRIORITY),
+            str(self.src_file).encode("utf-8"),
+        )
+        lnk_file = (self.tmp_dir / "new.lnk").resolve()
+        self.assertFalse(lnk_file.exists())
+        pf.Save(str(lnk_file), True)
+        self.assertTrue(lnk_file.exists())
+        self.assertEqual(pf.GetCurFile(), str(lnk_file))
+
+    def test_set_and_get_working_directory(self):
+        shortcut = self._create_shortcut()
+        shortcut.SetWorkingDirectory(str(self.tmp_dir).encode("utf-8"))
+        self.assertEqual(
+            shortcut.GetWorkingDirectory(), str(self.tmp_dir).encode("utf-8")
+        )
+
+    def test_set_and_get_arguments(self):
+        shortcut = self._create_shortcut()
+        shortcut.SetArguments(b"-f")
+        self.assertEqual(shortcut.GetArguments(), b"-f")
+
+    def test_set_and_get_hotkey(self):
+        HOTKEYF_ALT = 0x04
+        HOTKEYF_CONTROL = 0x02
+        shortcut = self._create_shortcut()
+        shortcut.Hotkey = HOTKEYF_ALT | HOTKEYF_CONTROL
+        self.assertEqual(shortcut.Hotkey, HOTKEYF_ALT | HOTKEYF_CONTROL)
+
+    def test_set_and_get_showcmd(self):
+        SW_SHOWMAXIMIZED = 0x03
+        shortcut = self._create_shortcut()
+        shortcut.ShowCmd = SW_SHOWMAXIMIZED
+        self.assertEqual(shortcut.ShowCmd, SW_SHOWMAXIMIZED)
+
+    def test_set_and_get_icon_location(self):
+        shortcut = self._create_shortcut()
+        shortcut.SetIconLocation(str(self.src_file).encode("utf-8"), 1)
+        icon_path, index = shortcut.GetIconLocation()
+        self.assertEqual(icon_path, str(self.src_file).encode("utf-8"))
+        self.assertEqual(index, 1)
+
+
+class Test_IShellLinkW(ut.TestCase):
+    def setUp(self):
+        td = tempfile.TemporaryDirectory()
+        self.addCleanup(td.cleanup)
+        self.tmp_dir = Path(td.name)
+        self.src_file = (self.tmp_dir / "src.txt").resolve()
+        self.src_file.touch()
+
+    def _create_shortcut(self) -> shelllink.IShellLinkW:
+        return CoCreateInstance(CLSID_ShellLink, interface=shelllink.IShellLinkW)
+
+    def test_set_and_get_path(self):
+        shortcut = self._create_shortcut()
+        pf = shortcut.QueryInterface(IPersistFile)
+        self.assertEqual(pf.IsDirty(), comtypes.hresult.S_FALSE)
+        shortcut.SetPath(str(self.src_file))
+        self.assertEqual(pf.IsDirty(), comtypes.hresult.S_OK)
+        self.assertEqual(
+            shortcut.GetPath(shelllink.SLGP_UNCPRIORITY),
+            str(self.src_file),
+        )
+        lnk_file = (self.tmp_dir / "new.lnk").resolve()
+        self.assertFalse(lnk_file.exists())
+        pf.Save(str(lnk_file), True)
+        self.assertTrue(lnk_file.exists())
+        self.assertEqual(pf.GetCurFile(), str(lnk_file))
+
+    def test_set_and_get_description(self):
+        shortcut = self._create_shortcut()
+        shortcut.SetDescription("sample")
+        self.assertEqual(shortcut.GetDescription(), "sample")
+
+    def test_set_and_get_working_directory(self):
+        shortcut = self._create_shortcut()
+        shortcut.SetWorkingDirectory(str(self.tmp_dir))
+        self.assertEqual(shortcut.GetWorkingDirectory(), str(self.tmp_dir))
+
+    def test_set_and_get_arguments(self):
+        shortcut = self._create_shortcut()
+        shortcut.SetArguments("-f")
+        self.assertEqual(shortcut.GetArguments(), "-f")
+
+    def test_set_and_get_hotkey(self):
+        HOTKEYF_ALT = 0x04
+        HOTKEYF_CONTROL = 0x02
+        shortcut = self._create_shortcut()
+        shortcut.Hotkey = HOTKEYF_ALT | HOTKEYF_CONTROL
+        self.assertEqual(shortcut.Hotkey, HOTKEYF_ALT | HOTKEYF_CONTROL)
+
+    def test_set_and_get_showcmd(self):
+        SW_SHOWMAXIMIZED = 0x03
+        shortcut = self._create_shortcut()
+        shortcut.ShowCmd = SW_SHOWMAXIMIZED
+        self.assertEqual(shortcut.ShowCmd, SW_SHOWMAXIMIZED)
+
+    def test_set_and_get_icon_location(self):
+        shortcut = self._create_shortcut()
+        shortcut.SetIconLocation(str(self.src_file), 1)
+        icon_path, index = shortcut.GetIconLocation()
+        self.assertEqual(icon_path, str(self.src_file))
+        self.assertEqual(index, 1)