Summary

Add a unit test for API root caching.

Review Request #4109 — Created May 6, 2013 and submitted May 10, 2013, 6:24 p.m.

Information

Owner

david

Repository

Review Board

Branch

master

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

Add a unit test for API root caching.

This adds a unit test to make sure that caching of uri_templates in the API root
takes into account the correct local site.

See http://reviews.reviewboard.org/r/4107/ for the actual fix.

Testing Done

Ran all tests.

Before the djblets change, the new test failed. After, it succeeds.

Issues

Description	From	Last Updated
Col: 80 E501 line too long (94 > 79 characters)	reviewbot	May 6, 2013, 3:51 p.m.
Col: 80 E501 line too long (80 > 79 characters)	reviewbot	May 6, 2013, 3:51 p.m.
Col: 80 E501 line too long (94 > 79 characters)	reviewbot	May 6, 2013, 3:51 p.m.
Col: 5 E303 too many blank lines (3)	reviewbot	May 6, 2013, 3:51 p.m.
Col: 80 E501 line too long (94 > 79 characters)	reviewbot	May 6, 2013, 3:55 p.m.
Col: 80 E501 line too long (94 > 79 characters)	reviewbot	May 6, 2013, 3:55 p.m.

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    reviewboard/webapi/tests.py
  Ignored Files:
    reviewboard/site/fixtures/test_site.json

reviewboard/webapi/tests.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (94 > 79 characters)
```
reviewboard/webapi/tests.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Col: 80
 E501 line too long (80 > 79 characters)
```
reviewboard/webapi/tests.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (94 > 79 characters)
```
reviewboard/webapi/tests.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Col: 5
 E303 too many blank lines (3)
```

Diff:

Revision 2 (+47)

Show changes

	reviewboard/site/fixtures/test_site.json
	reviewboard/webapi/tests.py

This is a review from Review Bot.
  Tool: PEP8 Style Checker
  Processed Files:
    reviewboard/webapi/tests.py
  Ignored Files:
    reviewboard/site/fixtures/test_site.json

reviewboard/webapi/tests.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (94 > 79 characters)
```
reviewboard/webapi/tests.py (Diff revision 2)
The issue has been dropped. Show all issues
```
Col: 80
 E501 line too long (94 > 79 characters)
```

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-1.7.x (547e1a0).

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

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

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

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