comtypes 1.4.6__zip → 1.4.8__zip

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/shelllink.py

@@ -1,27 +1,40 @@
-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
 SLGP_RAWPATH = 0x4
 
 # for SetShowCmd, GetShowCmd
-##SW_SHOWNORMAL
-##SW_SHOWMAXIMIZED
-##SW_SHOWMINNOACTIVE
-
+SW_SHOWNORMAL = 0x01
+SW_SHOWMAXIMIZED = 0x03
+SW_SHOWMINNOACTIVE = 0x07
 
 # for Resolve
-##SLR_INVOKE_MSI
-##SLR_NOLINKINFO
-##SLR_NO_UI
-##SLR_NOUPDATE
-##SLR_NOSEARCH
-##SLR_NOTRACK
-##SLR_UPDATE
+SLR_INVOKE_MSI = 0x0080
+SLR_NOLINKINFO = 0x0040
+SLR_NO_UI = 0x0001
+SLR_NOUPDATE = 0x0008
+SLR_NOSEARCH = 0x0010
+SLR_NOTRACK = 0x0020
+SLR_UPDATE = 0x0004
+
+# for Hotkey
+HOTKEYF_ALT = 0x04
+HOTKEYF_CONTROL = 0x02
+HOTKEYF_EXT = 0x08
+HOTKEYF_SHIFT = 0x01
 
 # fake these...
 ITEMIDLIST = c_int
@@ -113,31 +126,53 @@ class IShellLinkA(IUnknown):
         COMMETHOD([], HRESULT, "SetPath", (["in"], c_char_p, "pszFile")),
     ]
 
-    def GetPath(self, flags=SLGP_SHORTPATH):
+    if TYPE_CHECKING:
+
+        def GetIDList(self) -> hints.Incomplete: ...
+        def SetIDList(self, pidl: hints.Incomplete) -> hints.Incomplete: ...
+        def SetDescription(self, pszName: bytes) -> hints.Incomplete: ...
+        def SetWorkingDirectory(self, pszDir: bytes) -> hints.Hresult: ...
+        def SetArguments(self, pszArgs: bytes) -> hints.Hresult: ...
+        @property
+        def Hotkey(self) -> int: ...
+        @Hotkey.setter
+        def Hotkey(self, pwHotkey: int) -> None: ...
+        @property
+        def ShowCmd(self) -> int: ...
+        @ShowCmd.setter
+        def ShowCmd(self, piShowCmd: int) -> None: ...
+        def SetIconLocation(self, pszIconPath: bytes, iIcon: int) -> hints.Hresult: ...
+        def SetRelativePath(
+            self, pszPathRel: bytes, dwReserved: hints.Literal[0]
+        ) -> hints.Hresult: ...
+        def Resolve(self, hwnd: int, fFlags: int) -> hints.Hresult: ...
+        def SetPath(self, pszFile: bytes) -> hints.Hresult: ...
+
+    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 +261,53 @@ class IShellLinkW(IUnknown):
         COMMETHOD([], HRESULT, "SetPath", (["in"], c_wchar_p, "pszFile")),
     ]
 
-    def GetPath(self, flags=SLGP_SHORTPATH):
+    if TYPE_CHECKING:
+
+        def GetIDList(self) -> hints.Incomplete: ...
+        def SetIDList(self, pidl: hints.Incomplete) -> hints.Incomplete: ...
+        def SetDescription(self, pszName: str) -> hints.Incomplete: ...
+        def SetWorkingDirectory(self, pszDir: str) -> hints.Hresult: ...
+        def SetArguments(self, pszArgs: str) -> hints.Hresult: ...
+        @property
+        def Hotkey(self) -> int: ...
+        @Hotkey.setter
+        def Hotkey(self, pwHotkey: int) -> None: ...
+        @property
+        def ShowCmd(self) -> int: ...
+        @ShowCmd.setter
+        def ShowCmd(self, piShowCmd: int) -> None: ...
+        def SetIconLocation(self, pszIconPath: str, iIcon: int) -> hints.Hresult: ...
+        def SetRelativePath(
+            self, pszPathRel: str, dwReserved: hints.Literal[0]
+        ) -> hints.Hresult: ...
+        def Resolve(self, hwnd: int, fFlags: int) -> hints.Hresult: ...
+        def SetPath(self, pszFile: str) -> hints.Hresult: ...
+
+    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
 
 
@@ -263,7 +320,6 @@ class ShellLink(CoClass):
 
 
 if __name__ == "__main__":
-
     import sys
     import comtypes
     from comtypes.client import CreateObject

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/test/TestComServer.py

@@ -33,6 +33,7 @@ from comtypes.gen import TestComServerLib
 
 ################################################################
 
+
 # Implement the CoClass.  Use the coclass from the wrapper as base
 # class, and use DualDispMixin as base class which provides default
 # implementations of IDispatch, IProvideClassInfo, IProvideClassInfo2
@@ -42,7 +43,6 @@ class TestComServer(
     TestComServerLib.TestComServer,  # the coclass from the typelib wrapper
     comtypes.server.connectionpoints.ConnectableObjectMixin,
 ):
-
     # The default interface from the typelib MUST be the first
     # interface, other interfaces can follow
 

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/test/TestDispServer.py

@@ -31,6 +31,7 @@ from comtypes.gen import TestDispServerLib
 
 ################################################################
 
+
 # Implement the CoClass by defining a subclass of the
 # TestDispServerLib.TestDispServer class in the wrapper file.  The
 # COMObject base class provides default implementations of the
@@ -42,7 +43,6 @@ class TestDispServer(
     TestDispServerLib.TestDispServer,  # the coclass from the typelib wrapper
     comtypes.server.connectionpoints.ConnectableObjectMixin,
 ):
-
     # The default interface from the typelib MUST be the first
     # interface, other interfaces can follow
 

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/test/__init__.py

@@ -1,6 +1,5 @@
 # comtypes.test package.
 
-from __future__ import print_function
 import ctypes
 import getopt
 import os

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/test/find_memleak.py

@@ -1,4 +1,3 @@
-from __future__ import print_function
 import unittest, gc
 from ctypes import *
 from ctypes.wintypes import *

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/test/test_agilent.py

@@ -2,7 +2,6 @@
 # is installed.  It is not requires to have a physical instrument
 # connected, the driver is used in simulation mode.
 
-from __future__ import print_function
 import unittest
 from comtypes.test import ResourceDenied
 from comtypes.client import CreateObject

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/test/test_clear_cache.py

@@ -1,6 +1,7 @@
 """
 Test for the ``comtypes.clear_cache`` module.
 """
+
 import contextlib
 import runpy
 from unittest.mock import patch, call

{comtypes-1.4.6 → comtypes-1.4.8}/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.6 → comtypes-1.4.8}/comtypes/test/test_createwrappers.py

@@ -1,5 +1,3 @@
-from __future__ import print_function
-
 import glob
 import os
 import unittest

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/test/test_dispinterface.py

@@ -16,7 +16,6 @@ def setUpModule():
 
 
 class Test(unittest.TestCase):
-
     if is_resource_enabled("pythoncom"):
 
         def test_win32com(self):

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/test/test_excel.py

@@ -1,5 +1,4 @@
 # -*- coding: latin-1 -*-
-from __future__ import print_function
 
 import datetime
 import sys

{comtypes-1.4.6 → comtypes-1.4.8}/comtypes/test/test_inout_args.py

@@ -278,7 +278,7 @@ class Test_ArgspecPermutations(ut.TestCase):
                 (orig_0th, *orig_call_args), orig_kw = orig.call_args
                 self.assertEqual(orig_kw, {})
                 self.assertIs(orig_0th, self_)
-                for ((typ, f, val), orig) in zip(
+                for (typ, f, val), orig in zip(
                     testing_params.call_args_validators, orig_call_args
                 ):
                     self.assertIsInstance(orig, typ)

comtypes-1.4.8/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.8/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.6 → comtypes-1.4.8}/comtypes/test/test_server.py

@@ -125,6 +125,7 @@ try:
         )
         # implementation
         Name = "foo"
+
         # test
         def test_Name(self):
             p = wrap(self.create())
@@ -137,6 +138,7 @@ try:
         itf.add(
             "[id(101)] HRESULT MixedInOut([in] int a, [out] int *b, [in] int c, [out] int *d);"
         )
+
         # implementation
         def MixedInOut(self, a, c):
             return a + 1, c + 1
@@ -151,6 +153,7 @@ try:
         itf.add(
             "[id(102)] HRESULT MultiInOutArgs([in, out] int *pa, [in, out] int *pb);"
         )
+
         # implementation
         def MultiInOutArgs(self, pa, pb):
             return pa[0] * 3, pb[0] * 4
@@ -174,6 +177,7 @@ try:
         ################
         # definition
         itf.add("HRESULT MultiInOutArgs3([out] int *pa, [out] int *pb);")
+
         # implementation
         def MultiInOutArgs3(self):
             return 42, 43
@@ -186,6 +190,7 @@ try:
         ################
         # definition
         itf.add("HRESULT MultiInOutArgs4([out] int *pa, [in, out] int *pb);")
+
         # implementation
         def MultiInOutArgs4(self, pb):
             return pb[0] + 3, pb[0] + 4
@@ -238,6 +243,7 @@ try:
         # Test events.
         itf.add("""HRESULT DoSomething();""")
         outgoing.add("""[id(103)] HRESULT OnSomething();""")
+
         # implementation
         def DoSomething(self):
             "Implement the DoSomething method"

comtypes-1.4.8/comtypes/test/test_shelllink.py

@@ -0,0 +1,129 @@
+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):
+        hotkey = shelllink.HOTKEYF_ALT | shelllink.HOTKEYF_CONTROL
+        shortcut = self._create_shortcut()
+        shortcut.Hotkey = hotkey
+        self.assertEqual(shortcut.Hotkey, hotkey)
+
+    def test_set_and_get_showcmd(self):
+        shortcut = self._create_shortcut()
+        shortcut.ShowCmd = shelllink.SW_SHOWMAXIMIZED
+        self.assertEqual(shortcut.ShowCmd, shelllink.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):
+        hotkey = shelllink.HOTKEYF_ALT | shelllink.HOTKEYF_CONTROL
+        shortcut = self._create_shortcut()
+        shortcut.Hotkey = hotkey
+        self.assertEqual(shortcut.Hotkey, hotkey)
+
+    def test_set_and_get_showcmd(self):
+        shortcut = self._create_shortcut()
+        shortcut.ShowCmd = shelllink.SW_SHOWMAXIMIZED
+        self.assertEqual(shortcut.ShowCmd, shelllink.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)