andmemasin\myabstract\traits\ActiveRecordTestTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	testAttributeHintsForExistingAttributesOnly()	0	4	2
A	testRulesForExistingAttributesOnly()	0	5	3
A	testAttributeLabelsForExistingAttributesOnly()	0	4	2

<?php

namespace andmemasin\myabstract\traits;
use yii\db\ActiveRecord;

/**
 * Trait ActiveRecordTestTrait
 * @package andmemasin\myabstract\traits
 * @author Tõnis Ormisson <tonis@andmemasin.eu>
 * @property ActiveRecord $model
 */
trait ActiveRecordTestTrait
{
    public function testAttributeLabelsForExistingAttributesOnly() {
        // labels only for actually existing attributes
        foreach ($this->model->attributeLabels() as $key => $label) {
            $this->assertArrayHasKey($key, $this->model->attributes);

It seems like assertArrayHasKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }
    public function testAttributeHintsForExistingAttributesOnly() {
        // labels only for actually existing attributes
        foreach ($this->model->attributeHints() as $key => $label) {
            $this->assertArrayHasKey($key, $this->model->attributes);
        }
    }
    public function testRulesForExistingAttributesOnly() {
        // labels only for actually existing attributes
        foreach ($this->model->rules() as $rule) {
            foreach ($rule[0] as $attribute) {
                $this->assertArrayHasKey($attribute, $this->model->attributes);
            }
        }
    }

}