Fix deprecated method use in ORM tests.

tests/TestCase/ORM/AssociationCollectionTest.php

@@ -155,7 +155,7 @@ class AssociationCollectionTest extends TestCase
         $table = $this->getMockBuilder('Cake\ORM\Table')
             ->setMethods(['table'])
             ->getMock();
-        $table->schema([]);
+        $table->setSchema([]);
         $belongsTo = new BelongsTo('Users', [
             'sourceTable' => $table
         ]);
@@ -282,7 +282,7 @@ class AssociationCollectionTest extends TestCase
         $table = $this->getMockBuilder('Cake\ORM\Table')
             ->setMethods(['table'])
             ->getMock();
-        $table->schema([]);
+        $table->setSchema([]);
         $mockOne = $this->getMockBuilder('Cake\ORM\Association\BelongsTo')
             ->setMethods(['saveAssociated'])
             ->setConstructorArgs(['Parent', [
@@ -332,7 +332,7 @@ class AssociationCollectionTest extends TestCase
         $table = $this->getMockBuilder('Cake\ORM\Table')
             ->setMethods(['table'])
             ->getMock();
-        $table->schema([]);
+        $table->setSchema([]);
         $mockOne = $this->getMockBuilder('Cake\ORM\Association\BelongsTo')
             ->setMethods(['saveAssociated'])
             ->setConstructorArgs(['Parents', [
@@ -382,7 +382,7 @@ class AssociationCollectionTest extends TestCase
         $table = $this->getMockBuilder('Cake\ORM\Table')
             ->setMethods(['table'])
             ->getMock();
-        $table->schema([]);
+        $table->setSchema([]);
         $mockOne = $this->getMockBuilder('Cake\ORM\Association\HasMany')
             ->setMethods(['saveAssociated'])
             ->setConstructorArgs(['Comments', [

tests/TestCase/ORM/AssociationTest.php

@@ -27,7 +27,7 @@ class TestTable extends Table
 
     public function initialize(array $config = [])
     {
-        $this->schema(['id' => ['type' => 'integer']]);
+        $this->setSchema(['id' => ['type' => 'integer']]);
     }
 
     public function findPublished($query)
@@ -302,7 +302,7 @@ class AssociationTest extends TestCase
      */
     public function testBindingKeyDefault()
     {
-        $this->source->primaryKey(['id', 'site_id']);
+        $this->source->setPrimaryKey(['id', 'site_id']);
         $this->association
             ->expects($this->once())
             ->method('isOwningSide')
@@ -320,7 +320,7 @@ class AssociationTest extends TestCase
     public function testBindingDefaultNoOwningSide()
     {
         $target = new Table;
-        $target->primaryKey(['foo', 'site_id']);
+        $target->setPrimaryKey(['foo', 'site_id']);
         $this->association->setTarget($target);
 
         $this->association
@@ -584,7 +584,7 @@ class AssociationTest extends TestCase
     {
         $this->expectException(\PHPUnit\Framework\Error\Warning::class);
         $this->expectExceptionMessageRegExp('/^Association property name "foo" clashes with field of same name of table "test"/');
-        $this->source->schema(['foo' => ['type' => 'string']]);
+        $this->source->setSchema(['foo' => ['type' => 'string']]);
         $this->assertEquals('foo', $this->association->getProperty());
     }
 
@@ -595,7 +595,7 @@ class AssociationTest extends TestCase
      */
     public function testPropertyNameExplicitySet()
     {
-        $this->source->schema(['foo' => ['type' => 'string']]);
+        $this->source->setSchema(['foo' => ['type' => 'string']]);
 
         $config = [
             'className' => '\Cake\Test\TestCase\ORM\TestTable',

tests/TestCase/ORM/Behavior/BehaviorRegressionTest.php

@@ -52,10 +52,10 @@ class BehaviorRegressionTest extends TestCase
     public function testTreeAndTranslateIntegration()
     {
         $table = TableRegistry::get('NumberTrees');
-        $table->primaryKey(['id']);
+        $table->setPrimaryKey(['id']);
         $table->addBehavior('Tree');
         $table->addBehavior('Translate', ['fields' => ['name']]);
-        $table->entityClass(__NAMESPACE__ . '\\NumberTree');
+        $table->setEntityClass(__NAMESPACE__ . '\\NumberTree');
 
         $all = $table->find('threaded')->find('translations');
         $results = [];

tests/TestCase/ORM/Behavior/TranslateBehaviorTest.php

@@ -98,7 +98,7 @@ class TranslateBehaviorTest extends TestCase
 
         $this->assertEquals('\TestApp\Model\Table\I18nTable', $i18n->getName());
         $this->assertInstanceOf('TestApp\Model\Table\I18nTable', $i18n->getTarget());
-        $this->assertEquals('test_custom_i18n_datasource', $i18n->getTarget()->connection()->configName());
+        $this->assertEquals('test_custom_i18n_datasource', $i18n->getTarget()->getConnection()->configName());
         $this->assertEquals('custom_i18n_table', $i18n->getTarget()->getTable());
     }
 
@@ -1111,7 +1111,7 @@ class TranslateBehaviorTest extends TestCase
     {
         $table = TableRegistry::get('Articles');
         $table->addBehavior('Translate', ['fields' => ['title', 'body']]);
-        $table->entityClass(__NAMESPACE__ . '\Article');
+        $table->setEntityClass(__NAMESPACE__ . '\Article');
         I18n::setLocale('fra');
         $translations = [
             'fra' => ['title' => 'Un article'],
@@ -1188,7 +1188,7 @@ class TranslateBehaviorTest extends TestCase
     public function testChangingReferenceName()
     {
         $table = TableRegistry::get('Articles');
-        $table->alias('FavoritePost');
+        $table->setAlias('FavoritePost');
         $table->addBehavior(
             'Translate',
             ['fields' => ['body'], 'referenceName' => 'Posts']
@@ -1289,7 +1289,7 @@ class TranslateBehaviorTest extends TestCase
     {
         $table = TableRegistry::get('Articles');
         $table->addBehavior('Translate', ['fields' => ['title']]);
-        $table->entityClass(__NAMESPACE__ . '\Article');
+        $table->setEntityClass(__NAMESPACE__ . '\Article');
         I18n::setLocale('fra');
         $article = $table->get(1);
         $article->set('body', 'New Body');
@@ -1311,7 +1311,7 @@ class TranslateBehaviorTest extends TestCase
             'fields' => ['title'],
             'validator' => (new \Cake\Validation\Validator)->add('title', 'notBlank', ['rule' => 'notBlank'])
         ]);
-        $table->entityClass(__NAMESPACE__ . '\Article');
+        $table->setEntityClass(__NAMESPACE__ . '\Article');
 
         $data = [
             'author_id' => 1,
@@ -1395,7 +1395,7 @@ class TranslateBehaviorTest extends TestCase
     {
         $table = TableRegistry::get('Articles');
         $table->addBehavior('Translate', ['fields' => ['title', 'body']]);
-        $table->entityClass(__NAMESPACE__ . '\Article');
+        $table->setEntityClass(__NAMESPACE__ . '\Article');
 
         $data = [
             '_translations' => [
@@ -1428,7 +1428,7 @@ class TranslateBehaviorTest extends TestCase
     {
         $table = TableRegistry::get('Articles');
         $table->addBehavior('Translate', ['fields' => ['title', 'body']]);
-        $table->entityClass(__NAMESPACE__ . '\Article');
+        $table->setEntityClass(__NAMESPACE__ . '\Article');
 
         $data = [
             'author_id' => 1,

tests/TestCase/ORM/Behavior/TreeBehaviorTest.php

@@ -38,7 +38,7 @@ class TreeBehaviorTest extends TestCase
     {
         parent::setUp();
         $this->table = TableRegistry::get('NumberTrees');
-        $this->table->primaryKey(['id']);
+        $this->table->setPrimaryKey(['id']);
         $this->table->addBehavior('Tree');
     }
 
@@ -1378,7 +1378,7 @@ class TreeBehaviorTest extends TestCase
     {
         $table = $this->table;
         $other = TableRegistry::get('FriendlyTrees', [
-            'table' => $table->table()
+            'table' => $table->getTable()
         ]);
         $table->hasOne('FriendlyTrees', [
             'foreignKey' => 'id'
@@ -1487,11 +1487,11 @@ class TreeBehaviorTest extends TestCase
     public function assertMpttValues($expected, $table, $query = null)
     {
         $query = $query ?: $table->find();
-        $primaryKey = $table->primaryKey();
+        $primaryKey = $table->getPrimaryKey();
         if (is_array($primaryKey)) {
             $primaryKey = $primaryKey[0];
         }
-        $displayField = $table->displayField();
+        $displayField = $table->getDisplayField();
 
         $options = [
             'valuePath' => function ($item, $key, $iterator) use ($primaryKey, $displayField) {

tests/TestCase/ORM/CompositeKeysTest.php

@@ -498,7 +498,7 @@ class CompositeKeyTest extends TestCase
     public function testOneGenerateBelongsToManyEntitiesFromIds()
     {
         $articles = TableRegistry::get('SiteArticles');
-        $articles->entityClass(__NAMESPACE__ . '\OpenArticleEntity');
+        $articles->setEntityClass(__NAMESPACE__ . '\OpenArticleEntity');
         $tags = TableRegistry::get('SiteTags');
         $junction = TableRegistry::get('SiteArticlesTags');
         $articles->belongsToMany('SiteTags', [
@@ -543,7 +543,7 @@ class CompositeKeyTest extends TestCase
             'table' => 'site_authors',
             'connection' => $this->connection,
         ]);
-        $table->displayField('name');
+        $table->setDisplayField('name');
         $query = $table->find('list')
             ->enableHydration(false)
             ->order('id');
@@ -555,7 +555,7 @@ class CompositeKeyTest extends TestCase
         ];
         $this->assertEquals($expected, $query->toArray());
 
-        $table->displayField(['name', 'site_id']);
+        $table->setDisplayField(['name', 'site_id']);
         $query = $table->find('list')
             ->enableHydration(false)
             ->order('id');

tests/TestCase/ORM/Locator/TableLocatorTest.php

@@ -183,11 +183,11 @@ class TableLocatorTest extends TestCase
             'table' => 'my_articles',
         ]);
         $this->assertInstanceOf('Cake\ORM\Table', $result);
-        $this->assertEquals('my_articles', $result->table());
+        $this->assertEquals('my_articles', $result->getTable());
 
         $result2 = $this->_locator->get('Articles');
         $this->assertSame($result, $result2);
-        $this->assertEquals('my_articles', $result->table());
+        $this->assertEquals('my_articles', $result->getTable());
 
         $this->assertSame($this->_locator, $result->associations()->getTableLocator());
     }
@@ -201,33 +201,33 @@ class TableLocatorTest extends TestCase
     {
         $result = $this->_locator->get('Droids');
         $this->assertInstanceOf('Cake\ORM\Table', $result);
-        $this->assertEquals('droids', $result->table());
-        $this->assertEquals('Droids', $result->alias());
+        $this->assertEquals('droids', $result->getTable());
+        $this->assertEquals('Droids', $result->getAlias());
 
         $result = $this->_locator->get('R2D2', ['className' => 'Droids']);
         $this->assertInstanceOf('Cake\ORM\Table', $result);
-        $this->assertEquals('droids', $result->table(), 'The table should be derived from the className');
-        $this->assertEquals('R2D2', $result->alias());
+        $this->assertEquals('droids', $result->getTable(), 'The table should be derived from the className');
+        $this->assertEquals('R2D2', $result->getAlias());
 
         $result = $this->_locator->get('C3P0', ['className' => 'Droids', 'table' => 'rebels']);
         $this->assertInstanceOf('Cake\ORM\Table', $result);
-        $this->assertEquals('rebels', $result->table(), 'The table should be taken from options');
-        $this->assertEquals('C3P0', $result->alias());
+        $this->assertEquals('rebels', $result->getTable(), 'The table should be taken from options');
+        $this->assertEquals('C3P0', $result->getAlias());
 
         $result = $this->_locator->get('Funky.Chipmunks');
         $this->assertInstanceOf('Cake\ORM\Table', $result);
-        $this->assertEquals('chipmunks', $result->table(), 'The table should be derived from the alias');
-        $this->assertEquals('Chipmunks', $result->alias());
+        $this->assertEquals('chipmunks', $result->getTable(), 'The table should be derived from the alias');
+        $this->assertEquals('Chipmunks', $result->getAlias());
 
         $result = $this->_locator->get('Awesome', ['className' => 'Funky.Monkies']);
         $this->assertInstanceOf('Cake\ORM\Table', $result);
-        $this->assertEquals('monkies', $result->table(), 'The table should be derived from the classname');
-        $this->assertEquals('Awesome', $result->alias());
+        $this->assertEquals('monkies', $result->getTable(), 'The table should be derived from the classname');
+        $this->assertEquals('Awesome', $result->getAlias());
 
         $result = $this->_locator->get('Stuff', ['className' => 'Cake\ORM\Table']);
         $this->assertInstanceOf('Cake\ORM\Table', $result);
-        $this->assertEquals('stuff', $result->table(), 'The table should be derived from the alias');
-        $this->assertEquals('Stuff', $result->alias());
+        $this->assertEquals('stuff', $result->getTable(), 'The table should be derived from the alias');
+        $this->assertEquals('Stuff', $result->getAlias());
     }
 
     /**
@@ -241,7 +241,7 @@ class TableLocatorTest extends TestCase
             'table' => 'my_articles',
         ]);
         $result = $this->_locator->get('Articles');
-        $this->assertEquals('my_articles', $result->table(), 'Should use getConfig() data.');
+        $this->assertEquals('my_articles', $result->getTable(), 'Should use getConfig() data.');
     }
 
     /**
@@ -255,8 +255,8 @@ class TableLocatorTest extends TestCase
         $result = $this->_locator->get('Articles', [
             'connectionName' => 'testing'
         ]);
-        $this->assertEquals('articles', $result->table());
-        $this->assertEquals('test', $result->connection()->configName());
+        $this->assertEquals('articles', $result->getTable());
+        $this->assertEquals('test', $result->getConnection()->configName());
     }
 
     /**
@@ -434,11 +434,11 @@ class TableLocatorTest extends TestCase
 
         $table = $this->_locator->get('users', ['table' => 'users']);
         $this->assertInstanceOf('Cake\ORM\Table', $table);
-        $this->assertEquals('users', $table->table());
-        $this->assertEquals('users', $table->alias());
-        $this->assertSame($connection, $table->connection());
-        $this->assertEquals(array_keys($schema), $table->schema()->columns());
-        $this->assertEquals($schema['id']['type'], $table->schema()->getColumnType('id'));
+        $this->assertEquals('users', $table->getTable());
+        $this->assertEquals('users', $table->getAlias());
+        $this->assertSame($connection, $table->getConnection());
+        $this->assertEquals(array_keys($schema), $table->getSchema()->columns());
+        $this->assertEquals($schema['id']['type'], $table->getSchema()->getColumnType('id'));
 
         $this->_locator->clear();
         $this->assertEmpty($this->_locator->getConfig());
@@ -446,11 +446,11 @@ class TableLocatorTest extends TestCase
         $this->_locator->setConfig('users', $options);
         $table = $this->_locator->get('users', ['className' => __NAMESPACE__ . '\MyUsersTable']);
         $this->assertInstanceOf(__NAMESPACE__ . '\MyUsersTable', $table);
-        $this->assertEquals('users', $table->table());
-        $this->assertEquals('users', $table->alias());
-        $this->assertSame($connection, $table->connection());
-        $this->assertEquals(array_keys($schema), $table->schema()->columns());
-        $this->assertEquals($schema['id']['type'], $table->schema()->getColumnType('id'));
+        $this->assertEquals('users', $table->getTable());
+        $this->assertEquals('users', $table->getAlias());
+        $this->assertSame($connection, $table->getConnection());
+        $this->assertEquals(array_keys($schema), $table->getSchema()->columns());
+        $this->assertEquals($schema['id']['type'], $table->getSchema()->getColumnType('id'));
     }
 
     /**

tests/TestCase/ORM/MarshallerTest.php

@@ -71,8 +71,8 @@ class GreedyCommentsTable extends Table
      */
     public function initialize(array $config)
     {
-        $this->table('comments');
-        $this->alias('Comments');
+        $this->setTable('comments');
+        $this->setAlias('Comments');
     }
 
     /**
@@ -156,11 +156,11 @@ class MarshallerTest extends TestCase
         $this->comments->belongsTo('Articles');
         $this->comments->belongsTo('Users');
 
-        $this->articles->entityClass(__NAMESPACE__ . '\OpenEntity');
-        $this->comments->entityClass(__NAMESPACE__ . '\OpenEntity');
-        $this->users->entityClass(__NAMESPACE__ . '\OpenEntity');
-        $this->tags->entityClass(__NAMESPACE__ . '\OpenEntity');
-        $this->articleTags->entityClass(__NAMESPACE__ . '\OpenEntity');
+        $this->articles->setEntityClass(__NAMESPACE__ . '\OpenEntity');
+        $this->comments->setEntityClass(__NAMESPACE__ . '\OpenEntity');
+        $this->users->setEntityClass(__NAMESPACE__ . '\OpenEntity');
+        $this->tags->setEntityClass(__NAMESPACE__ . '\OpenEntity');
+        $this->articleTags->setEntityClass(__NAMESPACE__ . '\OpenEntity');
     }
 
     /**
@@ -283,7 +283,7 @@ class MarshallerTest extends TestCase
             'author_id' => 'derp',
             'created' => 'fale'
         ];
-        $this->articles->entityClass(__NAMESPACE__ . '\OpenEntity');
+        $this->articles->setEntityClass(__NAMESPACE__ . '\OpenEntity');
         $marshall = new Marshaller($this->articles);
         $result = $marshall->one($data, []);
 
@@ -305,7 +305,7 @@ class MarshallerTest extends TestCase
             'author_id' => 1,
             'not_in_schema' => true
         ];
-        $this->articles->entityClass(__NAMESPACE__ . '\ProtectedArticle');
+        $this->articles->setEntityClass(__NAMESPACE__ . '\ProtectedArticle');
         $marshall = new Marshaller($this->articles);
         $result = $marshall->one($data, []);
 
@@ -327,7 +327,7 @@ class MarshallerTest extends TestCase
             'author_id' => 1,
             'not_in_schema' => true
         ];
-        $this->articles->entityClass(__NAMESPACE__ . '\ProtectedArticle');
+        $this->articles->setEntityClass(__NAMESPACE__ . '\ProtectedArticle');
 
         $marshall = new Marshaller($this->articles);
 
@@ -423,8 +423,8 @@ class MarshallerTest extends TestCase
                 'username' => 'mark',
             ]
         ];
-        $this->articles->entityClass(__NAMESPACE__ . '\ProtectedArticle');
-        $this->users->entityClass(__NAMESPACE__ . '\ProtectedArticle');
+        $this->articles->setEntityClass(__NAMESPACE__ . '\ProtectedArticle');
+        $this->users->setEntityClass(__NAMESPACE__ . '\ProtectedArticle');
 
         $marshall = new Marshaller($this->articles);
 
@@ -829,7 +829,7 @@ class MarshallerTest extends TestCase
      */
     public function testBelongsToManyAddingNewExisting()
     {
-        $this->tags->entityClass(__NAMESPACE__ . '\Tag');
+        $this->tags->setEntityClass(__NAMESPACE__ . '\Tag');
         $data = [
             'title' => 'My title',
             'body' => 'My content',
@@ -3360,8 +3360,8 @@ class MarshallerTest extends TestCase
         ];
 
         $articles = TableRegistry::get('Articles');
-        $articles->schema()->dropConstraint('primary');
-        $articles->primaryKey('id');
+        $articles->getSchema()->dropConstraint('primary');
+        $articles->setPrimaryKey('id');
 
         $marshall = new Marshaller($articles);
         $result = $marshall->one($data);
@@ -3390,8 +3390,8 @@ class MarshallerTest extends TestCase
         ];
 
         $tags = TableRegistry::get('Tags');
-        $tags->schema()->dropConstraint('primary');
-        $tags->primaryKey('id');
+        $tags->getSchema()->dropConstraint('primary');
+        $tags->setPrimaryKey('id');
 
         $marshall = new Marshaller($this->articles);
         $result = $marshall->one($data, ['associated' => ['Tags']]);

tests/TestCase/ORM/QueryRegressionTest.php

@@ -667,7 +667,7 @@ class QueryRegressionTest extends TestCase
         $tags = TableRegistry::get('Tags');
 
         $this->skipIf(
-            $tags->connection()->getDriver() instanceof \Cake\Database\Driver\Sqlserver,
+            $tags->getConnection()->getDriver() instanceof \Cake\Database\Driver\Sqlserver,
             'SQL server is temporarily weird in this test, will investigate later'
         );
         $tags = TableRegistry::get('Tags');

tests/TestCase/ORM/ResultSetTest.php

@@ -141,7 +141,7 @@ class ResultSetTest extends TestCase
         foreach ($results as $i => $row) {
             $expected = new Entity($this->fixtureData[$i]);
             $expected->isNew(false);
-            $expected->source($this->table->alias());
+            $expected->source($this->table->getAlias());
             $expected->clean();
             $this->assertEquals($expected, $row, "Row $i does not match");
         }
@@ -235,7 +235,7 @@ class ResultSetTest extends TestCase
         $options = [
             'markNew' => false,
             'markClean' => true,
-            'source' => $this->table->alias()
+            'source' => $this->table->getAlias()
         ];
         $expected = [
             1 => [

tests/TestCase/ORM/TableRegressionTest.php

@@ -58,7 +58,7 @@ class TableRegressionTest extends TestCase
         $table->getEventManager()->on(
             'Model.afterSave',
             function () use ($table) {
-                $table->connection()->rollback();
+                $table->getConnection()->rollback();
             }
         );
         $entity = $table->newEntity(['name' => 'Jon']);