Skip to content

Commit

Permalink
Tidy
Browse files Browse the repository at this point in the history
  • Loading branch information
mwjames committed Jan 4, 2016
1 parent aad1092 commit a11cd20
Show file tree
Hide file tree
Showing 23 changed files with 48 additions and 48 deletions.
4 changes: 2 additions & 2 deletions RELEASE-NOTES.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
This file contains the RELEASE-NOTES of the Semantic External Query Lookup (a.k.a. SEQL) extension.

### 1.0.0 (2015-??-??)
### 1.0.0 (2016-??-??)

- Initial release
- Added `ByAskApiHttpRequestQueryLookup` to retrieve query results from an external endpoint specified in `$GLOBALS['seqlgExternalRepositoryEndpoints']`
- Added `ByHttpRequestQueryLookup` to retrieve query results from an external endpoint specified in `$GLOBALS['seqlgExternalRepositoryEndpoints']`
4 changes: 2 additions & 2 deletions SemanticExternalQueryLookup.php
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,8 @@
);

// Alias to keep LocalSettings independent from the internal NS usage
class_alias( 'SEQL\ByAskApiHttpRequestQueryLookup', 'SMWExternalQueryLookup' ); // deprecated
class_alias( 'SEQL\ByAskApiHttpRequestQueryLookup', 'SMWExternalAskQueryLookup' );
class_alias( 'SEQL\ByHttpRequestQueryLookup', 'SMWExternalQueryLookup' ); // deprecated
class_alias( 'SEQL\ByHttpRequestQueryLookup', 'SMWExternalAskQueryLookup' );

// Register message files
$GLOBALS['wgMessagesDirs']['semantic-external-query-lookup'] = __DIR__ . '/i18n';
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?php

namespace SEQL\ByAskApiHttpRequest;
namespace SEQL\ByHttpRequest;

use SMW\DataValueFactory;
use SMW\DIProperty;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?php

namespace SEQL\ByAskApiHttpRequest;
namespace SEQL\ByHttpRequest;

use SEQL\DataValueDeserializer;
use SMW\DIProperty;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?php

namespace SEQL\ByAskApiHttpRequest;
namespace SEQL\ByHttpRequest;

use SMWInfolink as Infolink;
use SMWQueryResult as RootQueryResult;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?php

namespace SEQL\ByAskApiHttpRequest;
namespace SEQL\ByHttpRequest;

use Onoi\HttpRequest\HttpRequestFactory;
use SEQL\QueryEncoder;
Expand Down Expand Up @@ -123,7 +123,7 @@ public function fetchQueryResult( Query $query ) {

$this->jsonResponseParser->doParse( $result );

$queryResult = $this->queryResultFactory->newByAskApiHttpRequestQueryResult(
$queryResult = $this->queryResultFactory->newByHttpRequestQueryResult(
$query,
$this->jsonResponseParser
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
namespace SEQL;

use Onoi\HttpRequest\HttpRequestFactory;
use SEQL\ByAskApiHttpRequest\JsonResponseParser;
use SEQL\ByAskApiHttpRequest\QueryResultFetcher;
use SEQL\ByHttpRequest\JsonResponseParser;
use SEQL\ByHttpRequest\QueryResultFetcher;
use SMW\SQLStore\SQLStore;
use SMW\ApplicationFactory;
use SMWQuery as Query;
Expand All @@ -17,7 +17,7 @@
*
* @author mwjames
*/
class ByAskApiHttpRequestQueryLookup extends SQLStore {
class ByHttpRequestQueryLookup extends SQLStore {

/**
* @var QueryResultFactory
Expand Down
8 changes: 4 additions & 4 deletions src/QueryResultFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@

namespace SEQL;

use SEQL\ByAskApiHttpRequest\JsonResponseParser;
use SEQL\ByAskApiHttpRequest\QueryResult as ByAskApiHttpRequestQueryResult;
use SEQL\ByHttpRequest\JsonResponseParser;
use SEQL\ByHttpRequest\QueryResult as ByHttpRequestQueryResult;
use SMW\Store;
use SMWQuery as Query;
use SMWQueryResult as QueryResult;
Expand Down Expand Up @@ -55,9 +55,9 @@ public function newEmptyQueryResult( Query $query ) {
*
* @return QueryResult
*/
public function newByAskApiHttpRequestQueryResult( Query $query, JsonResponseParser $jsonResponseParser ) {
public function newByHttpRequestQueryResult( Query $query, JsonResponseParser $jsonResponseParser ) {

$queryResult = new ByAskApiHttpRequestQueryResult(
$queryResult = new ByHttpRequestQueryResult(
$query->getDescription()->getPrintrequests(),
$query,
$jsonResponseParser->getResultSubjectList(),
Expand Down
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
<?php

namespace SEQL\Tests\Integration\ByAskApiHttpRequest;
namespace SEQL\Tests\Integration\ByHttpRequest;

use SMW\Tests\Utils\UtilityFactory;
use Onoi\HttpRequest\HttpRequestFactory;
use SEQL\ByAskApiHttpRequest\QueryResultFetcher;
use SEQL\ByAskApiHttpRequest\JsonResponseParser;
use SEQL\ByHttpRequest\QueryResultFetcher;
use SEQL\ByHttpRequest\JsonResponseParser;
use SEQL\QueryResultFactory;
use SEQL\DataValueDeserializer;
use SEQL\HookRegistry;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
<?php

namespace SEQL\ByAskApiHttpRequest\Tests;
namespace SEQL\ByHttpRequest\Tests;

use SEQL\ByAskApiHttpRequest\CannedResultArray;
use SEQL\ByHttpRequest\CannedResultArray;
use SMW\DIWikiPage;
use SMW\DIProperty;
use SMWDINumber as DINumber;

/**
* @covers \SEQL\ByAskApiHttpRequest\CannedResultArray
* @covers \SEQL\ByHttpRequest\CannedResultArray
* @group semantic-external-query-lookup
*
* @license GNU GPL v2+
Expand All @@ -22,7 +22,7 @@ class CannedResultArrayTest extends \PHPUnit_Framework_TestCase {

protected function setUp() {

$this->jsonResponseParser = $this->getMockBuilder( '\SEQL\ByAskApiHttpRequest\JsonResponseParser' )
$this->jsonResponseParser = $this->getMockBuilder( '\SEQL\ByHttpRequest\JsonResponseParser' )
->disableOriginalConstructor()
->getMock();
}
Expand All @@ -34,7 +34,7 @@ public function testCanConstruct() {
->getMock();

$this->assertInstanceOf(
'\SEQL\ByAskApiHttpRequest\CannedResultArray',
'\SEQL\ByHttpRequest\CannedResultArray',
new CannedResultArray( new DIWikiPage( 'Foo', NS_MAIN ), $printRequest, $this->jsonResponseParser )
);
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
<?php

namespace SEQL\ByAskApiHttpRequest\Tests;
namespace SEQL\ByHttpRequest\Tests;

use SEQL\ByAskApiHttpRequest\JsonResponseParser;
use SEQL\ByHttpRequest\JsonResponseParser;
use SMW\DIProperty;

/**
* @covers \SEQL\ByAskApiHttpRequest\JsonResponseParser
* @covers \SEQL\ByHttpRequest\JsonResponseParser
* @group semantic-external-query-lookup
*
* @license GNU GPL v2+
Expand All @@ -23,7 +23,7 @@ public function testCanConstruct() {
->getMock();

$this->assertInstanceOf(
'\SEQL\ByAskApiHttpRequest\JsonResponseParser',
'\SEQL\ByHttpRequest\JsonResponseParser',
new JsonResponseParser( $dataValueDeserializer )
);
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
<?php

namespace SEQL\ByAskApiHttpRequest\Tests;
namespace SEQL\ByHttpRequest\Tests;

use SEQL\ByAskApiHttpRequest\QueryResultFetcher;
use SEQL\ByHttpRequest\QueryResultFetcher;
use SEQL\QueryResultFactory;
use SMW\DIWikiPage;
use SMW\DIProperty;

/**
* @covers \SEQL\ByAskApiHttpRequest\QueryResultFetcher
* @covers \SEQL\ByHttpRequest\QueryResultFetcher
* @group semantic-external-query-lookup
*
* @license GNU GPL v2+
Expand Down Expand Up @@ -41,7 +41,7 @@ protected function setUp() {
->method( 'newCachedCurlRequest' )
->will( $this->returnValue( $this->httpRequest ) );

$this->jsonResponseParser = $this->getMockBuilder( '\SEQL\ByAskApiHttpRequest\JsonResponseParser' )
$this->jsonResponseParser = $this->getMockBuilder( '\SEQL\ByHttpRequest\JsonResponseParser' )
->disableOriginalConstructor()
->getMock();
}
Expand All @@ -53,7 +53,7 @@ public function testCanConstruct() {
->getMock();

$this->assertInstanceOf(
'\SEQL\ByAskApiHttpRequest\QueryResultFetcher',
'\SEQL\ByHttpRequest\QueryResultFetcher',
new QueryResultFetcher( $this->httpRequestFactory, $queryResultFactory, $this->jsonResponseParser )
);
}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
<?php

namespace SEQL\ByAskApiHttpRequest\Tests;
namespace SEQL\ByHttpRequest\Tests;

use SEQL\ByAskApiHttpRequest\QueryResult;
use SEQL\ByHttpRequest\QueryResult;
use SMW\DIWikiPage;

/**
* @covers \SEQL\ByAskApiHttpRequest\QueryResult
* @covers \SEQL\ByHttpRequest\QueryResult
* @group semantic-external-query-lookup
*
* @license GNU GPL v2+
Expand Down Expand Up @@ -35,14 +35,14 @@ public function testCanConstruct() {
$results = array();

$this->assertInstanceOf(
'\SEQL\ByAskApiHttpRequest\QueryResult',
'\SEQL\ByHttpRequest\QueryResult',
new QueryResult( $printRequests, $query, $results, $this->store, false )
);
}

public function testGetNext() {

$jsonResponseParser = $this->getMockBuilder( '\SEQL\ByAskApiHttpRequest\JsonResponseParser' )
$jsonResponseParser = $this->getMockBuilder( '\SEQL\ByHttpRequest\JsonResponseParser' )
->disableOriginalConstructor()
->getMock();

Expand All @@ -67,7 +67,7 @@ public function testGetNext() {

foreach ( $instance->getNext() as $result ) {
$this->assertInstanceOf(
'\SEQL\ByAskApiHttpRequest\CannedResultArray',
'\SEQL\ByHttpRequest\CannedResultArray',
$result
);
}
Expand All @@ -87,7 +87,7 @@ public function testToArray() {
->disableOriginalConstructor()
->getMock();

$jsonResponseParser = $this->getMockBuilder( '\SEQL\ByAskApiHttpRequest\JsonResponseParser' )
$jsonResponseParser = $this->getMockBuilder( '\SEQL\ByHttpRequest\JsonResponseParser' )
->disableOriginalConstructor()
->getMock();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,19 +5,19 @@
use SMWQuery as Query;

/**
* @covers \SEQL\ByAskApiHttpRequestQueryLookup
* @covers \SEQL\ByHttpRequestQueryLookup
* @group semantic-external-query-lookup
*
* @license GNU GPL v2+
* @since 1.0
*
* @author mwjames
*/
class ByAskApiHttpRequestQueryLookupTest extends \PHPUnit_Framework_TestCase {
class ByHttpRequestQueryLookupTest extends \PHPUnit_Framework_TestCase {

public function testCanConstruct() {

$instance = $this->getMockBuilder( '\SEQL\ByAskApiHttpRequestQueryLookup' )
$instance = $this->getMockBuilder( '\SEQL\ByHttpRequestQueryLookup' )
->disableOriginalConstructor()
->getMock();

Expand All @@ -41,7 +41,7 @@ public function testGetQueryResult() {
->method( 'getDescription' )
->will( $this->returnValue( $description ) );

$instance = $this->getMockBuilder( '\SEQL\ByAskApiHttpRequestQueryLookup' )
$instance = $this->getMockBuilder( '\SEQL\ByHttpRequestQueryLookup' )
->disableOriginalConstructor()
->setMethods( null )
->getMock();
Expand Down Expand Up @@ -69,7 +69,7 @@ public function testGetEmptyQueryResult_MODE_DEBUG() {
$query->expects( $this->once() )
->method( 'addErrors' );

$instance = $this->getMockBuilder( '\SEQL\ByAskApiHttpRequestQueryLookup' )
$instance = $this->getMockBuilder( '\SEQL\ByHttpRequestQueryLookup' )
->disableOriginalConstructor()
->setMethods( null )
->getMock();
Expand Down Expand Up @@ -108,7 +108,7 @@ public function testGetQueryResultForSimulatedInterwikiMatch() {
->method( 'getSortKeys' )
->will( $this->returnValue( array() ) );

$instance = $this->getMockBuilder( '\SEQL\ByAskApiHttpRequestQueryLookup' )
$instance = $this->getMockBuilder( '\SEQL\ByHttpRequestQueryLookup' )
->disableOriginalConstructor()
->setMethods( array( 'tryToMatchInterwikiFor' ) )
->getMock();
Expand Down
4 changes: 2 additions & 2 deletions tests/phpunit/Unit/QueryResultFactoryTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ public function testNewEmptyQueryResult() {

public function testNewByHttpLookupQueryResult() {

$jsonResponseParser = $this->getMockBuilder( '\SEQL\ByAskApiHttpRequest\JsonResponseParser' )
$jsonResponseParser = $this->getMockBuilder( '\SEQL\ByHttpRequest\JsonResponseParser' )
->disableOriginalConstructor()
->getMockForAbstractClass();

Expand All @@ -76,7 +76,7 @@ public function testNewByHttpLookupQueryResult() {

$this->assertInstanceOf(
'\SMWQueryResult',
$instance->newByAskApiHttpRequestQueryResult( $query, $jsonResponseParser )
$instance->newByHttpRequestQueryResult( $query, $jsonResponseParser )
);
}

Expand Down

0 comments on commit a11cd20

Please sign in to comment.