mbland / custom-links

98%

master: 98%

LAST BUILD ON BRANCH fixups

branch: fixups

CHANGE BRANCH

x

Reset

• fixups
• master
• my-links
• empty-hash-view
• codeclimate-token
• cc-test-reporter
• urlp-to-cl
• focus-element
• remove-tabindex
• show-view-promise
• my-links-e2e
• my-links-buttons
• delete-link
• extract-flash
• extract-api-helpers
• dialog
• do-operation
• style-touchups
• view-and-backend
• dialog-data
• stabilize-e2e
• backend-delete
• count-links
• create-unknown-link
• rename-url-to-link
• rename-redirectdb-to-linkdb
• rename-location-to-target
• create-and-update-stamps
• create-link-view
• my-links-landing-page
• show-timestamps
• fix-error-response
• show-view-handle-rejection
• api-json
• edit-link
• count-to-clicks
• edit-view-support
• edit-link-nonfunctional
• make-api-call
• create-click-handler
• refactor
• launch-redis
• readme
• docker
• alpine-fix
• env-config-override
• config-redis
• wait-for-redis
• redis-shutdown
• save-on-exit
• log
• headless-chrome
• cla-#147
• add-okta
• update-npms
• v0.1.0
• dockerize
• downcase-ids
• v0.1.1
• downcase-users-domains
• jsdoc
• search-links
• travis-ci-fix
• autocomplete-and-target-search
• fix-smoke-test-flakiness
• deindex-and-reindex
• redis-refactor
• search-on-target-link
• update-packages
• add-api-for-search
• fix-search-short-links
• target-index-sets
• migrate-autocomplete-target-url-indexes
• search-ui
• v0.2.0
• v0.1.2
• v0.2.1
• autocomplete-api
• autocomplete-ui
• bump-npms-and-dockerfile
• v0.3.0
• transfer-ownership

Build # 391

Build Type

push

travis-ci

Committed by mbland

Commit Message

public: Handle net::ERR_CONNECTION_REFUSED in XHR

As it turns out, the `XMLHttpRequest.onerror` handler won't handle
`net::ERR_CONNECTION_REFUSED` or other network errors. For that, one
needs to check for `xhr.status === 0` after `xhr.readyState === 4`.

The problem manifest when manually testing to simulate a link deletion
request failure by killing the server, and there was no accompanying
error message as expected from the `onerror` handler. The hint for the
solution came from:

https://stackoverflow.com/questions/28556398/how-to-catch-neterr-connection-refused

Run Details

241 of 265 branches covered (90.94%)

Branch coverage included in aggregate %.

1899 of 1921 relevant lines covered (98.85%)

12.43 hits per line