Paths

Table of Contentst

Differential D15744

Add missing RepositoryURI table + run storage adjustments in tests
ClosedPublic
Actions

Authored by epriestley on Apr 18 2016, 11:46 AM.

Tags

None

Referenced Files

	F17920968: D15744.id.diff
	Wed, Jul 30, 4:36 AM

	F17914827: D15744.diff
	Tue, Jul 29, 7:46 PM

	F17853976: D15744.id37943.diff
	Sun, Jul 27, 2:33 PM

	F17830605: D15744.id37941.diff
	Sat, Jul 26, 11:13 AM

	F17809663: D15744.diff
	Fri, Jul 25, 4:58 PM

	F17754650: D15744.id37941.diff
	Tue, Jul 22, 11:08 AM

	F17752307: D15744.id37943.diff
	Tue, Jul 22, 6:51 AM

	F17736401: D15744.diff
	Sun, Jul 20, 5:11 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T10366: General support for multiple URIs for a repository
T10830: phabricator_repository.repository_uri Missing

Commits

rP9352ed8abbd2: Add missing RepositoryURI table + run storage adjustments in tests

Summary

Fixes T10830. Ref T10366. I wasn't writing to this table yet so I didn't build it, but the fact that bin/storage adjust would complain slipped my mind.

Add the table.
Make the tests run adjust. This is a little slow (a few extra seconds) but we could eventually move some steps like this to run server-side only.

Test Plan

Ran bin/storage upgrade -f, got a clean adjust.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37941.Apr 18 2016, 11:46 AM

epriestley retitled this revision from to Add missing RepositoryURI table + run storage adjustments in tests.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added tasks: T10830: phabricator_repository.repository_uri Missing, T10366: General support for multiple URIs for a repository.

epriestley mentioned this in T10830: phabricator_repository.repository_uri Missing.Apr 18 2016, 11:54 AM

chad accepted this revision.Apr 18 2016, 1:43 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Apr 18 2016, 1:43 PM

Closed by commit rP9352ed8abbd2: Add missing RepositoryURI table + run storage adjustments in tests (authored by epriestley, committed by epriestley). · Explain WhyApr 18 2016, 2:54 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.Apr 19 2016, 9:06 PM

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20160418.repouri.1.sql

14 lines

src/

applications/

repository/

storage/

PhabricatorRepositoryURI.php

2 lines

infrastructure/

testing/

fixture/

PhabricatorStorageFixtureScopeGuard.php

2 lines

Diff 37943

resources/sql/autopatches/20160418.repouri.1.sql

Loading...

src/applications/repository/storage/PhabricatorRepositoryURI.php

Loading...

src/infrastructure/testing/fixture/PhabricatorStorageFixtureScopeGuard.php

Loading...