Completed
Push — master ( 83aeb0...5ccf93 )
by Jonathan
02:18
created

RawDrupalContext::login()   B

Complexity

Conditions 6
Paths 10

Size

Total Lines 31
Code Lines 18

Duplication

Lines 0
Ratio 0 %

Importance

Changes 2
Bugs 0 Features 0
Metric Value
c 2
b 0
f 0
dl 0
loc 31
rs 8.439
cc 6
eloc 18
nc 10
nop 0
1
<?php
2
3
namespace Drupal\DrupalExtension\Context;
4
5
use Behat\MinkExtension\Context\RawMinkContext;
6
use Behat\Mink\Exception\DriverException;
7
use Behat\Testwork\Hook\HookDispatcher;
8
9
use Drupal\DrupalDriverManager;
10
11
use Drupal\DrupalExtension\Hook\Scope\AfterLanguageEnableScope;
12
use Drupal\DrupalExtension\Hook\Scope\AfterNodeCreateScope;
13
use Drupal\DrupalExtension\Hook\Scope\AfterTermCreateScope;
14
use Drupal\DrupalExtension\Hook\Scope\AfterUserCreateScope;
15
use Drupal\DrupalExtension\Hook\Scope\BaseEntityScope;
16
use Drupal\DrupalExtension\Hook\Scope\BeforeLanguageEnableScope;
17
use Drupal\DrupalExtension\Hook\Scope\BeforeNodeCreateScope;
18
use Drupal\DrupalExtension\Hook\Scope\BeforeUserCreateScope;
19
use Drupal\DrupalExtension\Hook\Scope\BeforeTermCreateScope;
20
21
22
/**
23
 * Provides the raw functionality for interacting with Drupal.
24
 */
25
class RawDrupalContext extends RawMinkContext implements DrupalAwareInterface {
26
27
  /**
28
   * Drupal driver manager.
29
   *
30
   * @var \Drupal\DrupalDriverManager
31
   */
32
  private $drupal;
33
34
  /**
35
   * Test parameters.
36
   *
37
   * @var array
38
   */
39
  private $drupalParameters;
40
41
  /**
42
   * Event dispatcher object.
43
   *
44
   * @var \Behat\Testwork\Hook\HookDispatcher
45
   */
46
  protected $dispatcher;
47
48
  /**
49
   * Keep track of nodes so they can be cleaned up.
50
   *
51
   * @var array
52
   */
53
  protected $nodes = array();
54
55
  /**
56
   * Current authenticated user.
57
   *
58
   * A value of FALSE denotes an anonymous user.
59
   *
60
   * @var stdClass|bool
61
   */
62
  public $user = FALSE;
63
64
  /**
65
   * Keep track of all users that are created so they can easily be removed.
66
   *
67
   * @var array
68
   */
69
  protected $users = array();
70
71
  /**
72
   * Keep track of all terms that are created so they can easily be removed.
73
   *
74
   * @var array
75
   */
76
  protected $terms = array();
77
78
  /**
79
   * Keep track of any roles that are created so they can easily be removed.
80
   *
81
   * @var array
82
   */
83
  protected $roles = array();
84
85
  /**
86
   * Keep track of any languages that are created so they can easily be removed.
87
   *
88
   * @var array
89
   */
90
  protected $languages = array();
91
92
  /**
93
   * {@inheritDoc}
94
   */
95
  public function setDrupal(DrupalDriverManager $drupal) {
96
    $this->drupal = $drupal;
97
  }
98
99
  /**
100
   * {@inheritDoc}
101
   */
102
  public function getDrupal() {
103
    return $this->drupal;
104
  }
105
106
  /**
107
   * {@inheritDoc}
108
   */
109
  public function setDispatcher(HookDispatcher $dispatcher) {
110
    $this->dispatcher = $dispatcher;
111
  }
112
113
  /**
114
   * Set parameters provided for Drupal.
115
   */
116
  public function setDrupalParameters(array $parameters) {
117
    $this->drupalParameters = $parameters;
118
  }
119
120
  /**
121
   * Returns a specific Drupal parameter.
122
   *
123
   * @param string $name
124
   *   Parameter name.
125
   *
126
   * @return mixed
127
   */
128
  public function getDrupalParameter($name) {
129
    return isset($this->drupalParameters[$name]) ? $this->drupalParameters[$name] : NULL;
130
  }
131
132
  /**
133
   * Returns a specific Drupal text value.
134
   *
135
   * @param string $name
136
   *   Text value name, such as 'log_out', which corresponds to the default 'Log
137
   *   out' link text.
138
   * @throws \Exception
139
   * @return
140
   */
141
  public function getDrupalText($name) {
142
    $text = $this->getDrupalParameter('text');
143
    if (!isset($text[$name])) {
144
      throw new \Exception(sprintf('No such Drupal string: %s', $name));
145
    }
146
    return $text[$name];
147
  }
148
149
  /**
150
   * Returns a specific css selector.
151
   *
152
   * @param $name
153
   *   string CSS selector name
154
   */
155
  public function getDrupalSelector($name) {
156
    $text = $this->getDrupalParameter('selectors');
157
    if (!isset($text[$name])) {
158
      throw new \Exception(sprintf('No such selector configured: %s', $name));
159
    }
160
    return $text[$name];
161
  }
162
163
  /**
164
   * Get active Drupal Driver.
165
   *
166
   * @return \Drupal\Driver\DrupalDriver
167
   */
168
  public function getDriver($name = NULL) {
169
    return $this->getDrupal()->getDriver($name);
170
  }
171
172
  /**
173
   * Get driver's random generator.
174
   */
175
  public function getRandom() {
176
    return $this->getDriver()->getRandom();
177
  }
178
179
  /**
180
   * Massage node values to match the expectations on different Drupal versions.
181
   *
182
   * @beforeNodeCreate
183
   */
184
  public function alterNodeParameters(BeforeNodeCreateScope $scope) {
185
    $node = $scope->getEntity();
186
187
    // Get the Drupal API version if available. This is not available when
188
    // using e.g. the BlackBoxDriver or DrushDriver.
189
    $api_version = NULL;
190
    $driver = $scope->getContext()->getDrupal()->getDriver();
0 ignored issues
show
Bug introduced by
It seems like you code against a concrete implementation and not the interface Behat\Behat\Context\Context as the method getDrupal() does only exist in the following implementations of said interface: Drupal\DrupalExtension\Context\ConfigContext, Drupal\DrupalExtension\Context\DrupalContext, Drupal\DrupalExtension\C...xt\DrupalSubContextBase, Drupal\DrupalExtension\Context\DrushContext, Drupal\DrupalExtension\Context\MessageContext, Drupal\DrupalExtension\Context\RawDrupalContext.

Let’s take a look at an example:

interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break.

Available Fixes

  1. Change the type-hint for the parameter:

    class AuthSystem
    {
        public function authenticate(MyUser $user) { /* ... */ }
    }
    
  2. Add an additional type-check:

    class AuthSystem
    {
        public function authenticate(User $user)
        {
            if ($user instanceof MyUser) {
                $this->logger->info(/** ... */);
            }
    
            // or alternatively
            if ( ! $user instanceof MyUser) {
                throw new \LogicException(
                    '$user must be an instance of MyUser, '
                   .'other instances are not supported.'
                );
            }
    
        }
    }
    
Note: PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case.
  1. Add the method to the interface:

    interface User
    {
        /** @return string */
        public function getPassword();
    
        /** @return string */
        public function getDisplayName();
    }
    
Loading history...
191
    if ($driver instanceof \Drupal\Driver\DrupalDriver) {
192
      $api_version = $scope->getContext()->getDrupal()->getDriver()->version;
0 ignored issues
show
Bug introduced by
It seems like you code against a concrete implementation and not the interface Behat\Behat\Context\Context as the method getDrupal() does only exist in the following implementations of said interface: Drupal\DrupalExtension\Context\ConfigContext, Drupal\DrupalExtension\Context\DrupalContext, Drupal\DrupalExtension\C...xt\DrupalSubContextBase, Drupal\DrupalExtension\Context\DrushContext, Drupal\DrupalExtension\Context\MessageContext, Drupal\DrupalExtension\Context\RawDrupalContext.

Let’s take a look at an example:

interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break.

Available Fixes

  1. Change the type-hint for the parameter:

    class AuthSystem
    {
        public function authenticate(MyUser $user) { /* ... */ }
    }
    
  2. Add an additional type-check:

    class AuthSystem
    {
        public function authenticate(User $user)
        {
            if ($user instanceof MyUser) {
                $this->logger->info(/** ... */);
            }
    
            // or alternatively
            if ( ! $user instanceof MyUser) {
                throw new \LogicException(
                    '$user must be an instance of MyUser, '
                   .'other instances are not supported.'
                );
            }
    
        }
    }
    
Note: PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case.
  1. Add the method to the interface:

    interface User
    {
        /** @return string */
        public function getPassword();
    
        /** @return string */
        public function getDisplayName();
    }
    
Loading history...
193
    }
194
195
    // On Drupal 8 the timestamps should be in UNIX time.
196
    switch ($api_version) {
197
      case 8:
198
        foreach (array('changed', 'created', 'revision_timestamp') as $field) {
199
          if (!empty($node->$field) && !is_numeric($node->$field)) {
200
            $node->$field = strtotime($node->$field);
201
          }
202
        }
203
      break;
204
    }
205
  }
206
207
  /**
208
   * Remove any created nodes.
209
   *
210
   * @AfterScenario
211
   */
212
  public function cleanNodes() {
213
    // Remove any nodes that were created.
214
    foreach ($this->nodes as $node) {
215
      $this->getDriver()->nodeDelete($node);
216
    }
217
    $this->nodes = array();
218
  }
219
220
  /**
221
   * Remove any created users.
222
   *
223
   * @AfterScenario
224
   */
225
  public function cleanUsers() {
226
    // Remove any users that were created.
227
    if (!empty($this->users)) {
228
      foreach ($this->users as $user) {
229
        $this->getDriver()->userDelete($user);
230
      }
231
      $this->getDriver()->processBatch();
232
      $this->users = array();
233
    }
234
  }
235
236
  /**
237
   * Remove any created terms.
238
   *
239
   * @AfterScenario
240
   */
241
  public function cleanTerms() {
242
    // Remove any terms that were created.
243
    foreach ($this->terms as $term) {
244
      $this->getDriver()->termDelete($term);
245
    }
246
    $this->terms = array();
247
  }
248
249
  /**
250
   * Remove any created roles.
251
   *
252
   * @AfterScenario
253
   */
254
  public function cleanRoles() {
255
    // Remove any roles that were created.
256
    foreach ($this->roles as $rid) {
257
      $this->getDriver()->roleDelete($rid);
258
    }
259
    $this->roles = array();
260
  }
261
262
  /**
263
   * Remove any created languages.
264
   *
265
   * @AfterScenario
266
   */
267
  public function cleanLanguages() {
268
    // Delete any languages that were created.
269
    foreach ($this->languages as $language) {
270
      $this->getDriver()->languageDelete($language);
271
      unset($this->languages[$language->langcode]);
272
    }
273
  }
274
275
  /**
276
   * Clear static caches.
277
   *
278
   * @AfterScenario @api
279
   */
280
  public function clearStaticCaches() {
281
    $this->getDriver()->clearStaticCaches();
282
  }
283
284
  /**
285
   * Dispatch scope hooks.
286
   *
287
   * @param string $scope
0 ignored issues
show
Bug introduced by
There is no parameter named $scope. Was it maybe removed?

This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function.

Consider the following example. The parameter $italy is not defined by the method finale(...).

/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}

The most likely cause is that the parameter was removed, but the annotation was not.

Loading history...
288
   *   The entity scope to dispatch.
289
   * @param stdClass $entity
290
   *   The entity.
291
   */
292
  protected function dispatchHooks($scopeType, \stdClass $entity) {
293
    $fullScopeClass = 'Drupal\\DrupalExtension\\Hook\\Scope\\' . $scopeType;
294
    $scope = new $fullScopeClass($this->getDrupal()->getEnvironment(), $this, $entity);
295
    $callResults = $this->dispatcher->dispatchScopeHooks($scope);
296
297
    // The dispatcher suppresses exceptions, throw them here if there are any.
298
    foreach ($callResults as $result) {
299
      if ($result->hasException()) {
300
        $exception = $result->getException();
301
        throw $exception;
302
      }
303
    }
304
  }
305
306
  /**
307
   * Create a node.
308
   *
309
   * @return object
310
   *   The created node.
311
   */
312 View Code Duplication
  public function nodeCreate($node) {
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
313
    $this->dispatchHooks('BeforeNodeCreateScope', $node);
314
    $this->parseEntityFields('node', $node);
315
    $saved = $this->getDriver()->createNode($node);
316
    $this->dispatchHooks('AfterNodeCreateScope', $saved);
317
    $this->nodes[] = $saved;
318
    return $saved;
319
  }
320
321
  /**
322
   * Parse multi-value fields. Possible formats:
323
   *    A, B, C
324
   *    A - B, C - D, E - F
325
   *
326
   * @param string $entity_type
327
   *   The entity type.
328
   * @param \stdClass $entity
329
   *   An object containing the entity properties and fields as properties.
330
   */
331
  public function parseEntityFields($entity_type, \stdClass $entity) {
332
    $multicolumn_field = '';
333
    $multicolumn_fields = array();
334
335
    foreach (clone $entity as $field => $field_value) {
0 ignored issues
show
Bug introduced by
The expression clone $entity of type object<stdClass> is not traversable.
Loading history...
336
      // Reset the multicolumn field if the field name does not contain a column.
337
      if (strpos($field, ':') === FALSE) {
338
        $multicolumn_field = '';
339
      }
340
      // Start tracking a new multicolumn field if the field name contains a ':'
341
      // which is preceded by at least 1 character.
342
      elseif (strpos($field, ':', 1) !== FALSE) {
343
        list($multicolumn_field, $multicolumn_column) = explode(':', $field);
344
      }
345
      // If a field name starts with a ':' but we are not yet tracking a
346
      // multicolumn field we don't know to which field this belongs.
347
      elseif (empty($multicolumn_field)) {
348
        throw new \Exception('Field name missing for ' . $field);
349
      }
350
      // Update the column name if the field name starts with a ':' and we are
351
      // already tracking a multicolumn field.
352
      else {
353
        $multicolumn_column = substr($field, 1);
354
      }
355
356
      $is_multicolumn = $multicolumn_field && $multicolumn_column;
0 ignored issues
show
Bug introduced by
The variable $multicolumn_column does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
357
      $field_name = $multicolumn_field ?: $field;
358
      if ($this->getDriver()->isField($entity_type, $field_name)) {
359
        // Split up multiple values in multi-value fields.
360
        $values = array();
361
        foreach (explode(', ', $field_value) as $key => $value) {
362
          $columns = $value;
363
          // Split up field columns if the ' - ' separator is present.
364
          if (strstr($value, ' - ') !== FALSE) {
365
            $columns = array();
366
            foreach (explode(' - ', $value) as $column) {
367
              // Check if it is an inline named column.
368
              if (!$is_multicolumn && strpos($column, ': ', 1) !== FALSE) {
369
                list ($key, $column) = explode(': ', $column);
370
                $columns[$key] = $column;
371
              }
372
              else {
373
                $columns[] = $column;
374
              }
375
            }
376
          }
377
          // Use the column name if we are tracking a multicolumn field.
378
          if ($is_multicolumn) {
379
            $multicolumn_fields[$multicolumn_field][$key][$multicolumn_column] = $columns;
380
            unset($entity->$field);
381
          }
382
          else {
383
            $values[] = $columns;
384
          }
385
        }
386
        // Replace regular fields inline in the entity after parsing.
387
        if (!$is_multicolumn) {
388
          $entity->$field_name = $values;
389
        }
390
      }
391
    }
392
393
    // Add the multicolumn fields to the entity.
394
    foreach ($multicolumn_fields as $field_name => $columns) {
395
      $entity->$field_name = $columns;
396
    }
397
  }
398
399
  /**
400
   * Create a user.
401
   *
402
   * @return object
403
   *   The created user.
404
   */
405 View Code Duplication
  public function userCreate($user) {
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
406
    $this->dispatchHooks('BeforeUserCreateScope', $user);
407
    $this->parseEntityFields('user', $user);
408
    $this->getDriver()->userCreate($user);
409
    $this->dispatchHooks('AfterUserCreateScope', $user);
410
    $this->users[$user->name] = $this->user = $user;
411
    return $user;
412
  }
413
414
  /**
415
   * Create a term.
416
   *
417
   * @return object
418
   *   The created term.
419
   */
420 View Code Duplication
  public function termCreate($term) {
0 ignored issues
show
Duplication introduced by
This method seems to be duplicated in your project.

Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation.

You can also find more detailed suggestions in the “Code” section of your repository.

Loading history...
421
    $this->dispatchHooks('BeforeTermCreateScope', $term);
422
    $this->parseEntityFields('taxonomy_term', $term);
423
    $saved = $this->getDriver()->createTerm($term);
424
    $this->dispatchHooks('AfterTermCreateScope', $saved);
425
    $this->terms[] = $saved;
426
    return $saved;
427
  }
428
429
  /**
430
   * Creates a language.
431
   *
432
   * @param \stdClass $language
433
   *   An object with the following properties:
434
   *   - langcode: the langcode of the language to create.
435
   *
436
   * @return object|FALSE
437
   *   The created language, or FALSE if the language was already created.
438
   */
439
  public function languageCreate(\stdClass $language) {
440
    $this->dispatchHooks('BeforeLanguageCreateScope', $language);
441
    $language = $this->getDriver()->languageCreate($language);
442
    if ($language) {
443
      $this->dispatchHooks('AfterLanguageCreateScope', $language);
444
      $this->languages[$language->langcode] = $language;
445
    }
446
    return $language;
447
  }
448
449
  /**
450
   * Log-in the current user.
451
   */
452
  public function login() {
453
    // Check if logged in.
454
    if ($this->loggedIn()) {
455
      $this->logout();
456
    }
457
458
    if (!$this->user) {
459
      throw new \Exception('Tried to login without a user.');
460
    }
461
462
    $this->getSession()->visit($this->locatePath('/user'));
463
    $element = $this->getSession()->getPage();
464
    $element->fillField($this->getDrupalText('username_field'), $this->user->name);
465
    $element->fillField($this->getDrupalText('password_field'), $this->user->pass);
466
    $submit = $element->findButton($this->getDrupalText('log_in'));
467
    if (empty($submit)) {
468
      throw new \Exception(sprintf("No submit button at %s", $this->getSession()->getCurrentUrl()));
469
    }
470
471
    // Log in.
472
    $submit->click();
473
474
    if (!$this->loggedIn()) {
475
      if (isset($this->user->role)) {
476
        throw new \Exception(sprintf("Unable to determine if logged in because 'log_out' link cannot be found for user '%s' with role '%s'", $this->user->name, $this->user->role));
477
      }
478
      else {
479
        throw new \Exception(sprintf("Unable to determine if logged in because 'log_out' link cannot be found for user '%s'", $this->user->name));
480
      }
481
    }
482
  }
483
484
  /**
485
   * Logs the current user out.
486
   */
487
  public function logout() {
488
    $this->getSession()->visit($this->locatePath('/user/logout'));
489
  }
490
491
  /**
492
   * Determine if the a user is already logged in.
493
   *
494
   * @return boolean
495
   *   Returns TRUE if a user is logged in for this session.
496
   */
497
  public function loggedIn() {
498
    $session = $this->getSession();
499
    $page = $session->getPage();
500
501
    // Look for a css selector to determine if a user is logged in.
502
    // Default is the logged-in class on the body tag.
503
    // Which should work with almost any theme.
504
    try {
505
      if ($page->has('css', $this->getDrupalSelector('logged_in_selector'))) {
506
        return TRUE;
507
      }
508
    } catch (DriverException $e) {
509
      // This test may fail if the driver did not load any site yet.
510
    }
511
512
    // Some themes do not add that class to the body, so lets check if the
513
    // login form is displayed on /user/login.
514
    $session->visit($this->locatePath('/user/login'));
515
    if (!$page->has('css', $this->getDrupalSelector('login_form_selector'))) {
516
      return TRUE;
517
    }
518
519
    $session->visit($this->locatePath('/'));
520
521
    // As a last resort, if a logout link is found, we are logged in. While not
522
    // perfect, this is how Drupal SimpleTests currently work as well.
523
    return $page->findLink($this->getDrupalText('log_out'));
524
  }
525
526
  /**
527
   * User with a given role is already logged in.
528
   *
529
   * @param string $role
530
   *   A single role, or multiple comma-separated roles in a single string.
531
   *
532
   * @return boolean
533
   *   Returns TRUE if the current logged in user has this role (or roles).
534
   */
535
  public function loggedInWithRole($role) {
536
    return $this->loggedIn() && $this->user && isset($this->user->role) && $this->user->role == $role;
537
  }
538
539
}
540