CommentFileSizeAuthor
#5 2944952-5-test-rest.patch17.53 KBSutharsan
#3 2944952-3-test-rest.patch17.53 KBSutharsan
#2 2944952-test-rest.patch9.55 KBSutharsan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sutharsan created an issue. See original summary.

Sutharsan’s picture

Sutharsan CreditAttribution: Sutharsan commented
Status: Active » Needs review
FileSize
2944952-test-rest.patch9.55 KB

To allow testing the rest.php endpoint file, I did split-off the functional part which allowed overriding the file storage of the incidents and provided a way to bypass the header data and exits which are required for the endpoint, but made testing (very) hard.

Sutharsan’s picture

Sutharsan CreditAttribution: Sutharsan commented
FileSize
2944952-3-test-rest.patch17.53 KB

Status: Needs review » Needs work

The last submitted patch, 3: 2944952-3-test-rest.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Sutharsan’s picture

Sutharsan CreditAttribution: Sutharsan commented
Status: Needs work » Needs review
FileSize
2944952-5-test-rest.patch17.53 KB

Reverted the MockBuilder::onlyMethods to MockBuilder::setMethods (deprecated).

  • Sutharsan committed 645b23e on 8.x-3.x 
    Issue #2944952 by Sutharsan: Add (unit) test for Rest incident storage
Sutharsan’s picture

Sutharsan CreditAttribution: Sutharsan commented
Status: Needs review » Fixed

  • Sutharsan committed 5d5a346 on 4.0.x 
    Ported from 8.x-3.x. Issue #2944952: Add (unit) test for Rest incident...

Status: Fixed » Closed (fixed)

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