• 
      

    Split out ReviewGroup and ReviewGroupUser webapi tests.

    Review Request #4518 — Created Aug. 30, 2013 and submitted

    Information

    Review Board
    master

    Reviewers

    Split out ReviewGroup and ReviewGroupUser webapi tests.

    This moves out ReviewGroupResourceTests and ReviewGroupUserResourceTests
    out into their own files.

    Unit tests pass.

    Description From Last Updated

    'DefaultReviewerResourceTests' imported but unused

    reviewbotreviewbot

    'RepositoryBranchesResourceTests' imported but unused

    reviewbotreviewbot

    'RepositoryCommitsResourceTests' imported but unused

    reviewbotreviewbot

    'RepositoryInfoResourceTests' imported but unused

    reviewbotreviewbot

    'ReviewGroupResourceTests' imported but unused

    reviewbotreviewbot

    'ReviewGroupUserResourceTests' imported but unused

    reviewbotreviewbot

    'RootResourceTests' imported but unused

    reviewbotreviewbot

    'ServerInfoResourceTests' imported but unused

    reviewbotreviewbot

    'SessionResourceTests' imported but unused

    reviewbotreviewbot

    'ValidateDiffResourceTests' imported but unused

    reviewbotreviewbot

    Col: 80 E501 line too long (80 > 79 characters)

    reviewbotreviewbot

    Col: 80 E501 line too long (87 > 79 characters)

    reviewbotreviewbot

    Col: 80 E501 line too long (91 > 79 characters)

    reviewbotreviewbot

    Col: 80 E501 line too long (92 > 79 characters)

    reviewbotreviewbot

    Col: 80 E501 line too long (81 > 79 characters)

    reviewbotreviewbot

    Col: 80 E501 line too long (95 > 79 characters)

    reviewbotreviewbot

    Col: 1 W391 blank line at end of file

    reviewbotreviewbot

    Col: 80 E501 line too long (93 > 79 characters)

    reviewbotreviewbot

    Col: 80 E501 line too long (84 > 79 characters)

    reviewbotreviewbot

    Col: 80 E501 line too long (91 > 79 characters)

    reviewbotreviewbot

    Col: 80 E501 line too long (106 > 79 characters)

    reviewbotreviewbot
    reviewbot
    1. This is a review from Review Bot.
      Tool: PEP8 Style Checker
      Processed Files:
      reviewboard/webapi/tests/review_group.py
      reviewboard/webapi/tests/review_group_user.py
      reviewboard/webapi/tests/init.py
      Ignored Files:

    2. Show all issues

      Col: 80
      E501 line too long (80 > 79 characters)

    3. Show all issues

      Col: 80
      E501 line too long (87 > 79 characters)

    4. Show all issues

      Col: 80
      E501 line too long (91 > 79 characters)

    5. Show all issues

      Col: 80
      E501 line too long (92 > 79 characters)

    6. Show all issues

      Col: 80
      E501 line too long (81 > 79 characters)

    7. Show all issues

      Col: 80
      E501 line too long (95 > 79 characters)

    8. Show all issues

      Col: 1
      W391 blank line at end of file

    9. Show all issues

      Col: 80
      E501 line too long (93 > 79 characters)

    10. Show all issues

      Col: 80
      E501 line too long (84 > 79 characters)

    11. Show all issues

      Col: 80
      E501 line too long (91 > 79 characters)

    12. Show all issues

      Col: 80
      E501 line too long (106 > 79 characters)

    13. 
        
    reviewbot
    1. This is a review from Review Bot.
      Tool: Pyflakes
      Processed Files:
      reviewboard/webapi/tests/review_group.py
      reviewboard/webapi/tests/review_group_user.py
      reviewboard/webapi/tests/init.py
      Ignored Files:

    2. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'DefaultReviewerResourceTests' imported but unused

    3. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'RepositoryBranchesResourceTests' imported but unused

    4. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'RepositoryCommitsResourceTests' imported but unused

    5. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'RepositoryInfoResourceTests' imported but unused

    6. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'ReviewGroupResourceTests' imported but unused

    7. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'ReviewGroupUserResourceTests' imported but unused

    8. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'RootResourceTests' imported but unused

    9. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'ServerInfoResourceTests' imported but unused

    10. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'SessionResourceTests' imported but unused

    11. reviewboard/webapi/tests/__init__.py (Diff revision 1)
       
       
      Show all issues

      'ValidateDiffResourceTests' imported but unused

    12. 
        
    chipx86
    Review request changed
    Status:
    Completed