Differential D14639

Add missing test data
AbandonedPublic
Actions

Authored by joshuaspence on Dec 2 2015, 8:55 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Feb 8, 11:57 PM

	Unknown Object (File)
	Sat, Feb 8, 11:57 PM

	Unknown Object (File)
	Sat, Feb 8, 11:57 PM

	Unknown Object (File)
	Tue, Feb 4, 5:34 AM

	Unknown Object (File)
	Mon, Feb 3, 5:48 PM

	Unknown Object (File)
	Mon, Jan 27, 2:58 AM

	Unknown Object (File)
	Fri, Jan 24, 10:13 AM

	Unknown Object (File)
	Fri, Jan 24, 10:13 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Summary

D14627 added an additional test case but did not add the expected outpud data for this test case.

Test Plan

Actually ran the unit tests this time.

Diff Detail

Repository

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 9228
Build 10927: Run Core Tests
Build 10926: arc lint + arc unit

Event Timeline

joshuaspence updated this revision to Diff 35417.Dec 2 2015, 8:55 PM

joshuaspence retitled this revision from to Add missing test data.

joshuaspence updated this object.

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

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptDec 2 2015, 8:55 PM

Herald added a subscriber: Korvin. · View Herald Transcript

See rPHU0b37f385b8cd instead.

joshuaspence mentioned this in rPHU0b37f385b8cd: Fix build for a test case with no test data..Dec 5 2015, 12:23 AM

Oh, sorry, totally missed this. I think not requiring the data is fine, this test case doesn't need 6K lines of JSON to serve the purpose it's supposed to serve.

In D14639#166440, @epriestley wrote:

Oh, sorry, totally missed this. I think not requiring the data is fine, this test case doesn't need 6K lines of JSON to serve the purpose it's supposed to serve.

Yeah, agreed.

Revision Contents
Changeset List

Path

Size

src/

parser/

xhpast/

__tests__/

data/

yyinitdepth.php.test

5861 lines

Diff 35417

src/parser/xhpast/tests/data/yyinitdepth.php.test

Loading...