Skip to content

Fix invalid instantiation of TemplateResponse if client not found - #19964

Merged
rullzer merged 1 commit into
masterfrom
bug/19963/invalid-constructor-template-response
Mar 17, 2020
Merged

Fix invalid instantiation of TemplateResponse if client not found#19964
rullzer merged 1 commit into
masterfrom
bug/19963/invalid-constructor-template-response

Conversation

@kesselb

Copy link
Copy Markdown
Contributor

No description provided.

@kesselb

Copy link
Copy Markdown
ContributorAuthor

/backport to stable18


namespace OCA\OAuth2\Tests\Controller;

use OCA\Files_Sharing\Tests\TestCase;

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's weird. I guess it was not intended to use OCA\Files_Sharing\Tests\TestCase as base class for LoginRedirectorControllerTest.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙈

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hopefully it's unrelated but a lot of tests failed related to sharing. Restarted the test a few hours ago but job is still pending. Probably the CI want's a weekend too.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb
kesselbforce-pushed the bug/19963/invalid-constructor-template-response branch from 86046f0 to 509af24CompareMarch 15, 2020 10:55
@rullzer
rullzer merged commit 7e1bee3 into masterMar 17, 2020
@rullzer
rullzer deleted the bug/19963/invalid-constructor-template-response branch March 17, 2020 18:38
@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable18 in #20008

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

2 participants

@kesselb@rullzer