Update tests

nexcess · Sep 8, 2020 · f095efb · f095efb
1 parent ebf5b7b
commit f095efb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/tests/unit/CopyRequestTest.php b/tests/unit/CopyRequestTest.php
@@ -60,19 +60,20 @@ public function testDestruct()
         $tmpfile = tempnam(sys_get_temp_dir(), 'composer_unit_test_');
 
         $req = new CopyRequest('http://example.com/', $tmpfile, false, $this->iop->reveal(), $this->configp->reveal());
+        $req->getCurlOptions();
+        $req->makeSuccess();
         $this->assertFileExists($tmpfile);
 
-        // if $req->success === true ...
-        $req->makeSuccess();
+        // if $req->success === true (default, or on success) ...
         unset($req);
 
         // then tmpfile remain
         $this->assertFileExists($tmpfile);
         unlink($tmpfile);
 
         $req = new CopyRequest('http://example.com/', $tmpfile, false, $this->iop->reveal(), $this->configp->reveal());
-        // if $req->success === false (default) ...
-        // $req->makeSuccess();
+        // if $req->success === false (flagged when download starts) ...
+        $req->getCurlOptions();
         unset($req);
 
         // then cleaned tmpfile automatically

diff --git a/tests/unit/PrefetcherTest.php b/tests/unit/PrefetcherTest.php
@@ -26,7 +26,7 @@ public function testFetchAllOnFailure()
             CURLOPT_URL => 'file://uso800.txt',
             CURLOPT_FILE => tmpfile(),
         ));
-     
+
         $this->iop->writeError(arg::containingString("<warning>37: Couldn't open file /uso800.txt</warning>"), true, IOInterface::NORMAL)->shouldBeCalledTimes(1);
         $this->iop->writeError("    Finished: <comment>success: 0, skipped: 0, failure: 1, total: 1</comment>", true, IOInterface::NORMAL)->shouldBeCalledTimes(1);