globalstatedb: Skip Bcrypt test if -short (!5314) · Merge requests · Administrator / sourcegraph

Administrator requested to merge core/test-short into master Aug 20, 2019

Created by: keegancsmith

Bcrypt test is quite CPU intensive. On my laptop I have observed the globalstatedb tests take 6s when I run go test -short ./.... This is the only test which is this slow in short mode.

Test Plan: Ran all tests and then manually checked nothing was slow

go test -count=1 -short ./... | tee test.txt
cat test.txt | grep ^ok | sort -n -k 3

globalstatedb: Skip Bcrypt test if -short

Merge request reports