RepoMirrors
/
go-ceph
mirror of https://github.com/ceph/go-ceph
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

rados: naming conventions: fixes in test function TestMakeDeletePool 

Fix up variable names that don't meet Go standards.

Signed-off-by: John Mulligan <jmulligan@redhat.com>
This commit is contained in:
John Mulligan 2021-07-26 13:23:55 -04:00 committed by mergify[bot]
parent e3196e45e0
commit 810022af7b
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
rados/rados_test.go
View File

@ -289,19 +289,19 @@ func (suite *RadosTestSuite) TestMakeDeletePool() {
suite.SetupConnection() suite.SetupConnection()
// check that new pool name is unique // check that new pool name is unique
new_name := uuid.Must(uuid.NewV4()).String() newName := uuid.Must(uuid.NewV4()).String()
_, err := suite.conn.GetPoolByName(new_name) _, err := suite.conn.GetPoolByName(newName)
if err == nil { if err == nil {
suite.T().Error("Random pool name exists!") suite.T().Error("Random pool name exists!")
return return
} }
// create pool // create pool
err = suite.conn.MakePool(new_name) err = suite.conn.MakePool(newName)
assert.NoError(suite.T(), err) assert.NoError(suite.T(), err)
// verify that the new pool name exists // verify that the new pool name exists
pool, err := suite.conn.GetPoolByName(new_name) pool, err := suite.conn.GetPoolByName(newName)
assert.NoError(suite.T(), err) assert.NoError(suite.T(), err)
if pool == 0 { if pool == 0 {
@ -309,11 +309,11 @@ func (suite *RadosTestSuite) TestMakeDeletePool() {
} }
// delete the pool // delete the pool
err = suite.conn.DeletePool(new_name) err = suite.conn.DeletePool(newName)
assert.NoError(suite.T(), err) assert.NoError(suite.T(), err)
// verify that it is gone // verify that it is gone
pool, _ = suite.conn.GetPoolByName(new_name) pool, _ = suite.conn.GetPoolByName(newName)
if pool != 0 { if pool != 0 {
suite.T().Error("Deleted pool still exists") suite.T().Error("Deleted pool still exists")
} }