WMDE-leszek has submitted this change and it was merged.

Change subject: Avoid changing the URL during SliderView test
......................................................................


Avoid changing the URL during SliderView test

This was super annoying as if you went to refresh
the test page to run the tests again it would take
you to some broken diff page instead...

Change-Id: I43ca8edd005aa3ef72b74a5d812cbe39bccaf332
---
M tests/RevisionSlider.SliderView.test.js
1 file changed, 11 insertions(+), 1 deletion(-)

Approvals:
  WMDE-leszek: Verified; Looks good to me, approved
  jenkins-bot: Verified



diff --git a/tests/RevisionSlider.SliderView.test.js 
b/tests/RevisionSlider.SliderView.test.js
index 86db974..c2c7cf6 100644
--- a/tests/RevisionSlider.SliderView.test.js
+++ b/tests/RevisionSlider.SliderView.test.js
@@ -2,10 +2,20 @@
        var SliderView = mw.libs.revisionSlider.SliderView,
                Slider = mw.libs.revisionSlider.Slider,
                Revision = mw.libs.revisionSlider.Revision,
-               RevisionList = mw.libs.revisionSlider.RevisionList;
+               RevisionList = mw.libs.revisionSlider.RevisionList,
+               startHistoryState, startHref;
 
        QUnit.module( 'ext.RevisionSlider.SliderView' );
 
+       QUnit.testStart( function () {
+               startHistoryState = history.state;
+               startHref = window.location.href;
+       } );
+
+       QUnit.testDone( function () {
+               history.replaceState( startHistoryState, 'QUnit', startHref );
+       } );
+
        QUnit.test( 'render', function ( assert ) {
                var $container = $( '<div/>' ),
                        view = new SliderView( new Slider( new RevisionList( [

-- 
To view, visit https://gerrit.wikimedia.org/r/288431
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I43ca8edd005aa3ef72b74a5d812cbe39bccaf332
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/RevisionSlider
Gerrit-Branch: master
Gerrit-Owner: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: Gabriel Birke <gabriel.bi...@wikimedia.de>
Gerrit-Reviewer: Jakob <jakob.warkot...@wikimedia.de>
Gerrit-Reviewer: Kai Nissen (WMDE) <kai.nis...@wikimedia.de>
Gerrit-Reviewer: Tobias Gritschacher <tobias.gritschac...@wikimedia.de>
Gerrit-Reviewer: WMDE-Fisch <christoph.fisc...@wikimedia.de>
Gerrit-Reviewer: WMDE-leszek <leszek.mani...@wikimedia.de>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to