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

Issue 29890620: Noissue - Simplified qunit test wrapper (Closed)

Created:
Sept. 24, 2018, 6:50 p.m. by Sebastian Noack
Modified:
Sept. 25, 2018, 5:10 p.m.
Reviewers:
tlucas
Visibility:
Public.

Description

Noissue - Simplified qunit test wrapper

Patch Set 1 #

Total comments: 3

Patch Set 2 : Added missing semicolon #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -13 lines) Patch
M test/wrappers/qunit.js View 1 1 chunk +7 lines, -13 lines 0 comments Download

Messages

Total messages: 5
Sebastian Noack
https://codereview.adblockplus.org/29890620/diff/29890621/test/wrappers/qunit.js File test/wrappers/qunit.js (right): https://codereview.adblockplus.org/29890620/diff/29890621/test/wrappers/qunit.js#newcode27 test/wrappers/qunit.js:27: ).then(elem => .wait(until.elementLocated(...)) returns a promise that resolves with ...
Sept. 24, 2018, 6:56 p.m. (2018-09-24 18:56:05 UTC) #1
tlucas
Can you please file an additional merge request on gitlab.com for this? Not as a ...
Sept. 25, 2018, 12:24 p.m. (2018-09-25 12:24:45 UTC) #2
Sebastian Noack
There you go: https://gitlab.com/eyeo/adblockplus/adblockpluschrome/pipelines/31166535 The qunit tests still pass. However, the test pages fail. This, ...
Sept. 25, 2018, 3:55 p.m. (2018-09-25 15:55:20 UTC) #3
Sebastian Noack
I fixed the issue that caused the test pages integration to fail. Here is the ...
Sept. 25, 2018, 4:20 p.m. (2018-09-25 16:20:36 UTC) #4
tlucas
Sept. 25, 2018, 5:03 p.m. (2018-09-25 17:03:52 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld