Use template strings for multi-line SQL statements in tests

RESOLVED FIXED in mozilla34

Status

()

Product:
Toolkit
Component:
Places
Status:
RESOLVED FIXED
Reported:
5 years ago
Modified:
4 years ago

People

(Reporter: michael, Assigned: michael)

Tracking

Version:
unspecified
Target:
mozilla34
Points:
---
Bug Flags:
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Patch
29.81 KB, patch
ttaubert
: review+
Details | Diff | Splinter Review
(Assignee)

Description

•
5 years ago 
Following the adoption of template strings for multi-line SQL statements in bugs 1041429 and 1057782, I propose doing the same for tests.
(Assignee)

Comment 1

•
5 years ago 
Created attachment 8481849 [details] [diff] [review]
Patch

Here are the results from the try server:

https://tbpl.mozilla.org/?tree=Try&rev=ff1d31629410
Attachment #8481849 - Flags: review?(ttaubert)

Comment 2

•
5 years ago 
Comment on attachment 8481849 [details] [diff] [review]
Patch

Review of attachment 8481849 [details] [diff] [review]:
-----------------------------------------------------------------

Great, thanks!
Attachment #8481849 - Flags: review?(ttaubert) → review+
Status: NEW → ASSIGNED
 (Assignee)

Updated

•
5 years ago
Keywords: checkin-needed

Comment 4

•
5 years ago 
https://hg.mozilla.org/mozilla-central/rev/11e22b691251
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Flags: qe-verify-
You need to log in before you can comment on or make changes to this bug.