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/


  • 14
  • 2
  • 3
  • 1
  • 20
Description From Last Updated
234r guest7552 guest7552
This commit sucks in a way no other commits suck guest9193 guest9193
Test comment guest2904 guest2904
How can i comment on specific files ?? guest2904 guest2904
Hey guest7732 guest7732
I like that RB allows review comments on lines outside the scope of the changed lines. It's also great that ... guest44 guest44
fjfhj guest9517 guest9517
YOu should not strip the api from the uri guest2904 guest2904
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
If you remove the 'api' prefix, you also have to update the web security classes otherwise no endpoint will be ... guest9517 guest9517
resolved one more time. xxx guest7116 guest7116
Dont strip the api part of the uri it will broke other dependencies * Ok * * Testing 1 2 ... guest2904 guest2904
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. 
      
    1. that kind of... your opinion

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

  3. 
      
guest9517
  1. Fix that issue and then we should be good!

  2. If you remove the 'api' prefix, you also have to update the web security classes otherwise no endpoint will be reachable ;)
  3. 
      
guest5744
guest5744
  1. 
      
  2. hkhhkjh

  3. 
      
guest9517
  1. 
      
  2. reviewboard/webapi/tests/test_webhook.py (Diff revision 1)
     
     
     
     
     
  3. 
      
guest2904
  1. Review testing

  2. Test comment

  3. How can i comment on specific files ??

  4. YOu should not strip the api from the uri

  5. Dont strip the api part of the uri it will broke other dependencies

    * Ok *

    * Testing

    1 2 testing

    Reread comment above
    - a
    - b
    - c

  6. 
      
guest9348
  1. 
      
  2. reviewboard/webapi/tests/test_webhook.py (Diff revision 1)
     
     
     
     
     

    my comment

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

    function test() {
        console.log('Test failed');
    }
    
  4. 
      
guest6520
  1. Ship It!
  2. 
      
guest7732
guest87
  1. Ship It!
  2. 
      
guest6457
  1. Ship It!
  2. 
      
guest3217
  1. Ship It!
  2. 
      
guest3371
  1. 🏆

  2. 
      
guest5390
  1. Ship It!
  2. 
      
Loading...