Zookal_Mock_Test_Config_SystemTest::assertFieldDefined() - Code Metrics - Zookal/magento-mock - Measure and Improve Code Quality continuously with Scrutinizer

assertFieldDefined() A
last analyzed 2017-12-22 02:13 UTC

↳ Parent: Zookal_Mock_Test_Config_SystemTest

Complexity

Conditions	4
Paths	4

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.2
c	0
b	0
f	0
cc	4
eloc	9
nc	4
nop	2

<?php

/**
 * @category    Zookal_Mock
 * @package     Helper
 * @author      Cyrill Schumacher | {firstName}@{lastName}.fm | @SchumacherFM
 * @copyright   Copyright (c) Zookal Pty Ltd
 * @license     OSL - Open Software Licence 3.0 | http://opensource.org/licenses/osl-3.0.php
 */
class Zookal_Mock_Test_Config_SystemTest extends PHPUnit_Framework_TestCase
namespace YourVendor;

class YourClass { }
{
    /**
     * @return string
     */
    protected function getFile()
    {
        $dir = Mage::getModuleDir('etc', 'Zookal_Mock');
        return "$dir/system.xml";
    }

    protected function getXml()
    {
        $file = $this->getFile();
        $xml  = simplexml_load_file($file);
        return $xml;
    }

    /**
     * @param string $path
     */
    public function assertFieldDefined($path, $message = '')
    {
        $defaultMessage = '';
        @list($section, $group, $field) = explode('/', $path);
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
        $nodePath = "sections/$section/groups/$group/fields/$field";
        $result   = $this->getXml()->xpath($nodePath);
        if (!$message) {
            $defaultMessage = sprintf("System configuration field \"$path\" not defined");
        }
        if (!$result) {

            $this->fail($message ? : $defaultMessage);
        }
    }

    /**
     * @test
     */
    public function itShouldHaveASystemXml()
    {
        $this->assertFileExists($this->getFile());
    }

    /**
     * @test
     */
    public function itShouldHaveAnApiUrlField()
    {
        $this->assertFieldDefined('dev/zookalmock/enable_method_log');
    }
}

1			<?php
2
3			/**
4			* @category Zookal_Mock
5			* @package Helper
6			* @author Cyrill Schumacher \| {firstName}@{lastName}.fm \| @SchumacherFM
7			* @copyright Copyright (c) Zookal Pty Ltd
8			* @license OSL - Open Software Licence 3.0 \| http://opensource.org/licenses/osl-3.0.php
9			*/
10			class Zookal_Mock_Test_Config_SystemTest extends PHPUnit_Framework_TestCase
			0 ignored issues – show Coding Style Compatibility introduced 2016-02-18 07:22 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
11			{
12			/**
13			* @return string
14			*/
15			protected function getFile()
16			{
17			$dir = Mage::getModuleDir('etc', 'Zookal_Mock');
18			return "$dir/system.xml";
19			}
20
21			protected function getXml()
22			{
23			$file = $this->getFile();
24			$xml = simplexml_load_file($file);
25			return $xml;
26			}
27
28			/**
29			* @param string $path
30			*/
31			public function assertFieldDefined($path, $message = '')
32			{
33			$defaultMessage = '';
34			@list($section, $group, $field) = explode('/', $path);
			0 ignored issues – show Security Best Practice introduced 2016-02-18 07:22 UTC by Report Bug Copy Issue Report It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended. If you suppress an error, we recommend checking for the error condition explicitly: // For example instead of @mkdir($dir); // Better use if (@mkdir($dir) === false) { throw new \RuntimeException('The directory '.$dir.' could not be created.'); } Loading history...
35			$nodePath = "sections/$section/groups/$group/fields/$field";
36			$result = $this->getXml()->xpath($nodePath);
37			if (!$message) {
38			$defaultMessage = sprintf("System configuration field \"$path\" not defined");
39			}
40			if (!$result) {
			0 ignored issues – show Bug Best Practice introduced 2016-02-18 07:22 UTC by Report Bug Copy Issue Report The expression `$result` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
41			$this->fail($message ? : $defaultMessage);
42			}
43			}
44
45			/**
46			* @test
47			*/
48			public function itShouldHaveASystemXml()
49			{
50			$this->assertFileExists($this->getFile());
51			}
52
53			/**
54			* @test
55			*/
56			public function itShouldHaveAnApiUrlField()
57			{
58			$this->assertFieldDefined('dev/zookalmock/enable_method_log');
59			}
60			}

Zookal / magento-mock

GitHub Access Token became invalid

assertFieldDefined() A last analyzed 2017-12-22 02:13 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

assertFieldDefined() A
last analyzed 2017-12-22 02:13 UTC