mirror of
https://github.com/microsoft/PowerToys.git
synced 2026-02-24 04:00:02 +01:00
<!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? --> ## Summary of the Pull Request As title <!-- Please review the items on the PR checklist before submitting--> ## PR Checklist - [ ] Closes: #xxx <!-- - [ ] Closes: #yyy (add separate lines for additional resolved issues) --> - [ ] **Communication:** I've discussed this with core contributors already. If the work hasn't been agreed, this work might be rejected - [ ] **Tests:** Added/updated and all pass - [ ] **Localization:** All end-user-facing strings can be localized - [ ] **Dev docs:** Added/updated - [ ] **New binaries:** Added on the required places - [ ] [JSON for signing](https://github.com/microsoft/PowerToys/blob/main/.pipelines/ESRPSigning_core.json) for new binaries - [ ] [WXS for installer](https://github.com/microsoft/PowerToys/blob/main/installer/PowerToysSetup/Product.wxs) for new binaries and localization folder - [ ] [YML for CI pipeline](https://github.com/microsoft/PowerToys/blob/main/.pipelines/ci/templates/build-powertoys-steps.yml) for new test projects - [ ] [YML for signed pipeline](https://github.com/microsoft/PowerToys/blob/main/.pipelines/release.yml) - [ ] **Documentation updated:** If checked, please file a pull request on [our docs repo](https://github.com/MicrosoftDocs/windows-uwp/tree/docs/hub/powertoys) and link it here: #xxx <!-- Provide a more detailed description of the PR, other things fixed, or any additional comments/features here --> ## Detailed Description of the Pull Request / Additional comments <!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well --> ## Validation Steps Performed Tests should be picked up and run and pass
121 lines
4.3 KiB
C++
121 lines
4.3 KiB
C++
#include "pch.h"
|
|
#include "TestHelpers.h"
|
|
#include <appMutex.h>
|
|
|
|
using namespace Microsoft::VisualStudio::CppUnitTestFramework;
|
|
|
|
namespace UnitTestsCommonUtils
|
|
{
|
|
TEST_CLASS(AppMutexTests)
|
|
{
|
|
public:
|
|
TEST_METHOD(CreateAppMutex_ValidName_ReturnsHandle)
|
|
{
|
|
std::wstring mutexName = L"TestMutex_" + std::to_wstring(GetCurrentProcessId()) + L"_1";
|
|
auto handle = createAppMutex(mutexName);
|
|
Assert::IsNotNull(handle.get());
|
|
}
|
|
|
|
TEST_METHOD(CreateAppMutex_SameName_ReturnsExistingHandle)
|
|
{
|
|
std::wstring mutexName = L"TestMutex_" + std::to_wstring(GetCurrentProcessId()) + L"_2";
|
|
|
|
auto handle1 = createAppMutex(mutexName);
|
|
Assert::IsNotNull(handle1.get());
|
|
|
|
auto handle2 = createAppMutex(mutexName);
|
|
Assert::IsNull(handle2.get());
|
|
}
|
|
|
|
TEST_METHOD(CreateAppMutex_DifferentNames_ReturnsDifferentHandles)
|
|
{
|
|
std::wstring mutexName1 = L"TestMutex_" + std::to_wstring(GetCurrentProcessId()) + L"_A";
|
|
std::wstring mutexName2 = L"TestMutex_" + std::to_wstring(GetCurrentProcessId()) + L"_B";
|
|
|
|
auto handle1 = createAppMutex(mutexName1);
|
|
auto handle2 = createAppMutex(mutexName2);
|
|
|
|
Assert::IsNotNull(handle1.get());
|
|
Assert::IsNotNull(handle2.get());
|
|
Assert::AreNotEqual(handle1.get(), handle2.get());
|
|
}
|
|
|
|
TEST_METHOD(CreateAppMutex_EmptyName_ReturnsHandle)
|
|
{
|
|
// Empty name creates unnamed mutex
|
|
auto handle = createAppMutex(L"");
|
|
// CreateMutexW with empty string should still work
|
|
Assert::IsTrue(true);
|
|
// Test passes regardless - just checking it doesn't crash
|
|
Assert::IsTrue(true);
|
|
}
|
|
|
|
TEST_METHOD(CreateAppMutex_LongName_ReturnsHandle)
|
|
{
|
|
// Create a long mutex name
|
|
std::wstring mutexName = L"TestMutex_" + std::to_wstring(GetCurrentProcessId()) + L"_";
|
|
for (int i = 0; i < 50; ++i)
|
|
{
|
|
mutexName += L"LongNameSegment";
|
|
}
|
|
|
|
auto handle = createAppMutex(mutexName);
|
|
// Long names might fail, but shouldn't crash
|
|
Assert::IsTrue(true);
|
|
}
|
|
|
|
TEST_METHOD(CreateAppMutex_SpecialCharacters_ReturnsHandle)
|
|
{
|
|
std::wstring mutexName = L"TestMutex_" + std::to_wstring(GetCurrentProcessId()) + L"_Special!@#$%";
|
|
|
|
auto handle = createAppMutex(mutexName);
|
|
// Some special characters might not be valid in mutex names
|
|
Assert::IsTrue(true);
|
|
}
|
|
|
|
TEST_METHOD(CreateAppMutex_GlobalPrefix_ReturnsHandle)
|
|
{
|
|
// Global prefix for cross-session mutex
|
|
std::wstring mutexName = L"Global\\TestMutex_" + std::to_wstring(GetCurrentProcessId());
|
|
|
|
auto handle = createAppMutex(mutexName);
|
|
// Might require elevation, but shouldn't crash
|
|
Assert::IsTrue(true);
|
|
}
|
|
|
|
TEST_METHOD(CreateAppMutex_LocalPrefix_ReturnsHandle)
|
|
{
|
|
std::wstring mutexName = L"Local\\TestMutex_" + std::to_wstring(GetCurrentProcessId());
|
|
|
|
auto handle = createAppMutex(mutexName);
|
|
Assert::IsNotNull(handle.get());
|
|
}
|
|
|
|
TEST_METHOD(CreateAppMutex_MultipleCalls_AllSucceed)
|
|
{
|
|
std::vector<wil::unique_mutex_nothrow> handles;
|
|
for (int i = 0; i < 10; ++i)
|
|
{
|
|
std::wstring mutexName = L"TestMutex_" + std::to_wstring(GetCurrentProcessId()) +
|
|
L"_Multi_" + std::to_wstring(i);
|
|
auto handle = createAppMutex(mutexName);
|
|
Assert::IsNotNull(handle.get());
|
|
handles.push_back(std::move(handle));
|
|
}
|
|
}
|
|
|
|
TEST_METHOD(CreateAppMutex_ReleaseAndRecreate_Works)
|
|
{
|
|
std::wstring mutexName = L"TestMutex_" + std::to_wstring(GetCurrentProcessId()) + L"_Recreate";
|
|
|
|
auto handle1 = createAppMutex(mutexName);
|
|
Assert::IsNotNull(handle1.get());
|
|
handle1.reset();
|
|
|
|
// After closing, should be able to create again
|
|
auto handle2 = createAppMutex(mutexName);
|
|
Assert::IsNotNull(handle2.get());
|
|
}
|
|
};
|
|
}
|