fixing use of require vs assert

Signed-off-by: David Lawrence <david.lawrence@docker.com> (github: endophage)
This commit is contained in:
David Lawrence 2016-01-13 15:59:33 -08:00
parent 26d3f3f92b
commit a60f228189
1 changed files with 14 additions and 14 deletions

View File

@ -2407,7 +2407,7 @@ func TestRemoveDelegationErrorWritingChanges(t *testing.T) {
// a TUF server
func TestBootstrapClientBadURL(t *testing.T) {
tempBaseDir, err := ioutil.TempDir("", "notary-test-")
require.NoError(t, err, "failed to create a temporary directory: %s", err)
assert.NoError(t, err, "failed to create a temporary directory: %s", err)
repo, err := NewNotaryRepository(
tempBaseDir,
"testGun",
@ -2415,20 +2415,20 @@ func TestBootstrapClientBadURL(t *testing.T) {
http.DefaultTransport,
passphraseRetriever,
)
require.NoError(t, err, "error creating repo: %s", err)
assert.NoError(t, err, "error creating repo: %s", err)
c, err := repo.bootstrapClient(false)
require.Nil(t, c)
require.Error(t, err)
assert.Nil(t, c)
assert.Error(t, err)
c, err2 := repo.bootstrapClient(true)
require.Nil(t, c)
require.Error(t, err2)
assert.Nil(t, c)
assert.Error(t, err2)
// same error should be returned because we don't have local data
// and are requesting remote root regardless of checkInitialized
// value
require.EqualError(t, err, err2.Error())
assert.EqualError(t, err, err2.Error())
}
// TestBootstrapClientInvalidURL checks that bootstrapClient correctly
@ -2436,7 +2436,7 @@ func TestBootstrapClientBadURL(t *testing.T) {
// a TUF server
func TestBootstrapClientInvalidURL(t *testing.T) {
tempBaseDir, err := ioutil.TempDir("", "notary-test-")
require.NoError(t, err, "failed to create a temporary directory: %s", err)
assert.NoError(t, err, "failed to create a temporary directory: %s", err)
repo, err := NewNotaryRepository(
tempBaseDir,
"testGun",
@ -2444,18 +2444,18 @@ func TestBootstrapClientInvalidURL(t *testing.T) {
http.DefaultTransport,
passphraseRetriever,
)
require.NoError(t, err, "error creating repo: %s", err)
assert.NoError(t, err, "error creating repo: %s", err)
c, err := repo.bootstrapClient(false)
require.Nil(t, c)
require.Error(t, err)
assert.Nil(t, c)
assert.Error(t, err)
c, err2 := repo.bootstrapClient(true)
require.Nil(t, c)
require.Error(t, err2)
assert.Nil(t, c)
assert.Error(t, err2)
// same error should be returned because we don't have local data
// and are requesting remote root regardless of checkInitialized
// value
require.EqualError(t, err, err2.Error())
assert.EqualError(t, err, err2.Error())
}