- Issue created by @baldwinlouie
- Merge request !2208Adding BDD test/screenshot for OpenStack Notification Settings and OpenStack Settings β (Merged) created by baldwinlouie
- last update
10 months ago 289 pass, 48 fail - Status changed to Needs review
10 months ago 5:22am 11 January 2024 - First commit to issue fork.
- last update
10 months ago 289 pass, 48 fail - πΊπΈUnited States yas California πΊπΈ
@baldwinlouie
Thank you for adding the BDD test scenario. I'll merge the patch to
5.x
and6.x
, and close this issue as Fixed. - Status changed to RTBC
10 months ago 8:35am 11 January 2024 -
yas β
committed c28469df on 6.x authored by
baldwinlouie β
Issue #3413929 by baldwinlouie, yas: Add a BDD test step to take a...
-
yas β
committed c28469df on 6.x authored by
baldwinlouie β
-
yas β
committed 8bf71cfd on 5.x authored by
baldwinlouie β
Issue #3413929 by baldwinlouie, yas: Add a BDD test step to take a...
-
yas β
committed 8bf71cfd on 5.x authored by
baldwinlouie β
- Status changed to Fixed
10 months ago 8:36am 11 January 2024 - Status changed to Needs work
10 months ago 1:42am 12 January 2024 - Open on Drupal.org βCore: 10.2.1 + Environment: PHP 8.2 & MySQL 8last update
10 months ago Waiting for branch to pass - Open on Drupal.org βCore: 10.2.1 + Environment: PHP 8.2 & MySQL 8last update
10 months ago Waiting for branch to pass - Open on Drupal.org βCore: 10.2.1 + Environment: PHP 8.2 & MySQL 8last update
10 months ago Waiting for branch to pass - Status changed to Needs review
10 months ago 4:16am 12 January 2024 - πΊπΈUnited States yas California πΊπΈ
@baldwinlouie
Thank you for the hotfix. I posted my comment. It looks we need to place the test scenario to the other place. Thanks
- Status changed to Needs work
10 months ago 8:37am 12 January 2024 - last update
10 months ago 289 pass, 48 fail - last update
10 months ago 289 pass, 48 fail - πΊπΈUnited States baldwinlouie
@yas, please review the updated patch. I have moved the tests into its own feature file
OpenStack-ATDD-00-04-CheckConfigurations.feature
- πΊπΈUnited States yas California πΊπΈ
@baldwinlouie
Thank you for the update. Can you please double-check the following coding standard error?
FILE: .../Behat/features/templates/OpenStack-ATDD-00-04-CheckConfigurations.feature -------------------------------------------------------------------------------- FOUND 1 ERROR AFFECTING 1 LINE -------------------------------------------------------------------------------- 14 | ERROR | [x] Expected 1 newline at end of file; 2 found --------------------------------------------------------------------------------
Thanks
- last update
10 months ago 289 pass, 48 fail - Status changed to Needs review
10 months ago 11:40pm 12 January 2024 - πΊπΈUnited States baldwinlouie
@yas, I've fixed the whitespace error. Please review the patch again.
- πΊπΈUnited States yas California πΊπΈ
@baldwinlouie
Thank you for the fix. I ran the BDD tests and it worked fine. I'll merge the patch to
5.x
and6.x
, and close this issue as Fixed. - Open on Drupal.org βCore: 10.2.1 + Environment: PHP 8.2 & MySQL 8last update
10 months ago Waiting for branch to pass -
yas β
committed dd633107 on 6.x authored by
baldwinlouie β
Issue #3413929 by baldwinlouie, yas: Add a BDD test step to take a...
-
yas β
committed dd633107 on 6.x authored by
baldwinlouie β
-
yas β
committed 1fd0927e on 5.x authored by
baldwinlouie β
Issue #3413929 by baldwinlouie, yas: Hotfix - Add a BDD test step to...
-
yas β
committed 1fd0927e on 5.x authored by
baldwinlouie β
- Status changed to Fixed
10 months ago 11:51pm 12 January 2024 Automatically closed - issue fixed for 2 weeks with no activity.