Closed
Bug 944693
Opened 11 years ago
Closed 8 years ago
Use shared secret auth in tests instead of oauth
Categories
(Marketplace Graveyard :: Code Quality, defect, P3)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: mat, Unassigned)
Details
(Whiteboard: [marketplace-transition])
Pretty much all of our API tests that test views/resources rely on OAuthClient, but the main authentication scheme is what fireplace, rocketfuel etc use, and that's SharedSecret. In fact, we even had a bug recently (bug 943471) that was due to an edge case in RestSharedSecretAuthentication code; view tests would have caught that if they had used Shared Secret. I propose we introduce a new test client class, SharedSecretClient, and use that in our API tests. (It might even be faster, who knows)
Reporter | ||
Updated•11 years ago
|
Priority: -- → P3
Updated•10 years ago
|
Component: API → Code Quality
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
Whiteboard: [marketplace-transition]
You need to log in
before you can comment on or make changes to this bug.
Description
•