AITC testcases shouldn't use URLs like /apps/TESTAPP

RESOLVED FIXED

Status

Cloud Services
Server: Sync
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: rfkelly, Assigned: rfkelly)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa+])

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Some data-validation testcases in AITC are putting malformed data to /apps/TESTAPP.  This implicitly assumes that the validation will occur *before* checking that the URL is a valid b64 id, which may not be the case.
Whiteboard: [qa+]
(Assignee)

Comment 1

6 years ago
Created attachment 627028 [details] [diff] [review]
patch to use a valid app id in the tests

Something like this...
Assignee: nobody → rfkelly
Attachment #627028 - Flags: review?(telliott)
Attachment #627028 - Flags: review?(telliott) → review+
(Assignee)

Comment 2

6 years ago
https://github.com/mozilla-services/server-aitc/commit/5efea92f16476f5689c5f107536bfe8825c9c9c3
(Assignee)

Updated

6 years ago
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.