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 5


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...

   
  		
  82e6d4656e0f8c6c13cc0dc6c3977fb7d342f82c
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Deleted print statements in unit tests

   
  		
  d954f7e915b623c4c4ff11b465b6e7d365e1e77b
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Changed variable name to automate

   
  		
  6c5160b12d4039eea432e095c51bbc3ce282a828
  sng06
 


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

   
  		
  90f9fc167785fff32dfda6c33434964fd8ea6768
  sng06
 


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

   
  		
  89b0f1f05544fd471336b15faa7a00c0325b2eee
  sng06
 


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

   
  		
  4ff6a2fba5f9825a9a6398eec5f6cc5c1daa1b03
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added unit tests to review_request and test_email_sending test suites

   
  		
  4842bf0893f569e60b6f90f4debc41fc20e13e15
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added default value to automate argument in close_review_request method

   
  		
  645cbdd6867ff75c97733f7db260ca3db44c1bdb
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Changed assertEqual to assertTrue for test_post_commit_hook unit test

   
  		
  eaa3c4d2db3b0c554a35961140f7cff2f2b4694d
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added optional for automate field in docstrings

   
  		
  5bd81e90eb551e1dcd96efdf0f2dcd779a46e1fc
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Fixed review_request_close unit test spy assertion

   
  		
  66f36fe7a1a20965a48ce68b7c96179756230ac3
  sng06
 


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

   
  		
  3376fedddde90d296929cfb63045bd0032deeb15
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added or edited docstrings and fixed formatting issues

   
  		
  0b6eb109d4e5167b022ecd5ccc6bc645b66cf48a
  sng06
 


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

   
  		
  e959cedd54e4986ef3448c762a86b634df5af3ea
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Added unit tests to test changeEntry

   
  		
  e4b46714b7522a40917137125685fe2b11f458e5
  sng06
 


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

   
  		
  d11641558fd1ebcae91b4df9bac51266f15fb41a
  sng06
 


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

   
  		
  bcf023cdf739ea416e8e74781acc9e35feed02d1
  sng06
 


 
  
   
  
  
   
  
 
 
  
   
    
Deleted print statements

   
  		
  797a220df1eb342c1f15be33f9b2779d8cb92f31
  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/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/tests/test_review_request.py
     
    

   
  

 










     

    

   

  

  

   
   Loading...