Fix the sample API URL for the WebHooks unit tests.

Review Request #535 - Created March 31, 2017 and updated

guest6871
Review Board
1448466...
demo
The WebHooks unit tests were prefixing each API URL with "api/", which
other unit tests don't do. This change simply removes the "api/" prefix.

Testing Done:
Ran unit tests. Saw that they didn't include the prefix anymore.

Reviewed at https://reviews.reviewboard.org/r/8852/


  • 7
  • 2
  • 1
  • 0
  • 10
Description From Last Updated
234r guest7552 guest7552
This commit sucks in a way no other commits suck guest9193 guest9193
I like that RB allows review comments on lines outside the scope of the changed lines. It's also great that ... guest44 guest44
asdfasdfa sdfa sdf asd fasdf asdf asd fasd fas dfa sdfasdf guest7116 guest7116
I like the api.. please mr. don't take it away from me!! :'( guest9193 guest9193
resolved one more time. xxx guest7116 guest7116
qwe guest7552 guest7552
guest6871
  1. Please review it

  2. 
      
guest6871
  1. Another test
  2. 
      
guest6871
guest6871
  1. All fine

  2. 
      
guest7116
  1. 
      
  2. asdfasdfa sdfa sdf asd fasdf asdf
    asd
    fasd
    fas
    dfa
    sdfasdf

  3. reviewboard/webapi/tests/test_webhook.py (Diff revision 1)
     
     
     
     
     

    resolved one more time. xxx

    1. do you like it?
      ===============

  4. 
      
guest831
  1. 
      
  2. csdcxc

  3. 
      
guest44
  1. 
      
  2. reviewboard/webapi/tests/test_webhook.py (Diff revision 1)
     
     
     
     
     
     
     
     
     
     

    I like that RB allows review comments on lines outside the scope of the changed lines. It's also great that multiple lines can be selected. I wish GitHub pull request code reviews had these two features.

  3. How can we get a unified diff instead of side-by-side view?

  4. 
      
guest5047
  1. Ship It!

  2. 
      
guest8184
  1. fuck it

  2. 
      
guest7467
  1. that sounds good.

  2. 
      
guest7923
  1. Ran unit tests. Saw that they didn't include the prefix anymore.?

  2. 
      
guest7552
  1. Ship It!

  2. 
      
guest7552
guest1661
  1. 
      
  2. trying to create an issue
  3. 
      
guest9193
  1. 
      
  2. I like the api.. please mr. don't take it away from me!! :'(

  3. 
      
guest9193
  1. 
      
  2. This commit sucks in a way no other commits suck

  3. 
      
Loading...