From 69261bc2c14bad7f936970abce5ae18b55421b27 Mon Sep 17 00:00:00 2001
From: Kathleen Tuite <kathleen.tuite@gmail.com>
Date: Mon, 25 Sep 2023 13:40:26 -0700
Subject: [PATCH] Revert "safeNextPathFrom(): add tests for bad content
 (#1003)"

This reverts commit 0d2efaabf93dc50124d7d298d0116ea762394dfc.
---
 test/unit/util/html.js | 42 +++---------------------------------------
 1 file changed, 3 insertions(+), 39 deletions(-)

diff --git a/test/unit/util/html.js b/test/unit/util/html.js
index e45d9c771..31b946abc 100644
--- a/test/unit/util/html.js
+++ b/test/unit/util/html.js
@@ -27,45 +27,9 @@ describe('util/html', () => {
 
   describe('safeNextPathFrom()', () => {
     [
-      // odk-central-frontend
-      [ '/account/edit',          '/#/account/edit' ],            // eslint-disable-line no-multi-spaces
-      [ '/users',                 '/#/users' ],                   // eslint-disable-line no-multi-spaces
-      [ '/users"><badTag ',       '/#/users%22%3E%3CbadTag' ],    // eslint-disable-line no-multi-spaces
-
-      // query params
-      [ '/users?"><badTag ',      '/#/users?%22%3E%3CbadTag' ],   // eslint-disable-line no-multi-spaces
-      [ '/users?="><badTag ',     '/#/users?=%22%3E%3CbadTag' ],  // eslint-disable-line no-multi-spaces
-      [ '/users?a="><badTag ',    '/#/users?a=%22%3E%3CbadTag' ], // eslint-disable-line no-multi-spaces
-      [ '/users?"=><badTag ',     '/#/users?%22=%3E%3CbadTag' ],  // eslint-disable-line no-multi-spaces
-
-      // fragments
-      [ '/users#"><badTag ',      '/#/users#%22%3E%3CbadTag' ],   // eslint-disable-line no-multi-spaces
-      [ '/users#="><badTag ',     '/#/users#=%22%3E%3CbadTag' ],  // eslint-disable-line no-multi-spaces
-      [ '/users#a="><badTag ',    '/#/users#a=%22%3E%3CbadTag' ], // eslint-disable-line no-multi-spaces
-      [ '/users#"=><badTag ',     '/#/users#%22=%3E%3CbadTag' ],  // eslint-disable-line no-multi-spaces
-
-      // query string & fragment
-      [ '/users?"=1#"=><badTag ', '/#/users?%22=1#%22=%3E%3CbadTag' ], // eslint-disable-line no-multi-spaces
-
-      // enketo-express
-      [ '/-/xyz',                 'http://localhost:8989/-/xyz' ],                       // eslint-disable-line no-multi-spaces
-      [ '/-/xyz?"><b',            'http://localhost:8989/-/xyz?%22%3E%3Cb' ],            // eslint-disable-line no-multi-spaces
-      [ '/-/xyz#"><b',            'http://localhost:8989/-/xyz#%22%3E%3Cb' ],            // eslint-disable-line no-multi-spaces
-      [ '/-/xyz?"><b#"><b',       'http://localhost:8989/-/xyz?%22%3E%3Cb#%22%3E%3Cb' ], // eslint-disable-line no-multi-spaces
-
-      // bad domain
-      [ 'http://example.com',                  '/#/' ], // eslint-disable-line no-multi-spaces
-      // with @ char - not a problem if positioned in fragment or after first `/`:
-      [ '@baddomain.com',                      '/#/@baddomain.com' ],                         // eslint-disable-line no-multi-spaces
-      [ '/-/@baddomain.com',                   'http://localhost:8989/-/@baddomain.com' ],    // eslint-disable-line no-multi-spaces
-      [ '&64;baddomain.com',                   '/#/&64;baddomain.com' ],                      // eslint-disable-line no-multi-spaces
-      [ '/-/&64;baddomain.com',                'http://localhost:8989/-/&64;baddomain.com' ], // eslint-disable-line no-multi-spaces
-      [ 'http://localhost:8989@baddomain.com', '/#/' ],                                       // eslint-disable-line no-multi-spaces
-      [ 'http://localhost:8989@baddomain.com', '/#/' ],                                       // eslint-disable-line no-multi-spaces
-
-      // bad protocols
-      [ 'https://localhost:8989', '/#/' ], // eslint-disable-line no-multi-spaces
-      [ 'javascript:badFn()',     '/#/' ], // eslint-disable-line no-multi-spaces,no-script-url
+      [ '/-/xyz',                    'http://localhost:8989/-/xyz' ], // eslint-disable-line no-multi-spaces
+      [ '/account/edit',             '/#/account/edit' ],             // eslint-disable-line no-multi-spaces
+      [ '/users',                    '/#/users' ],                    // eslint-disable-line no-multi-spaces
     ].forEach(([next, expected]) => {
       it(`should convert next=${next} to ${expected}`, () => {
         safeNextPathFrom(next).should.equal(expected);