Page MenuHomePhabricator
Differential D16952

Fix local time test case for logged-out viewers using global settings
ClosedPublic
Actions

Authored by epriestley on Nov 28 2016, 2:53 PM.
Tags
None
Referenced Files
F19872264: D16952.diff
Sun, Mar 15, 12:43 PM
F19610136: D16952.id40806.diff
Feb 3 2026, 3:16 PM
F19562501: D16952.id40799.diff
Jan 31 2026, 2:13 PM
F19136888: D16952.id40799.diff
Dec 10 2025, 9:55 AM
F19107235: D16952.id40806.diff
Dec 7 2025, 10:53 PM
F19094082: D16952.id40799.diff
Dec 3 2025, 11:15 PM
F19074110: D16952.id.diff
Dec 1 2025, 4:23 AM
F19067864: D16952.diff
Nov 30 2025, 11:05 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Commits
rPab3b70739614: Fix local time test case for logged-out viewers using global settings
Summary

In D16936, I changed logged-out viewers so they use global settings.

This can lead to a SELECT from an isolated unit test. Instead, give the test fixtures and use standard generateNewUser() stuff.

Test Plan

Ran arc unit --everything.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 40799.Nov 28 2016, 2:53 PM
epriestley retitled this revision from to Fix local time test case for logged-out viewers using global settings.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad accepted this revision.Nov 28 2016, 4:59 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Nov 28 2016, 4:59 PM
Closed by commit rPab3b70739614: Fix local time test case for logged-out viewers using global settings (authored by epriestley, committed by epriestley). · Explain WhyNov 28 2016, 5:27 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
view/
__tests__/
10 lines

Diff 40806

View Options

src/view/__tests__/PhabricatorLocalTimeTestCase.php

Loading...
Phabricator · Built by Phacility