bluetree-service/event

View on GitHub
test/EventDispatcherTest.php

Summary

Maintainability
C
1 day
Test Coverage

File EventDispatcherTest.php has 406 lines of code (exceeds 250 allowed). Consider refactoring.
Open

<?php

/**
 * test Event Dispatcher class
 *
Severity: Minor
Found in test/EventDispatcherTest.php - About 5 hrs to fix

    The class EventDispatcherTest has 22 public methods. Consider refactoring EventDispatcherTest to keep number of public methods under 10.
    Open

    class EventDispatcherTest extends TestCase
    {
        /**
         * name of test event log file
         */
    Severity: Minor
    Found in test/EventDispatcherTest.php by phpmd

    TooManyPublicMethods

    Since: 0.1

    A class with too many public methods is probably a good suspect for refactoring, in order to reduce its complexity and find a way to have more fine grained objects.

    By default it ignores methods starting with 'get' or 'set'.

    Example

    Source https://phpmd.org/rules/codesize.html#toomanypublicmethods

    EventDispatcherTest has 26 functions (exceeds 20 allowed). Consider refactoring.
    Open

    class EventDispatcherTest extends TestCase
    {
        /**
         * name of test event log file
         */
    Severity: Minor
    Found in test/EventDispatcherTest.php - About 3 hrs to fix

      Method testSetEventDispatcherConfiguration has 45 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

          public function testSetEventDispatcherConfiguration($options)
          {
              $eventDispatcher = new EventDispatcher;
              $eventDispatcher->setEventConfiguration($options['events']);
      
      
      Severity: Minor
      Found in test/EventDispatcherTest.php - About 1 hr to fix

        Method testTriggerMultipleEvents has 31 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

            public function testTriggerMultipleEvents()
            {
                $testData = [];
                $instance = new EventDispatcher(['events' => [
                    'test_event' => [
        Severity: Minor
        Found in test/EventDispatcherTest.php - About 1 hr to fix

          Missing class import via use statement (line '525', column '19').
          Open

                  throw new \Exception('Test error');
          Severity: Minor
          Found in test/EventDispatcherTest.php by phpmd

          MissingImport

          Since: 2.7.0

          Importing all external classes in a file through use statements makes them clearly visible.

          Example

          function make() {
              return new \stdClass();
          }

          Source http://phpmd.org/rules/cleancode.html#MissingImport

          Missing class import via use statement (line '392', column '40').
          Open

                      'log_object'        => new \SimpleLog\Log,
          Severity: Minor
          Found in test/EventDispatcherTest.php by phpmd

          MissingImport

          Since: 2.7.0

          Importing all external classes in a file through use statements makes them clearly visible.

          Example

          function make() {
              return new \stdClass();
          }

          Source http://phpmd.org/rules/cleancode.html#MissingImport

          There are no issues that match your filters.

          Category
          Status