Fix bug where messages got lost when using FileProvider #93

Author: czenker

src/Provider/FileProvider.php

@@ -132,7 +132,7 @@ public function cleanUp()
             ->name('*.json')
         ;
         $finder->date(
-            sprintf('> %d seconds ago', $this->options['message_expiration'])
+            sprintf('< %d seconds ago', $this->options['message_expiration'])
         );
         /** @var SplFileInfo $file */
         foreach ($finder as $file) {

tests/Provider/FileProviderTest.php

@@ -2,6 +2,7 @@
 
 namespace Uecode\Bundle\QPushBundle\Tests\Provider;
 
+use Symfony\Component\Finder\Finder;
 use Uecode\Bundle\QPushBundle\Event\MessageEvent;
 use Uecode\Bundle\QPushBundle\Provider\FileProvider;
 
@@ -169,4 +170,23 @@ public function testCleanUp()
         $messages = $provider->receive();
         $this->assertEmpty($messages);
     }
+
+    /**
+     * @see https://github.com/uecode/qpush-bundle/issues/93
+     */
+    public function testCleanUpDoesNotRemoveCurrentMessages() {
+        $this->provider->create();
+        $provider = $this->getFileProvider([
+            'message_expiration' => 2,
+        ]);
+        $provider->publish(['testing']);
+        $provider->publish(['testing 123']);
+        sleep(2);
+        $currentMessage = ['dont remove me'];
+        $provider->publish($currentMessage);
+        $provider->cleanUp();
+        $messages = $provider->receive();
+        $this->assertCount(1, $messages);
+        $this->assertSame($currentMessage, $messages[0]->getBody());
+    }
 }