jquery / jquery-mobile / 2042
43%

DEFAULT BRANCH: master
Ran
Jobs 12
Files 84
Run time 32min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
2042

push

travis-ci

gabrielschulhof 
Navigation: Correctly (un)escape data-url all throughout the code

Also removes code dealing with nested list URL tokens (subPageUrlKey)

The initial bug was regarding the need to use selector-escaping when looking
for pages inside the DOM via their data-url attribute, because the the data-url
may contain "weird" characters like parantheses, single, and double quotes.

During the process of fixing this it became clear that the data-url attribute
can sometimes end up having a URL-encoded value - but not always. This makes
proper string comparison, much less selector-escaping, impossible. So, it
became necessary to require that data-url never be url-encoded.

Conversely, the URL that ends up in the location when using replaceState() must
be URL-encoded, otherwise, upon deep linking, one may end up with an invalid
URL. For example, if the URL contains an actual percent sign, and the URL is
placed in the location via replaceState(), the percent sign must be URL-encoded
in order for deep-linking to work.

Below are the original commit messages that have been squashed:

Pagecontainer: Escape dataUrl when trying to find page based on it
Navigation: Make sure location is encoded when going to a funky page
Pagecontainer: Test that _find() throws not when looking for weird URLs
Init: Correctly assign data-url for initial page
Init: Make sure "data-url" attribute for initial page is decoded
Pagecontainer: Decode URI when assigning to "data-url" during _parse()
Pagecontainer: Use decoded URL to build selector for data-url
Pagecontainer: Call convertUrlToDataUrl() via _createDataUrl()
Path: Always uri-decode when computing dataUrl
Pagecontainer: Remove extra decoding step, now part of _createDataUrl()

This also removes the possibility that a URL gets double-decoded.
Pagecontainer: Test behavior of _find() in the face of weird URLs
Navigation: Use data-url retrieved from Ajax request in its encoded form
Pagecontainer: Pass encoded URL to $.mobile.navigate()
Init: ... (continued)

3632 of 4380 relevant lines covered (82.92%)

954.96 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
7
88.79
 -1.03% /home/travis/build/jquery/jquery-mobile/js/navigation/path.js
17
94.5
 -0.93% /home/travis/build/jquery/jquery-mobile/js/widgets/pagecontainer.js
23
67.42
 -12.92% /home/travis/build/jquery/jquery-mobile/js/widgets/panel.js
Jobs
ID Job ID Ran Files Coverage
1 2042.1 (TYPES=unit JQUERIES=1.8.3) 84
70.26
 Travis Job 2042.1
2 2042.2 (TYPES=unit JQUERIES=1.11.1) 84
70.21
 Travis Job 2042.2
3 2042.3 (TYPES=unit JQUERIES=2.1.1) 84
70.21
 Travis Job 2042.3
4 2042.4 (TYPES=unit JQUERIES=git) 84
70.21
 Travis Job 2042.4
5 2042.5 (TYPES=integration SUITES=!navigation JQUERIES=1.8.3) 58
75.52
 Travis Job 2042.5
6 2042.6 (TYPES=integration SUITES=!navigation JQUERIES=1.11.1) 58
75.34
 Travis Job 2042.6
7 2042.7 (TYPES=integration SUITES=!navigation JQUERIES=2.1.1) 58
75.34
 Travis Job 2042.7
8 2042.8 (TYPES=integration SUITES=!navigation JQUERIES=git) 58
75.34
 Travis Job 2042.8
9 2042.9 (TYPES=integration SUITES=navigation JQUERIES=1.8.3) 37
72.38
 Travis Job 2042.9
10 2042.10 (TYPES=integration SUITES=navigation JQUERIES=1.11.1) 37
72.29
 Travis Job 2042.10
11 2042.11 (TYPES=integration SUITES=navigation JQUERIES=2.1.1) 37
72.29
 Travis Job 2042.11
12 2042.12 (TYPES=integration SUITES=navigation JQUERIES=git) 37
72.29
 Travis Job 2042.12
Source Files on build 2042
Coverage ∆ File Lines Relevant Covered Missed Hits/Line