Problem/Motivation

Add test cases for Cloud Admin Setting forms:

  • Drupal\Tests\cloud\Functional\Config\CloudAdminSettings::testCloudAdminSettings()

Specification:

  • Implement Drupal\Tests\cloud\Traits\CloudTestFormDataTrait\createCloudSettingsTestFormData() to return an array including the random input data for /admin/config/services/cloud/settings
  • Access (HTTP Get) /admin/config/services/cloud/settings and post the random input data.

Issue fork cloud-3244513

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

Comments

yutong.li created an issue. See original summary.

yutong.li credited yas.

yutong.li’s picture

yutong.li commented
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Related issues: +#3232932: Refactor test cases for AWS Cloud Admin Setting forms, +#3245650: Add test cases for K8s Admin Setting forms (Settings), +#3246044: Add test cases for OpenStack Admin Setting forms (Settings), +#3245877: Add test cases for VMware Admin Setting forms (Settings)
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Title: Add test cases for Cloud Admin Setting forms (Settings) » Add test cases for Cloud Admin Setting forms (Settings)

yutong.li opened merge request !490

yutong.li’s picture

yutong.li commented
Status: Active » Needs review
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Needs review » Reviewed & tested by the community

@yutong.li

Thank you for adding the test case. It looks good. I'll merge the patch to 3.x and 4.x, and close this issue as Fixed.

yas closed merge request !490

  • yas committed d14be5d on 4.x authored by yutong.li 
    Issue #3244513 by yutong.li, yas: Add test cases for Cloud Admin Setting...

  • yas committed 7f2c03f on 3.x authored by yutong.li 
    Issue #3244513 by yutong.li, yas: Add test cases for Cloud Admin Setting...

yas opened merge request !493

  • yas committed d262ea5 on 4.x authored by yutong.li 
    Issue #3244513 by yas: Hotfix - Add test cases for Cloud Admin Setting...

  • yas committed 2d1a09e on 3.x authored by yutong.li 
    Issue #3244513 by yas: Hotfix - Add test cases for Cloud Admin Setting...
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
commented
Status: Reviewed & tested by the community » Fixed

yas closed merge request !493

Status: Fixed » Closed (fixed)

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