Skip to content

Commit 8eb38bd

Browse files
committed
Improve testing
1 parent 175593f commit 8eb38bd

37 files changed

+2720
-31
lines changed

go.mod

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
module github.com/dwin/goSecretBoxPassword
22

33
require (
4+
github.com/corpix/uarand v0.0.0 // indirect
5+
github.com/icrowley/fake v0.0.0-20180203215853-4178557ae428
6+
github.com/stretchr/testify v1.3.0 // indirect
47
golang.org/x/crypto v0.0.0-20181203042331-505ab145d0a9
58
golang.org/x/sys v0.0.0-20181213200352-4d1cda033e06 // indirect
69
)

go.sum

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,14 @@
1+
github.com/corpix/uarand v0.0.0 h1:mNbzro1GwUcZ1hmO2rWXytkR3JBxNxxctzjyuhO+Aig=
2+
github.com/corpix/uarand v0.0.0/go.mod h1:JSm890tOkDN+M1jqN8pUGDKnzJrsVbJwSMHBY4zwz7M=
3+
github.com/davecgh/go-spew v1.1.0 h1:ZDRjVQ15GmhC3fiQ8ni8+OwkZQO4DARzQgrnXU1Liz8=
4+
github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
5+
github.com/icrowley/fake v0.0.0-20180203215853-4178557ae428 h1:Mo9W14pwbO9VfRe+ygqZ8dFbPpoIK1HFrG/zjTuQ+nc=
6+
github.com/icrowley/fake v0.0.0-20180203215853-4178557ae428/go.mod h1:uhpZMVGznybq1itEKXj6RYw9I71qK4kH+OGMjRC4KEo=
7+
github.com/pmezard/go-difflib v1.0.0 h1:4DBwDE0NGyQoBHbLQYPwSUPoCMWR5BEzIk/f1lZbAQM=
8+
github.com/pmezard/go-difflib v1.0.0/go.mod h1:iKH77koFhYxTK1pcRnkKkqfTogsbg7gZNVY4sRDYZ/4=
9+
github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
10+
github.com/stretchr/testify v1.3.0 h1:TivCn/peBQ7UY8ooIcPgZFpTNSz0Q2U6UrFlUfqbe0Q=
11+
github.com/stretchr/testify v1.3.0/go.mod h1:M5WIy9Dh21IEIfnGCwXGc5bZfKNJtfHm1UVUgZn+9EI=
112
golang.org/x/crypto v0.0.0-20181203042331-505ab145d0a9 h1:mKdxBk7AujPs8kU4m80U72y/zjbZ3UcXC7dClwKbUI0=
213
golang.org/x/crypto v0.0.0-20181203042331-505ab145d0a9/go.mod h1:6SG95UA2DQfeDnfUPMdvaQW0Q7yPrPDi9nlGo2tz2b4=
314
golang.org/x/sys v0.0.0-20181213200352-4d1cda033e06 h1:0oC8rFnE+74kEmuHZ46F6KHsMr5Gx2gUQPuNz28iQZM=

password_test.go

Lines changed: 4 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,9 @@ package password
1919

2020
import (
2121
"fmt"
22-
"math/rand"
2322
"testing"
24-
"time"
23+
24+
"github.com/icrowley/fake"
2525
)
2626

2727
func TestBench(t *testing.T) {
@@ -48,42 +48,15 @@ func TestBench(t *testing.T) {
4848
}
4949
}
5050

51-
var src = rand.NewSource(time.Now().UnixNano())
52-
53-
const (
54-
chars = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ!@#$%^&*()_+<>?/,.:;[]-~`"
55-
letterIdxBits = 6 // 6 bits to represent a letter index
56-
letterIdxMask = 1<<letterIdxBits - 1 // All 1-bits, as many as letterIdxBits
57-
letterIdxMax = 63 / letterIdxBits // # of letter indices fitting in 63 bits
58-
)
59-
60-
func RandStringBytesMaskImprSrc(n int) string {
61-
b := make([]byte, n)
62-
// A src.Int63() generates 63 random bits, enough for letterIdxMax characters!
63-
for i, cache, remain := n-1, src.Int63(), letterIdxMax; i >= 0; {
64-
if remain == 0 {
65-
cache, remain = src.Int63(), letterIdxMax
66-
}
67-
if idx := int(cache & letterIdxMask); idx < len(chars) {
68-
b[i] = chars[idx]
69-
i--
70-
}
71-
cache >>= letterIdxBits
72-
remain--
73-
}
74-
75-
return string(b)
76-
}
77-
7851
func TestHash(t *testing.T) {
7952

8053
var total int
8154
runs := 52
8255
// Test expect success
8356
for i := 2; i < runs; i++ {
8457
// Generate Test Password
85-
testUserPass := RandStringBytesMaskImprSrc(i * 4)
86-
testMasterPass := RandStringBytesMaskImprSrc(i * 4)
58+
testUserPass := fake.Password(i*4, 1000, true, true, true)
59+
testMasterPass := fake.Password(i*4, 1000, true, true, true)
8760
// Hash Password
8861
output, err := Hash(testUserPass, testMasterPass, 0, ScryptParams{N: 32768, R: 16, P: 1}, DefaultParams)
8962
if err != nil {

vendor/github.com/corpix/uarand/.gitignore

Lines changed: 3 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/corpix/uarand/.go-makefile.json

Lines changed: 11 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/corpix/uarand/.travis.yml

Lines changed: 9 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/corpix/uarand/LICENSE

Lines changed: 21 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/corpix/uarand/Makefile

Lines changed: 52 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/corpix/uarand/README.md

Lines changed: 38 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

vendor/github.com/corpix/uarand/glide.lock

Lines changed: 16 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)