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.

Sign up for GitHub

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

Jump to bottom

FancyZone unittest failure - CreateZoneWindowWithActiveZoneTmpFile #8866

Closed
crutkas opened this issue Dec 31, 2020 · 2 comments
Closed

FancyZone unittest failure - CreateZoneWindowWithActiveZoneTmpFile #8866

crutkas opened this issue Dec 31, 2020 · 2 comments
Assignees
@SeraphimaZykova
Labels
Area-Tests issues that relate to tests Issue-Bug Something isn't working Resolution-Already Fixed/Doesn't Apply A change in the product has made the issue obsolete.

Comments

@crutkas
Copy link
Member

crutkas commented Dec 31, 2020

This is failing on my machine locally.

CreateZoneWindowWithActiveZoneTmpFile
Source: ZoneWindow.Spec.cpp line 178
Duration: 137 ms

Message:
Exception Code: C0000005
Stack Trace:
_Atomic_storage<int,4>::load() line 922
logger::should_log() line 276
logwinrt::hstring() line 230
log<wchar_t [76],winrt::hstring>() line 94
trace<wchar_t [76],winrt::hstring>() line 100
trace<wchar_t [76],winrt::hstring>() line 22
JSONHelpers::ParseDeviceInfoFromTmpFile() line 698
FancyZonesData::ParseDeviceInfoFromTmpFile() line 498
ZoneWindowCreationUnitTests::CreateZoneWindowWithActiveZoneTmpFile() line 195
@crutkas crutkas added Issue-Bug Something isn't working Area-Tests issues that relate to tests labels Dec 31, 2020
@enricogior
Copy link
Contributor

This test was poorly designed since it uses the actual temp file used by a running instance of PowerToys.
It will be obsolete when #8148 will be merged since we will not use the temp file anymore.

@SeraphimaZykova
Copy link
Collaborator

This test was removed in the commit 646d61b

@enricogior enricogior added the Resolution-Already Fixed/Doesn't Apply A change in the product has made the issue obsolete. label Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SeraphimaZykova SeraphimaZykova

Labels
Area-Tests issues that relate to tests Issue-Bug Something isn't working Resolution-Already Fixed/Doesn't Apply A change in the product has made the issue obsolete.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@crutkas @enricogior @SeraphimaZykova