Convert locale tests to PSR-0 [#1591950]

Comment	File	Size	Author
#7	1591950-locale-tests-7.patch	185.38 KB	aspilicious
#2	1591950-locale-tests-2.patch	185.36 KB	aspilicious
#1	1591950-locale-tests-1.patch	185.37 KB	aspilicious

Status	File	Size
new	1591950-locale-tests-1.patch	185.37 KB

Comment #2

aspilicious commented 21 May 2012 at 08:31

Status	File	Size
new	1591950-locale-tests-2.patch	185.36 KB

Previous one should fail...

Log in or register to post comments

Comment #3

gábor hojtsy

he/him

Hungarian

Hungary

commented 21 May 2012 at 09:36

Issue tags:

+sprint, +language-base

Adding sprint tag for D8MI. Thanks for working on this.

Log in or register to post comments

Comment #4

21 May 2012 at 10:42

Status:	Needs review	» Needs work
Issue tags:	-D8MI, -sprint, -PSR-0, -language-base

The last submitted patch, 1591950-locale-tests-2.patch, failed testing.

Log in or register to post comments

Comment #5

aspilicious commented 21 May 2012 at 10:43

Status:

Needs work

» Needs review

#2: 1591950-locale-tests-2.patch queued for re-testing.

Log in or register to post comments

Comment #6

21 May 2012 at 11:09

Status:	Needs review	» Needs work
Issue tags:		+D8MI, +sprint, +PSR-0, +language-base

The last submitted patch, 1591950-locale-tests-2.patch, failed testing.

Log in or register to post comments

Comment #7

aspilicious commented 21 May 2012 at 11:53

Status:

Needs work

» Needs review

Status	File	Size
new	1591950-locale-tests-7.patch	185.38 KB

Green please!

Log in or register to post comments

Comment #8

gábor hojtsy

he/him

Hungarian

Hungary

commented 22 May 2012 at 11:31

Status:

Needs review

» Reviewed & tested by the community

I did not review the tests line by line, trust @aspilicious that only the required changes were done. Since we are doing other test cleanup too (eg. #1561004: Many tests still reference locale although they only need language module) and this is a pain to reroll, let's get this in sooner than later.