Come testare il panico?


92

Attualmente sto riflettendo su come scrivere test che controllino se un dato pezzo di codice è andato nel panico? So che Go usa recoverper prendere il panico, ma a differenza del codice Java, non puoi davvero specificare quale codice dovrebbe essere saltato in caso di panico o cosa hai. Quindi se ho una funzione:

func f(t *testing.T) {
    defer func() {
        if r := recover(); r != nil {
            fmt.Println("Recovered in f", r)
        }
    }()
    OtherFunctionThatPanics()
    t.Errorf("The code did not panic")
}

Non posso davvero dire se OtherFunctionThatPanicssiamo stati presi dal panico e ci siamo ripresi, o se la funzione non è andata affatto nel panico. Come faccio a specificare quale codice saltare se non c'è panico e quale codice eseguire se c'è panico? Come posso verificare se c'è stato qualche panico da cui ci siamo ripresi?

Risposte:


107

testingnon ha realmente il concetto di "successo", ma solo fallimento. Quindi il tuo codice sopra è corretto. Potresti trovare questo stile leggermente più chiaro, ma fondamentalmente è la stessa cosa.

func TestPanic(t *testing.T) {
    defer func() {
        if r := recover(); r == nil {
            t.Errorf("The code did not panic")
        }
    }()

    // The following is the code under test
    OtherFunctionThatPanics()
}

In genere trovo testingdi essere abbastanza debole. Potresti essere interessato a motori di test più potenti come Ginkgo . Anche se non desideri il sistema Ginkgo completo, puoi utilizzare solo la sua libreria di matcher, Gomega , che può essere utilizzata insieme a testing. Gomega include matcher come:

Expect(OtherFunctionThatPanics).To(Panic())

Puoi anche concludere il controllo antipanico in una semplice funzione:

func TestPanic(t *testing.T) {
    assertPanic(t, OtherFunctionThatPanics)
}

func assertPanic(t *testing.T, f func()) {
    defer func() {
        if r := recover(); r == nil {
            t.Errorf("The code did not panic")
        }
    }()
    f()
}

@IgorMikushkin in Go 1.11, utilizzando il primo modulo descritto da Rob Napier funziona effettivamente per la copertura.
MGF

C'è qualche motivo che usi r := recover(); r == nile non solo recover() == nil?
Duncan Jones

@ DuncanJones Non proprio in questo caso. È un tipico modello Go per rendere disponibile l'errore nel blocco, quindi probabilmente era abitudine per l'OP scriverlo in quel modo (e ho portato avanti il ​​suo codice), ma in questo caso non è effettivamente utilizzato.
Rob Napier,

46

Se usi testify / assert , allora è una battuta:

func TestOtherFunctionThatPanics(t *testing.T) {
  assert.Panics(t, OtherFunctionThatPanics, "The code did not panic")
}

Oppure, se hai OtherFunctionThatPanicsuna firma diversa da func():

func TestOtherFunctionThatPanics(t *testing.T) {
  assert.Panics(t, func() { OtherFunctionThatPanics(arg) }, "The code did not panic")
}

Se non hai ancora provato a testimoniare, dai un'occhiata anche a testimoniare / simulare . Affermazioni e derisioni semplicissime.


7

Quando si esegue il loop su più casi di test, sceglierei qualcosa del genere:

package main

import (
    "reflect"
    "testing"
)


func TestYourFunc(t *testing.T) {
    type args struct {
        arg1 int
        arg2 int
        arg3 int
    }
    tests := []struct {
        name      string
        args      args
        want      []int
        wantErr   bool
        wantPanic bool
    }{
        //TODO: write test cases
    }
    for _, tt := range tests {
        t.Run(tt.name, func(t *testing.T) {
            defer func() {
                r := recover()
                if (r != nil) != tt.wantPanic {
                    t.Errorf("SequenceInt() recover = %v, wantPanic = %v", r, tt.wantPanic)
                }
            }()
            got, err := YourFunc(tt.args.arg1, tt.args.arg2, tt.args.arg3)
            if (err != nil) != tt.wantErr {
                t.Errorf("YourFunc() error = %v, wantErr %v", err, tt.wantErr)
                return
            }
            if !reflect.DeepEqual(got, tt.want) {
                t.Errorf("YourFunc() = %v, want %v", got, tt.want)
            }
        })
    }
}

Vai al parco giochi


5

Modo succinto

Per me, la soluzione seguente è di facile lettura e mostra a un manutentore il flusso di codice naturale sotto test.

func TestPanic(t *testing.T) {
    // No need to check whether `recover()` is nil. Just turn off the panic.
    defer func() { recover() }()

    OtherFunctionThatPanics()

    // Never reaches here if `OtherFunctionThatPanics` panics.
    t.Errorf("did not panic")
}

Per una soluzione più generale, puoi anche farlo in questo modo:

func TestPanic(t *testing.T) {
    shouldPanic(t, OtherFunctionThatPanics)
}

func shouldPanic(t *testing.T, f func()) {
    defer func() { recover() }()
    f()
    t.Errorf("should have panicked")
}

4

Quando è necessario controllare il contenuto del panico, è possibile digitare il valore recuperato:

func TestIsAheadComparedToPanicsWithDifferingStreams(t *testing.T) {
    defer func() {
        err := recover().(error)

        if err.Error() != "Cursor: cannot compare cursors from different streams" {
            t.Fatalf("Wrong panic message: %s", err.Error())
        }
    }()

    c1 := CursorFromserializedMust("/foo:0:0")
    c2 := CursorFromserializedMust("/bar:0:0")

    // must panic
    c1.IsAheadComparedTo(c2)
}

Se il codice che stai testando non va in panico O in panico con un errore O in panico con il messaggio di errore che ti aspetti, il test fallirà (che è quello che vorresti).


1
È più affidabile l'asserzione del tipo su un tipo di errore specifico (ad esempio os.SyscallError) che confrontare i messaggi di errore, che possono cambiare (ad esempio) da una versione di Go a quella successiva.
Michael

+ Michael Aug, questo è probabilmente l'approccio migliore, per quando è disponibile un tipo specifico.
joonas.fi

3

Nel tuo caso puoi fare:

func f(t *testing.T) {
    recovered := func() (r bool) {
        defer func() {
            if r := recover(); r != nil {
                r = true
            }
        }()
        OtherFunctionThatPanics()
        // NOT BE EXECUTED IF PANICS
        // ....
    }
    if ! recovered() {
        t.Errorf("The code did not panic")

        // EXECUTED IF PANICS
        // ....
    }
}

Come funzione di router antipanico generico, funzionerà anche:

https://github.com/7d4b9/recover

package recover

func Recovered(IfPanic, Else func(), Then func(recover interface{})) (recoverElse interface{}) {
    defer func() {
        if r := recover(); r != nil {
            {
                // EXECUTED IF PANICS
                if Then != nil {
                    Then(r)
                }
            }
        }
    }()

    IfPanic()

    {
        // NOT BE EXECUTED IF PANICS
        if Else != nil {
            defer func() {
                recoverElse = recover()
            }()
            Else()
        }
    }
    return
}

var testError = errors.New("expected error")

func TestRecover(t *testing.T) {
    Recovered(
        func() {
            panic(testError)
        },
        func() {
            t.Errorf("The code did not panic")
        },
        func(r interface{}) {
            if err := r.(error); err != nil {
                assert.Error(t, testError, err)
                return
            }
            t.Errorf("The code did an unexpected panic")
        },
    )
}

0

È possibile verificare quale funzione panica dando un input al panico

package main

import "fmt"

func explode() {
    // Cause a panic.
    panic("WRONG")
}

func explode1() {
    // Cause a panic.
    panic("WRONG1")
}

func main() {
    // Handle errors in defer func with recover.
    defer func() {
        if r := recover(); r != nil {
            var ok bool
            err, ok := r.(error)
            if !ok {
                err = fmt.Errorf("pkg: %v", r)
                fmt.Println(err)
            }
        }

    }()
    // These causes an error. change between these
    explode()
    //explode1()

    fmt.Println("Everything fine")

}

http://play.golang.org/p/ORWBqmPSVA

Utilizzando il nostro sito, riconosci di aver letto e compreso le nostre Informativa sui cookie e Informativa sulla privacy.
Licensed under cc by-sa 3.0 with attribution required.