Problem/Motivation

  • Refactor a BDD test scenario to delete a created user to clean up

Issue fork cloud-3314804

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yas created an issue. See original summary.

yas opened merge request !1366

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas as a volunteer and at DOCOMO Innovations, Inc. commented
Status: Needs work » Needs review
Related issues: +#3313552: Refactor to reassign UID for Cloud Config, Cloud Project and Cloud Store

@baldwinlouie @kumikoono

Can you please review the patch? Thanks

baldwinlouie’s picture

baldwinlouie CreditAttribution: baldwinlouie commented

@yas, thank you for providing the patch. It looks good to me.

kumikoono’s picture

kumikoono CreditAttribution: kumikoono at DOCOMO Innovations, Inc. commented

Looks good to me. Thanks!

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas as a volunteer and at DOCOMO Innovations, Inc. commented
Status: Needs review » Reviewed & tested by the community

@baldwinlouie @kumikoono

Thank you for your review. I'll merge the patch to 4.x and 5.x, and close this issue as Fixed.

  • yas committed 3d2c348 on 5.x 
    Issue #3314804 by yas, baldwinlouie, kumikoono: Refactor a BDD test...

  • yas committed caab329 on 4.x 
    Issue #3314804 by yas, baldwinlouie, kumikoono: Refactor a BDD test...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas as a volunteer and at DOCOMO Innovations, Inc. commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.