language/lexer: fixes panic on empty blockstrings

graphql-go / graphql

An implementation of GraphQL for Go / Golang

MIT License

9.91k stars 839 forks source link

language/lexer: fixes panic on empty blockstrings #456

Closed chris-ramon closed 5 years ago

chris-ramon commented 5 years ago

Overview

Built on top of: https://github.com/graphql-go/graphql/pull/441 — Thanks a lot @miiton :+1:

Test plan

go test ./...

coveralls commented 5 years ago

Coverage remained the same at 92.17% when pulling 750c09f8aed34ccf2cacedccb4cf2a68ece8e317 on fix-panic-blockstring into c13c47a830350f168f8085333860ff2198ff4130 on master.

chris-ramon commented 5 years ago

If you could take a look to this PR when you get a chance :smiley: — A PR review approval is appreciate it.

/cc: @miiton @Fontinalis