forked from wallabag/wallabag
Integrate bdunogier/guzzle-site-authenticator in core
This commit is contained in:
Yassine Guedidi
@ -0,0 +1,26 @@
|
||||
<?php
|
||||
|
||||
namespace Tests\Wallabag\CoreBundle\SiteConfig;
|
||||
|
||||
use PHPUnit\Framework\TestCase;
|
||||
use Wallabag\CoreBundle\SiteConfig\ArraySiteConfigBuilder;
|
||||
use Wallabag\CoreBundle\SiteConfig\SiteConfig;
|
||||
|
||||
class ArraySiteConfigBuilderTest extends TestCase
|
||||
{
|
||||
public function testItReturnsSiteConfigThatExists()
|
||||
{
|
||||
$builder = new ArraySiteConfigBuilder(['example.com' => []]);
|
||||
$res = $builder->buildForHost('www.example.com');
|
||||
|
||||
$this->assertInstanceOf(SiteConfig::class, $res);
|
||||
}
|
||||
|
||||
public function testItReturnsFalseOnAHostThatDoesNotExist()
|
||||
{
|
||||
$builder = new ArraySiteConfigBuilder(['anotherexample.com' => []]);
|
||||
$res = $builder->buildForHost('example.com');
|
||||
|
||||
$this->assertfalse($res);
|
||||
}
|
||||
}
|
||||
@ -0,0 +1,235 @@
|
||||
<?php
|
||||
|
||||
namespace Tests\Wallabag\CoreBundle\SiteConfig\Authenticator;
|
||||
|
||||
use GuzzleHttp\Client;
|
||||
use GuzzleHttp\Message\Response;
|
||||
use GuzzleHttp\Stream\Stream;
|
||||
use GuzzleHttp\Subscriber\Mock;
|
||||
use PHPUnit\Framework\TestCase;
|
||||
use Wallabag\CoreBundle\SiteConfig\Authenticator\LoginFormAuthenticator;
|
||||
use Wallabag\CoreBundle\SiteConfig\SiteConfig;
|
||||
|
||||
class LoginFormAuthenticatorTest extends TestCase
|
||||
{
|
||||
public function testLoginPost()
|
||||
{
|
||||
$response = new Response(
|
||||
200,
|
||||
['content-type' => 'text/html'],
|
||||
Stream::factory('')
|
||||
);
|
||||
$guzzle = new Client();
|
||||
$guzzle->getEmitter()->attach(new Mock([$response]));
|
||||
|
||||
$siteConfig = new SiteConfig([
|
||||
'host' => 'example.com',
|
||||
'loginUri' => 'http://example.com/login',
|
||||
'usernameField' => 'username',
|
||||
'passwordField' => 'password',
|
||||
'extraFields' => [
|
||||
'action' => 'login',
|
||||
'foo' => 'bar',
|
||||
],
|
||||
'username' => 'johndoe',
|
||||
'password' => 'unkn0wn',
|
||||
]);
|
||||
|
||||
$auth = new LoginFormAuthenticator($siteConfig);
|
||||
$res = $auth->login($guzzle);
|
||||
|
||||
$this->assertInstanceOf(LoginFormAuthenticator::class, $res);
|
||||
}
|
||||
|
||||
public function testLoginPostWithExtraFieldsButEmptyHtml()
|
||||
{
|
||||
$response = new Response(
|
||||
200,
|
||||
['content-type' => 'text/html'],
|
||||
Stream::factory('<html></html>')
|
||||
);
|
||||
$guzzle = new Client();
|
||||
$guzzle->getEmitter()->attach(new Mock([$response, $response]));
|
||||
|
||||
$siteConfig = new SiteConfig([
|
||||
'host' => 'example.com',
|
||||
'loginUri' => 'http://example.com/login',
|
||||
'usernameField' => 'username',
|
||||
'passwordField' => 'password',
|
||||
'extraFields' => [
|
||||
'action' => 'login',
|
||||
'foo' => 'bar',
|
||||
'security' => '@=xpath(\'substring(//script[contains(text(), "security")]/text(), 112, 10)\', request_html(\'https://aoc.media/\'))',
|
||||
],
|
||||
'username' => 'johndoe',
|
||||
'password' => 'unkn0wn',
|
||||
]);
|
||||
|
||||
$auth = new LoginFormAuthenticator($siteConfig);
|
||||
$res = $auth->login($guzzle);
|
||||
|
||||
$this->assertInstanceOf(LoginFormAuthenticator::class, $res);
|
||||
}
|
||||
|
||||
// testing preg_match
|
||||
public function testLoginPostWithExtraFieldsWithRegex()
|
||||
{
|
||||
$response = $this->getMockBuilder(Response::class)
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
|
||||
$response->expects($this->any())
|
||||
->method('getBody')
|
||||
->willReturn(file_get_contents(__DIR__ . '/../../fixtures/aoc.media.html'));
|
||||
|
||||
$response->expects($this->any())
|
||||
->method('getStatusCode')
|
||||
->willReturn(200);
|
||||
|
||||
$client = $this->getMockBuilder(Client::class)
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
|
||||
$client->expects($this->any())
|
||||
->method('post')
|
||||
->with(
|
||||
$this->equalTo('https://aoc.media/wp-admin/admin-ajax.php'),
|
||||
$this->equalTo([
|
||||
'body' => [
|
||||
'nom' => 'johndoe',
|
||||
'password' => 'unkn0wn',
|
||||
'security' => 'c506c1b8bc',
|
||||
'action' => 'login_user',
|
||||
],
|
||||
'allow_redirects' => true,
|
||||
'verify' => false,
|
||||
])
|
||||
)
|
||||
->willReturn($response);
|
||||
|
||||
$client->expects($this->any())
|
||||
->method('get')
|
||||
->with(
|
||||
$this->equalTo('https://aoc.media/'),
|
||||
$this->equalTo([])
|
||||
)
|
||||
->willReturn($response);
|
||||
|
||||
$siteConfig = new SiteConfig([
|
||||
'host' => 'aoc.media',
|
||||
'loginUri' => 'https://aoc.media/wp-admin/admin-ajax.php',
|
||||
'usernameField' => 'nom',
|
||||
'passwordField' => 'password',
|
||||
'extraFields' => [
|
||||
'action' => 'login_user',
|
||||
'security' => '@=preg_match(\'/security\\\":\\\"([a-z0-9]+)\\\"/si\', request_html(\'https://aoc.media/\'))',
|
||||
],
|
||||
'username' => 'johndoe',
|
||||
'password' => 'unkn0wn',
|
||||
]);
|
||||
|
||||
$auth = new LoginFormAuthenticator($siteConfig);
|
||||
$res = $auth->login($client);
|
||||
|
||||
$this->assertInstanceOf(LoginFormAuthenticator::class, $res);
|
||||
}
|
||||
|
||||
public function testLoginPostWithExtraFieldsWithData()
|
||||
{
|
||||
$response = $this->getMockBuilder(Response::class)
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
|
||||
$response->expects($this->any())
|
||||
->method('getBody')
|
||||
->willReturn(file_get_contents(__DIR__ . '/../../fixtures/nextinpact-login.html'));
|
||||
|
||||
$response->expects($this->any())
|
||||
->method('getStatusCode')
|
||||
->willReturn(200);
|
||||
|
||||
$client = $this->getMockBuilder(Client::class)
|
||||
->disableOriginalConstructor()
|
||||
->getMock();
|
||||
|
||||
$client->expects($this->any())
|
||||
->method('post')
|
||||
->with(
|
||||
$this->equalTo('https://compte.nextinpact.com/Account/Login'),
|
||||
$this->equalTo([
|
||||
'body' => [
|
||||
'UserName' => 'johndoe',
|
||||
'Password' => 'unkn0wn',
|
||||
'__RequestVerificationToken' => 's6x2QcnQDUL92mkKSi_JuUBXcgUYx_Plf-KyQ2eJypKAjQZIeTvaFHOsfEdTrcSXt3dt2CW39V7r9V16LUtvjszodAU1',
|
||||
'returnUrl' => 'https://www.nextinpact.com/news/102835-pour-cour-comptes-fonctionnement-actuel-vote-par-internet-nest-pas-satisfaisant.htm',
|
||||
],
|
||||
'allow_redirects' => true,
|
||||
'verify' => false,
|
||||
])
|
||||
)
|
||||
->willReturn($response);
|
||||
|
||||
$client->expects($this->any())
|
||||
->method('get')
|
||||
->with(
|
||||
$this->equalTo('https://compte.nextinpact.com/Account/Login?http://www.nextinpact.com/'),
|
||||
$this->equalTo([
|
||||
'headers' => [
|
||||
'X-Requested-With' => 'XMLHttpRequest',
|
||||
],
|
||||
])
|
||||
)
|
||||
->willReturn($response);
|
||||
|
||||
$siteConfig = new SiteConfig([
|
||||
'host' => 'nextinpact.com',
|
||||
'loginUri' => 'https://compte.nextinpact.com/Account/Login',
|
||||
'usernameField' => 'UserName',
|
||||
'passwordField' => 'Password',
|
||||
'extraFields' => [
|
||||
'__RequestVerificationToken' => '@=xpath(\'//form[@action="/Account/Login"]/input[@name="__RequestVerificationToken"]\', request_html(\'https://compte.nextinpact.com/Account/Login?http://www.nextinpact.com/\', {\'headers\': {\'X-Requested-With\':\'XMLHttpRequest\'}}))',
|
||||
'returnUrl' => 'https://www.nextinpact.com/news/102835-pour-cour-comptes-fonctionnement-actuel-vote-par-internet-nest-pas-satisfaisant.htm',
|
||||
],
|
||||
'username' => 'johndoe',
|
||||
'password' => 'unkn0wn',
|
||||
]);
|
||||
|
||||
$auth = new LoginFormAuthenticator($siteConfig);
|
||||
$res = $auth->login($client);
|
||||
|
||||
$this->assertInstanceOf(LoginFormAuthenticator::class, $res);
|
||||
}
|
||||
|
||||
public function testLoginWithBadSiteConfigNotLoggedInData()
|
||||
{
|
||||
$siteConfig = new SiteConfig([
|
||||
'host' => 'nextinpact.com',
|
||||
'loginUri' => 'https://compte.nextinpact.com/Account/Login',
|
||||
'usernameField' => 'UserName',
|
||||
'username' => 'johndoe',
|
||||
'password' => 'unkn0wn',
|
||||
]);
|
||||
|
||||
$auth = new LoginFormAuthenticator($siteConfig);
|
||||
$loginRequired = $auth->isLoginRequired(file_get_contents(__DIR__ . '/../../fixtures/nextinpact-login.html'));
|
||||
|
||||
$this->assertFalse($loginRequired);
|
||||
}
|
||||
|
||||
public function testLoginWithGoodSiteConfigNotLoggedInData()
|
||||
{
|
||||
$siteConfig = new SiteConfig([
|
||||
'host' => 'nextinpact.com',
|
||||
'loginUri' => 'https://compte.nextinpact.com/Account/Login',
|
||||
'usernameField' => 'UserName',
|
||||
'username' => 'johndoe',
|
||||
'password' => 'unkn0wn',
|
||||
'notLoggedInXpath' => '//h2[@class="title_reserve_article"]',
|
||||
]);
|
||||
|
||||
$auth = new LoginFormAuthenticator($siteConfig);
|
||||
$loginRequired = $auth->isLoginRequired(file_get_contents(__DIR__ . '/../../fixtures/nextinpact-article.html'));
|
||||
|
||||
$this->assertTrue($loginRequired);
|
||||
}
|
||||
}
|
||||
44
tests/Wallabag/CoreBundle/SiteConfig/SiteConfigTest.php
Normal file
44
tests/Wallabag/CoreBundle/SiteConfig/SiteConfigTest.php
Normal file
@ -0,0 +1,44 @@
|
||||
<?php
|
||||
|
||||
namespace Tests\Wallabag\CoreBundle\SiteConfig;
|
||||
|
||||
use PHPUnit\Framework\TestCase;
|
||||
use Wallabag\CoreBundle\SiteConfig\SiteConfig;
|
||||
|
||||
class SiteConfigTest extends TestCase
|
||||
{
|
||||
public function testInitSiteConfig()
|
||||
{
|
||||
$config = new SiteConfig([]);
|
||||
|
||||
$this->assertInstanceOf(SiteConfig::class, $config);
|
||||
}
|
||||
|
||||
public function testUnknownProperty()
|
||||
{
|
||||
$this->expectException(\InvalidArgumentException::class);
|
||||
$this->expectExceptionMessage('Unknown property: "bad"');
|
||||
|
||||
new SiteConfig(['bad' => true]);
|
||||
}
|
||||
|
||||
public function testInitSiteConfigWillFullOptions()
|
||||
{
|
||||
$config = new SiteConfig([
|
||||
'host' => 'example.com',
|
||||
'requiresLogin' => true,
|
||||
'notLoggedInXpath' => '//all',
|
||||
'loginUri' => 'https://example.com/login',
|
||||
'usernameField' => 'username',
|
||||
'passwordField' => 'password',
|
||||
'extraFields' => [
|
||||
'action' => 'login',
|
||||
'foo' => 'bar',
|
||||
],
|
||||
'username' => 'johndoe',
|
||||
'password' => 'unkn0wn',
|
||||
]);
|
||||
|
||||
$this->assertInstanceOf(SiteConfig::class, $config);
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user