Fixing tests

Most were a case of adding void return statement to setUp and tearDown,
also needed to add 6th `null` param to indieauth method. Its for
code_verifier, issue created to work out what I should actually be doing
with that.
This commit is contained in:
Jonny Barnes 2019-03-17 14:06:18 +00:00
parent a71d6d7845
commit c96ba5e82a
6 changed files with 8 additions and 7 deletions

View file

@ -48,7 +48,8 @@ class TokenEndpointController extends Controller
request()->input('code'), request()->input('code'),
request()->input('me'), request()->input('me'),
request()->input('redirect_uri'), request()->input('redirect_uri'),
request()->input('client_id') request()->input('client_id'),
null // code_verifier
); );
if (array_key_exists('me', $auth)) { if (array_key_exists('me', $auth)) {
$scope = $auth['scope'] ?? ''; $scope = $auth['scope'] ?? '';

View file

@ -15,7 +15,7 @@ class ContactsTest extends TestCase
{ {
use DatabaseTransactions; use DatabaseTransactions;
public function tearDown() protected function tearDown(): void
{ {
if (file_exists(public_path() . '/assets/profile-images/tantek.com/image')) { if (file_exists(public_path() . '/assets/profile-images/tantek.com/image')) {
unlink(public_path() . '/assets/profile-images/tantek.com/image'); unlink(public_path() . '/assets/profile-images/tantek.com/image');

View file

@ -12,7 +12,7 @@ class ParseCachedWebMentionsTest extends TestCase
{ {
use DatabaseTransactions; use DatabaseTransactions;
public function setUp() protected function setUp(): void
{ {
parent::setUp(); parent::setUp();
@ -43,7 +43,7 @@ class ParseCachedWebMentionsTest extends TestCase
$this->assertTrue($webmentionTantek->updated_at->gt($webmentionTantek->created_at)); $this->assertTrue($webmentionTantek->updated_at->gt($webmentionTantek->created_at));
} }
public function tearDown() protected function tearDown(): void
{ {
$fs = new FileSystem(); $fs = new FileSystem();
if ($fs->exists(storage_path() . '/HTML/https')) { if ($fs->exists(storage_path() . '/HTML/https')) {

View file

@ -13,7 +13,7 @@ use Illuminate\Foundation\Testing\RefreshDatabase;
class DownloadWebMentionJobTest extends TestCase class DownloadWebMentionJobTest extends TestCase
{ {
public function tearDown() protected function tearDown(): void
{ {
$fs = new FileSystem(); $fs = new FileSystem();
if ($fs->exists(storage_path() . '/HTML/https')) { if ($fs->exists(storage_path() . '/HTML/https')) {

View file

@ -20,7 +20,7 @@ class ProcessWebMentionJobTest extends TestCase
{ {
use DatabaseTransactions; use DatabaseTransactions;
public function tearDown() protected function tearDown(): void
{ {
$fs = new FileSystem(); $fs = new FileSystem();
if ($fs->exists(storage_path() . '/HTML/https')) { if ($fs->exists(storage_path() . '/HTML/https')) {

View file

@ -14,7 +14,7 @@ use Jonnybarnes\WebmentionsParser\Exceptions\AuthorshipParserException;
class SaveProfileImageJobTest extends TestCase class SaveProfileImageJobTest extends TestCase
{ {
public function tearDown() protected function tearDown(): void
{ {
if (file_exists(public_path() . '/assets/profile-images/example.org/image')) { if (file_exists(public_path() . '/assets/profile-images/example.org/image')) {
unlink(public_path() . '/assets/profile-images/example.org/image'); unlink(public_path() . '/assets/profile-images/example.org/image');