#1561 - Fixed a bug where we might override the existing path in a supplied url.

[aaronjwhiteside]

Description

Fixed a bug where we might override the existing path in a supplied url.

• Relevant Issues : Path encoding issue after upgrade from 0.9.6 to 1.0.1 #1561
• Type of change :
  • Bug fix for existing feature

[aaronjwhiteside]

@ptrthomas

Build error looks to be unrelated to my change:

Error:  Failures: 
Error:    DockerRunner.testMock:29 match failed: CONTAINS
  $ | actual does not contain expected (STRING:STRING)
  'about:blank'
  '11_tab'

src/test/java/driver/11.feature:11
src/test/java/driver/00.feature:40 ==> expected: <0> but was: <1>

And for what it's worth, the build passes for me locally:

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for karate-parent 2.0.0:
[INFO] 
[INFO] karate-parent ...................................... SUCCESS [  0.514 s]
[INFO] karate-core ........................................ SUCCESS [ 20.276 s]
[INFO] karate-junit4 ...................................... SUCCESS [  8.560 s]
[INFO] karate-junit5 ...................................... SUCCESS [  3.664 s]
[INFO] karate-demo ........................................ SUCCESS [ 38.654 s]
[INFO] karate-gatling ..................................... SUCCESS [ 31.275 s]
[INFO] karate-mock-servlet ................................ SUCCESS [ 11.156 s]
[INFO] karate-robot ....................................... SUCCESS [  2.900 s]
[INFO] karate-e2e-tests ................................... SUCCESS [  0.129 s]
[INFO] karate-archetype ................................... SUCCESS [  0.134 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  01:57 min
[INFO] Finished at: 2021-05-28T16:37:03+10:00
[INFO] ------------------------------------------------------------------------

[ptrthomas]

@aaronjwhiteside no worries, yes the CI fail is a ui test that's flaky, I'll look into it