tests/Gui/SettingsTest.php
Static call to undeclared method \Gui_SettingsTest::markTestSkipped
Open
Open
static::markTestSkipped('unfinished');
- Exclude checks
Possibly zero references to use statement for classlike/namespace WebDriverBy
(\Facebook\WebDriver\WebDriverBy)
Open
Open
use Facebook\WebDriver\WebDriverBy;
- Exclude checks
Static call to undeclared method \Gui_SettingsTest::markTestSkipped
Open
Open
static::markTestSkipped('unfinished');
- Exclude checks
Remove this commented out code. Open
Open
// $this->url('index.php?parent=Settings&module=YetiForce&view=Shop');
- Read upRead up
- Exclude checks
Programmers should not comment out code as it bloats programs and reduces readability.
Unused code should be deleted and can be retrieved from source control history if required.
See
- MISRA C:2004, 2.4 - Sections of code should not be "commented out".
- MISRA C++:2008, 2-7-2 - Sections of code shall not be "commented out" using C-style comments.
- MISRA C++:2008, 2-7-3 - Sections of code should not be "commented out" using C++ comments.
- MISRA C:2012, Dir. 4.4 - Sections of code should not be "commented out"
Remove this commented out code. Open
Open
// $this->url('index.php?parent=Settings&module=Vtiger&view=Index');
- Read upRead up
- Exclude checks
Programmers should not comment out code as it bloats programs and reduces readability.
Unused code should be deleted and can be retrieved from source control history if required.
See
- MISRA C:2004, 2.4 - Sections of code should not be "commented out".
- MISRA C++:2008, 2-7-2 - Sections of code shall not be "commented out" using C-style comments.
- MISRA C++:2008, 2-7-3 - Sections of code should not be "commented out" using C++ comments.
- MISRA C:2012, Dir. 4.4 - Sections of code should not be "commented out"
Each class must be in a namespace of at least one level (a top-level vendor name) Open
Open
final class Gui_SettingsTest extends \Tests\GuiBase
- Exclude checks
The class Gui_SettingsTest is not named in CamelCase. Open
Open
final class Gui_SettingsTest extends \Tests\GuiBase
{
/**
* Testing marketplace view.
*
- Read upRead up
- Exclude checks
CamelCaseClassName
Since: 0.2
It is considered best practice to use the CamelCase notation to name classes.
Example
class class_name {
}
Source
Spaces must be used to indent lines; tabs are not allowed Open
Open
* @return void
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// $this->findError();
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// $this->url('index.php?parent=Settings&module=YetiForce&view=Shop');
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
* Testing marketplace view.
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
/**
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
/**
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// static::assertSame('Vtiger', $this->driver->findElement(WebDriverBy::id('module'))->getAttribute('value'));
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
{
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// static::assertSame('Shop', $this->driver->findElement(WebDriverBy::id('view'))->getAttribute('value'));
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
static::markTestSkipped('unfinished');
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
*
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
*/
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// $this->findError();
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// $this->url('index.php?parent=Settings&module=Vtiger&view=Index');
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// static::assertSame('Index', $this->driver->findElement(WebDriverBy::id('view'))->getAttribute('value'));
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
}
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
* @return void
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
}
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
{
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
public function testDashboard(): void
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
static::markTestSkipped('unfinished');
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
*/
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// static::assertSame('Settings', $this->driver->findElement(WebDriverBy::id('parent'))->getAttribute('value'));
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// static::assertSame('YetiForce', $this->driver->findElement(WebDriverBy::id('module'))->getAttribute('value'));
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
*
- Exclude checks
Line exceeds 120 characters; contains 121 characters Open
Open
// static::assertSame('YetiForce', $this->driver->findElement(WebDriverBy::id('module'))->getAttribute('value'));
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
public function testMarketplace(): void
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
* Testing dashboard view.
- Exclude checks
Spaces must be used to indent lines; tabs are not allowed Open
Open
// static::assertSame('Settings', $this->driver->findElement(WebDriverBy::id('parent'))->getAttribute('value'));
- Exclude checks
Class name "Gui_SettingsTest" is not in camel caps format Open
Open
final class Gui_SettingsTest extends \Tests\GuiBase
- Exclude checks