Move batch tests to the right directory

Test file needs to be deleted from .info file

+++ b/core/modules/system/lib/system/Tests/Batch/PercentagesTest.phpundefined
@@ -0,0 +1,104 @@
+class PercentagesTest extends UnitTestBase {

I would change this to PercentageUnitTest but I don't care if someone else would rtbc this

3 days to next Drupal core point release.

#3: 1598552.patch queued for re-testing.

Rerolled and applied the name change from #4 because it made sense.

+++ b/core/modules/system/lib/system/Tests/Batch/PercentagesTest.phpundefined
@@ -0,0 +1,104 @@
+ * Definition of Drupal\system\Tests\Batch\PercentagesTest.
...
+class PercentagesUnitTest extends UnitTestBase {

Forgot to change the name in the header

23 days to next Drupal core point release.

Also, I didn't change the name of the file.

I remembered about this last night as I was going to bed.

good!

Um. Take a look at this patch... it's all weird characters.... can't read it at all?

jhodgdon: can you please look again http://drupal.org/files/1598552_9_batch_psr.patch

I don't see the weird characters you speak of.

? I'm not sure what jhogdon means but you forgot to remove the line from the info file.

this patch attempts to fix both issues:

the previous patch is much smaller because it attempts to move instead of copy the code from the existing file to the new file.

This newer file is viewable, but when I look at the patch in #9, this is what I see. Weird.

$ enca -L none 1598552_9_batch_psr.patch 
Universal character set 2 bytes; UCS-2; BMP
  CRLF line terminators
  Byte order reversed in pairs (1,2 -> 2,1)

So that one had windows line endings, UCS character set and reverse byte order.

$ enca -L none 1598552_14_batch_psr.patch 
7bit ASCII characters

The newer patch is fine. (Character set. Didn't review it.)

back to rtbc

Thanks. Committed/pushed to 8.x.

Looks like these were accidantely moved to the wrong directory core/modules/system/lib/system/Tests/Batch, rather than core/modules/system/lib/Drupal/system/Tests/Batch. Bumping to major, because the tests are currently not executed.

Excellent, thanks. (Assuming tests pass. However could be not, because it's the first time they run with PSR-0.)

Oops. Committed/pushed, thanks!

Looks like these were accidantely moved to the wrong directory core/modules/system/lib/system/Tests/Batch, rather than core/modules/system/lib/Drupal/system/Tests/Batch. Bumping to major, because the tests are currently not executed.

Since that's pretty bad, I've written a patch at #1632364: Write tests to ensure that all classes in Drupal can actually be found by the autoloader that should help us catch these mistakes automatically in the future, rather than having to find them manually.