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)

x86_64
Linux
defect

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)
Priority: -- → P3
Component: API → Code Quality
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.