PHPUnit Mocking функции, которые используют соединение с базой данных - PullRequest
Новая
       3

PHPUnit Mocking функции, которые используют соединение с базой данных

0 голосов
/ 02 ноября 2018

Я новичок в использовании PHPUnit и пытаюсь протестировать функцию getAllTasks (), которая должна извлекать все задачи из базы данных. Я попробовал все, но я только ухудшаю свой код. Поэтому, пожалуйста, помогите мне решить проблему. TaskTest.php это то, что я пытался сделать тест, но он не работает. И конечно, если есть лучшие способы сделать что-то, я тоже люблю изучать новые вещи. Вот мой код:

РЕДАКТИРОВАТЬ: я изменил код для TaskTest.php, и мне удалось получить тестовый проход. Может кто-нибудь сказать мне, если это хороший способ проверить эту функцию, или есть лучшие способы? Спасибо!

Task.php 

<?php

  require_once 'Database.php';

  class Task {
    private $db;

    public function __construct() {
      $this->db = new Database;
    }

    public function getAllTasks() {
      $this->db->query('SELECT * FROM tasks');
      $results = $this->db->resultSet();

      return $results;
    }
}

database.php 

<?php
  class Database {

    private $host = 'localhost';
    private $user = 'root';
    private $pass = '123456';
    private $dbname = 'todolist';

    private $dbh;
    private $stmt;
    private $error;

    public function __construct(){
      // Set DSN
      $dsn = 'mysql:host=' . $this->host . ';dbname=' . $this->dbname;
      $options = array(
        PDO::ATTR_PERSISTENT => true,
        PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION
      );

        // Create PDO instance
      try {
        $this->dbh = new PDO($dsn, $this->user, $this->pass, $options);
      } catch(PDOException $e){
        $this->error = $e->getMessage();
        echo $this->error;
      }
    }

public function query($sql){
      $this->stmt = $this->dbh->prepare($sql);
      $this->execute();
    }

public function execute(){
      return $this->stmt->execute();
    }

    public function resultSet(){
      $this->execute();
      return $this->stmt->fetchAll(PDO::FETCH_ASSOC);

    }
}

TaskTest.php 

<?php
require_once './src/Task.php';
require_once './src/Database.php';
use PHPUnit\Framework\TestCase;

  class TaskTest extends TestCase {

    public function testGetAllTasks() {

      $table = array(
        array(
          'task_id' => '1',
          'task_desc' => 'Task One Test'
        ),
        array(
          'task_id' => '2',
          'task_desc' => 'Task Two Test'
        )
      );

      $dbase = $this->getMockBuilder('Database')
        ->getMock();

      $dbase->method('resultSet')
          ->will($this->returnValue($table));

      $expectedResult = [
                          'task_id' => '1',
                          'task_desc' => 'Task One Test',
                        ];

      $task = new Task();
      $actualResult =  $task->getAllTasks();

      $this->assertEquals($expectedResult, $actualResult[0]);

    }
}

1 Ответ

0 голосов
/ 02 ноября 2018

Вы передаете макет конструктору класса Task, но он ничего с этим не делает.

$task = new Task($resultSetMock);

Обновлен код, так что он будет использоваться: 

class Task {
    private $db;

    public function __construct( ?Database $db = null ) {
        // set the db if none is provided
        if( is_null($db) )
        {
            $db = new Database;
        }

        $this->db = $db;
    }

    // ...
}
...