из теста, паника: sql: соединение вернулось, что никогда не было - PullRequest
0 голосов
/ 29 августа 2018

Во время выполнения теста я получаю следующую панику:

panic: sql: connection returned that was never out

Тест

Фактический тест является частью набора тестов и определяется следующим образом:

func (suite *RowsSuite) TestReadFrom_SanityTest() {
    t := suite.T()

    rows := new(sql.Rows)
    *rows = suite.nonEmptyRows

    assert.True(t, rows.Next())
    // hit the function like it's supposed to be hit
    err := ReadFrom(rows,
        suite.fnStubFalse,
        suite.spots[0],
        suite.spots[1],
        suite.spots[2])

    // There better be no error
    if !assert.Nil(t, err) {
        t.Error(err.ToString(false))
    }

}

тестируемый код

Я отвечаю за тестирование этой функции:

// ReadFrom reads values from `rows` into `valuePlaceholders`
func ReadFrom(rows *sql.Rows,
    readerFunc func(rowIndex int, readValues ...interface{}) (bool, *errors.ErrorSt),
    valuePlaceholders ...interface{}) (err *errors.ErrorSt) {

    rowLine := 1
    for rows.Next() {
        if err := rows.Scan(valuePlaceholders...); err != nil {
            return errors.Database().AddDetails(err.Error(), valuePlaceholders)
        }
        if readerFunc != nil {
            skipRest, err := readerFunc(rowLine, valuePlaceholders...)
            if err != nil {
                return err
            }
            if skipRest {
                break
            }
        }
        rowLine++
    }
    if rowLine == 1 {
        return errors.Get(appErrors.ACNoRows)
    }
    return nil
}

Настройка

suite.fnStubFalse это просто заглушка функции, которая возвращает false,nil suite.spots это просто []*interface{} размера 3. Проще говоря, это три пятна до Scan до.

Остальные определения, относящиеся к тесту, определены в этом вспомогательном методе, который вызывается при настройке пакета:

func (suite *RowsSuite) setupRowStates() {
    // for throwing fatal error right away
    t := suite.T()
    // fire up the mock
    suite.db, suite.mock, suite.err = sqlmock.New()
    // if there's an error, fatally throw it right away!
    if !assert.Nilf(t,
        suite.err,
        "Error with initializing a stub database connection") {
        t.Fatal()
    }

    // define the possible expectant result sets
    noRows := sqlmock.NewRows(suite.columns)
    nonEmptyRows := sqlmock.NewRows(suite.columns).
        AddRow(381, "Beans", 1.59).
        AddRow(34981, "Frozen Pizza", 5.49)

    // define our sql behavior
    regex := "^SELECT (.+) FROM items$"
    sql := "SELECT (item_id, item_name, item_price) FROM items"

    specificRegex := "^SELECT (.+) FROM items (.+)$"
    specificSQL := `
    SELECT (item_id, item_name, item_price) FROM items i
    INNER JOIN stock s
    ON s.item_id = i.item_id
    WHERE TIME_TO_SEC(s.stock_time) > TIME_TO_SEC(NOW())`

    // setup general query to return non-empty rows
    suite.mock.ExpectQuery(regex).
        WillReturnRows(nonEmptyRows)
    // setup specific query to return empty rows
    suite.mock.ExpectQuery(specificRegex).
        WillReturnRows(noRows)

    // hit both queries right now and store the state of their
    //  return values, terminating right away on any errors
    var err error
    rows, err := suite.db.Query(sql)
    if err != nil {
        t.Fatal(err.Error())
    }
    suite.nonEmptyRows = *rows
    emptyRows, err := suite.db.Query(specificSQL)
    if err != nil {
        t.Fatal(err.Error())
    }
    suite.noRows = *emptyRows

}

Полная ошибка

Это чудовище:

Running tool: C:\Go\bin\go.exe test -timeout 30s ezsoft\apiserver_sdk\db -run ^TestRowsSuite$

panic: sql: connection returned that was never out

goroutine 22 [running]:
database/sql.(*DB).putConn(0xc04204d400, 0xc04212a080, 0x0, 0x0, 0xc04213de01)
    C:/Go/src/database/sql/sql.go:1158 +0x351
database/sql.(*driverConn).releaseConn(0xc04212a080, 0x0, 0x0)
    C:/Go/src/database/sql/sql.go:383 +0x53
database/sql.(*driverConn).(database/sql.releaseConn)-fm(0x0, 0x0)
    C:/Go/src/database/sql/sql.go:706 +0x45
database/sql.(*Rows).close(0xc04212a100, 0x899be0, 0xc042048380, 0x0, 0x0)
    C:/Go/src/database/sql/sql.go:2932 +0x159
database/sql.(*Rows).awaitDone(0xc04212a100, 0x89d260, 0xc042050b00, 0x0, 0x0)
    C:/Go/src/database/sql/sql.go:2588 +0x12f
created by database/sql.(*Rows).initContextClose
    C:/Go/src/database/sql/sql.go:2572 +0xa3
FAIL    ezsoft/apiserver_sdk/db 0.429s
Error: Tests failed.

Использованы сторонние библиотеки

Я использую testify и go-sqlmock. обручи в настройке.)

Понятия не имею, что послужило причиной этого сбоя. Когда я удаляю тест и запускаю сам набор, все работает

1 Ответ

0 голосов
/ 29 августа 2018

Оказывается, я обдумывал эту ситуацию.

Я исправил это, сохранив два состояния строки как указатели , а затем установив их в RowsSuite.SetupTest (на уровне теста, а не на уровне комплекта). Оттуда я просто выбрал одно из состояний в тесте, и мне было хорошо идти. А именно, вместо этого (что бы хорошо работать в C ++):

func (suite *RowsSuite) TestReadFrom_SanityTest() {
    t := suite.T()

    rows := new(sql.Rows)
    *rows = suite.nonEmptyRows

    assert.True(t, rows.Next())
    // hit the function like it's supposed to be hit
    err := ReadFrom(rows,
        suite.fnStubFalse,
        suite.spots[0],
        suite.spots[1],
        suite.spots[2])

    // There better be no error
    if !assert.Nil(t, err) {
        t.Error(err.ToString(false))
    }

}

Я сделал это:

func (suite *RowsSuite) TestReadFrom_SanityTest() {
    t := suite.T()

    rows := suite.nonEmptyRows

    // hit the function like it's supposed to be hit
    errSt := ReadFrom(rows,
        suite.fnStubFalse,
        suite.spots[0],
        suite.spots[1],
        suite.spots[2])

    // There better be no error
    if !assert.Nil(t, errSt) {
        t.Error(errSt.ToString(false))
    }

}

где nonEmptyRows - это одно из состояний, заданных непосредственно перед запуском теста

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...