Add a test to ensure all of the basic UI functionality works as intended.

CommentFileSizeAuthor
#11 backup_migrate-n2937023-11.patch8.51 KBDamienMcKenna
#10 backup_migrate-n2937023-10.interdiff.txt4.5 KBDamienMcKenna
#10 backup_migrate-n2937023-10.patch6.93 KBDamienMcKenna
#7 backup_migrate-n2937023-7.patch6.57 KBDamienMcKenna
#7 backup_migrate-n2937023-7.interdiff.txt4.92 KBDamienMcKenna
#5 backup_migrate-n2937023-5.patch3.77 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna created an issue. See original summary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Priority: Normal » Major
Parent issue: #2912459: Add an initial test (D7) »
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Active
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Related issues: +#2937021: Add test for the module's permissions (D7)
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Active » Needs review
FileSize
backup_migrate-n2937023-5.patch3.77 KB

First step - step through the basic backup process.

Status: Needs review » Needs work

The last submitted patch, 5: backup_migrate-n2937023-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review
FileSize
backup_migrate-n2937023-7.interdiff.txt4.92 KB
backup_migrate-n2937023-7.patch6.57 KB

Testing the main page a bit more.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Issue tags: +#fldc18

Status: Needs review » Needs work

The last submitted patch, 7: backup_migrate-n2937023-7.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review
FileSize
backup_migrate-n2937023-10.patch6.93 KB
backup_migrate-n2937023-10.interdiff.txt4.5 KB

Commenting out the file check for now.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
backup_migrate-n2937023-11.patch8.51 KB

Renamed the classes a little bit.

  • DamienMcKenna committed 89573f7 on 7.x-3.x 
    Issue #2937023 by DamienMcKenna: Add a test for the basic UI...
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed
Parent issue: » #2942331: Plan for Backup and Migrate 7.x-3.6

Committed.

Status: Fixed » Closed (fixed)

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