mbland / custom-links

98%

master: 98%

LAST BUILD ON BRANCH fix-smoke-test-flakiness

branch: fix-smoke-test-flakiness

CHANGE BRANCH

x

Reset

• fix-smoke-test-flakiness
• 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
• fixups
• 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
• 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 # 636

Build Type

push

travis-ci

Committed by mbland

Commit Message

smoke-test: Fix "bad Redis" failure case flakiness

The previous implementation relied on failure due to a bad Redis host
name. This worked most of the time, but while running the test suite in
a coffee shop, this test case would timeout and leave a stale
redis-server running in the background. Eventually I realized this was
because the coffee shop DNS provider was converting failed name lookups
to search queries.

This implementation instead relies on connecting to an unused port, and
doesn't start a redis-server instance to begin with.

Run Details

297 of 326 branches covered (91.1%)

Branch coverage included in aggregate %.

2643 of 2672 relevant lines covered (98.91%)

21.35 hits per line