| Summary: | t/db_dependent/api/v1/patrons.t is failing | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> | 
| Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> | 
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> | 
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | kyle, martin.renvoize, tomascohen | 
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | 
            https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17621 https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17654 https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21219  | 
      ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 | 
| Patch complexity: | --- | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: | Circulation function: | ||
| Attachments: | 
              Bug 17651: borrowers.updated_on can be null - swagger def
               Bug 17651: borrowers.updated_on can be null - swagger def  | 
      ||
| 
 
        
          Description
        
        
          Jonathan Druart
        
        
        
        
          2016-11-17 15:07:25 UTC
        
       
    If borrowers.updated_on is null, the tests will fail Created attachment 57604 [details] [review] Bug 17651: borrowers.updated_on can be null - swagger def The swagger file does not consider borrowers.updated_on as a nullable value, it should Test plan: 0/ Do not apply this patch 1/ update borrowers set updated_on=null; 2/ prove t/db_dependent/api/v1/patrons.t => Fail 3/ Apply this patch 4/ prove t/db_dependent/api/v1/patrons.t => green Created attachment 57607 [details] [review] Bug 17651: borrowers.updated_on can be null - swagger def The swagger file does not consider borrowers.updated_on as a nullable value, it should Test plan: 0/ Do not apply this patch 1/ update borrowers set updated_on=null; 2/ prove t/db_dependent/api/v1/patrons.t => Fail 3/ Apply this patch 4/ prove t/db_dependent/api/v1/patrons.t => green Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Trivial tests fix, passing QA. Pushed to master for 16.11, thanks Jonathan!  |