test: Remove commented-out test cases

preactjs · Aug 31, 2024 · e551281 · e551281
1 parent 5d7bb24
commit e551281
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/test/router.test.js b/test/router.test.js
@@ -242,7 +242,7 @@ describe('Router', () => {
 		expect(scratch).to.have.property('innerHTML', '<h1>A</h1><p>hello</p>');
 		// We should never re-invoke <A /> while loading <B /> (that would be a remount of the old route):
 		// ...but we do
-		// expect(A).not.to.have.been.called;
+		expect(A).not.to.have.been.called;
 		expect(B).to.have.been.calledWith({ path: '/b', query: {}, params: {}, rest: '' });
 
 		B.resetHistory();
@@ -268,7 +268,7 @@ describe('Router', () => {
 		expect(scratch).to.have.property('innerHTML', '<h1>B</h1><p>hello</p>');
 		// We should never re-invoke <B /> while loading <C /> (that would be a remount of the old route):
 		// ...but we do
-		//expect(B).not.to.have.been.called;
+		expect(B).not.to.have.been.called;
 		expect(C).to.have.been.calledWith({ path: '/c', query: {}, params: {}, rest: '' });
 
 		C.resetHistory();
@@ -286,9 +286,10 @@ describe('Router', () => {
 
 		expect(scratch).to.have.property('innerHTML', '<h1>B</h1><p>hello</p>');
 		expect(C).not.to.have.been.called;
-		// expect(B).to.have.been.calledOnce();
+		expect(B).to.have.been.calledOnce;
 		expect(B).to.have.been.calledWith({ path: '/b', query: {}, params: {}, rest: '' });
 
+		A.resetHistory();
 		B.resetHistory();
 		loc.route('/');
 		await sleep(1);