Added automated flag when closing or editing review requests

 






 

 


  Review Request #12051
  —
  Created Feb. 10, 2022 and updated  — Latest diff uploaded 

 


















 

  

   Information

  





 
 

  sheenaNg
 









 
 

  Review Board
 









 
 

  master
 









 
 

  
 









 
 

  12038
 









 
 

  12223
 






 




 

  

   Reviewers

  





 
 

  reviewboard
 









 
 

  
 






 








 





  


 
 

  
    

  • Added automated flag to multiple methods related to hostingsvcs close 
    
hook, ReviewRequestResourceAPI Update(), and review_request.close() 
    
to store the value in ChangeDescription model to track automated 
    
closing of review requests 
    • 





    

  • Added automated flag to 
    
ReviewRequestResourceDraftAPI Update(), Create(), 
    
review_request.publish(), draft.publish() to store the value in
    
ChangeDescription model to track automated upddating/publishing 
    
of review requests 
    • 





    

  • Added automated flag to closed/published signals and email-related methods to render the 
    
review_request_email.txt differently when automated=True 
    • 





    

  • Added codes to change.html template to render RB logo in the 
    
review request change entry when automated=True
    • 


 








  


 
 

  
    

  • Added unit tests to test_email_sending.py
    • 

  • Added unit tests to test_review_request.py
    • 

  • Added unit tests to test_review_request_draft.py
    • 

  • Added unit tests to test_entries.py
    • 

  • Added unit tests to webapi/tests/test_review_request.py
    • 

  • Added unit tests to webapi/tests/test_review_request_draft.py
    • 

  • Added assertion to test_github.py, test_rbgateway.py, 
    
test_bitbucket.py
    • 



All tests passed. 


    

  • Attached screenshots to show changes in close email .txt and 
    
frontend changeEntry.
    • 


 







 









 





  








 




 







   

    
    
Diff Revision 6


This is not the most recent revision of the diff. The latest diff is revision 10. See what's changed.

    

 

 

 

 
orig
1
2
3
4
5
6
7
8
9
10

 



    


    

      
Commits

      

 

 

  
   First
   Last
  
  Summary
  ID
  Author
 



 
  
   
  
  
   
  
 
 
  
   
    
Added isAutomatedClose flag through close hook flow to ChangeDescription's ex...

   
  		
  15d6fe45b3ce459167df54f7019e3fce9ce5685d
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Deleted print statements in unit tests

   
  		
  b8d6558376dfd06da13dfeda8421d5ef33bac926
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Changed variable name to automate

   
  		
  8aa66d65c2be8136bc2b43db23c5d10c3dca5ca4
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added automate field to ReviewRequestResource API update method and passed it...

   
  		
  94c874518d9f105f829facb72bde5d5855b959b1
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added automate fields and made changes to  email-related methods based on aut...

   
  		
  6096736f849593d99d88b525f018430e753bcc19
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Made changes to close email txt template based on automate flag value

   
  		
  8c254457115746a5bc3f68169f071eb5d59d1498
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added unit tests to review_request and test_email_sending test suites

   
  		
  8ca0e35dd81b36849514a31e10eb6eb02dfc9627
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added default value to automate argument in close_review_request method

   
  		
  80c374d3fbaf65df568a733c181560942fa2f0bc
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Changed assertEqual to assertTrue for test_post_commit_hook unit test

   
  		
  7e7061c68d3a6d0828c704597b3089bca76d504c
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added optional for automate field in docstrings

   
  		
  96a34c4d3e87bcbe285e066e0b4e98265ad5bcde
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Fixed review_request_close unit test spy assertion

   
  		
  4532790323e090df538f5148e517d52bc1d8c895
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added codes to render Review Board logo in review request change box when it ...

   
  		
  1e8a4fa9780e5f82c76ecf87da3c4f766cc51af0
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added or edited docstrings and fixed formatting issues

   
  		
  3a382f9b42fcb3add8eb77c756e78bee3152f901
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added class tag to RB logo avatar rendering in change.html template

   
  		
  e4c5ad6b3a0a6cf18a5c15058e5f77bee6b6d9ab
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added unit tests to test changeEntry

   
  		
  fe28911e4d1f6eb470e93aab7c68b784aea351d4
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added automate field to webapi-request-fields decorator for both create and u...

   
  		
  797995df7da75c2cdef2d315b1780efd2df84186
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
added unit tests to test PUT api for submitted and discarded actions

   
  		
  bd5a3f00b096d3579082c6cc09af7b4887b1e523
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Deleted print statements

   
  		
  568feed51055cd9b9bc143513a6a9c6cd76b8a16
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Edited automate field's description

   
  		
  ec687ac43eb477f5e20374a59c72f839e446e8dc
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added automate field to draft resource PUT endpoint, review request publish-r...

   
  		
  d53d2f7b31dbc98bbcd280076b3b1e9198de26de
  sng06
 




    


    
    

   

  

 









 

  

   
    

     
      
      reviewboard/hostingsvcs/hook_utils.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/hostingsvcs/tests/test_bitbucket.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/hostingsvcs/tests/test_github.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/hostingsvcs/tests/test_rbgateway.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/notifications/email/message.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/notifications/email/signal_handlers.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/notifications/email/utils.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/notifications/tests/test_email_sending.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/reviews/signals.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/reviews/models/review_request.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/reviews/models/review_request_draft.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/reviews/tests/test_entries.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/reviews/tests/test_review_request.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/templates/notifications/review_request_email.txt
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/templates/reviews/entries/change.html
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/webapi/resources/review_request.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/webapi/resources/review_request_draft.py
     
    

   
  

 



 

  

   
    

     
      
      reviewboard/webapi/tests/test_review_request.py
     
    

   
  

 










     

    

   

  

  

   
   Loading...