Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29891661: Noissue - Fixed test pages integration tests after change of subscription URL (Closed)

Created:
Sept. 25, 2018, 4:07 p.m. by Sebastian Noack
Modified:
Sept. 25, 2018, 5:11 p.m.
Reviewers:
tlucas
CC:
kzar
Visibility:
Public.

Description

Noissue - Fixed test pages integration tests after change of subscription URL

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -3 lines) Patch
M test/wrappers/pages.js View 2 chunks +2 lines, -3 lines 2 comments Download

Messages

Total messages: 3
Sebastian Noack
https://codereview.adblockplus.org/29891661/diff/29891662/test/wrappers/pages.js File test/wrappers/pages.js (right): https://codereview.adblockplus.org/29891661/diff/29891662/test/wrappers/pages.js#newcode20 test/wrappers/pages.js:20: const TEST_PAGES_URL = "https://testpages.adblockplus.org/en/"; Today a change landed for ...
Sept. 25, 2018, 4:12 p.m. (2018-09-25 16:12:06 UTC) #1
Sebastian Noack
Here is the pipeline for a branch that I pushed to GitLab that includes all ...
Sept. 25, 2018, 4:22 p.m. (2018-09-25 16:22:43 UTC) #2
tlucas
Sept. 25, 2018, 5:02 p.m. (2018-09-25 17:02:30 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld