Add some more tests to cover more input classes

peterh / liner

Pure Go line editor with history, inspired by linenoise

MIT License

1.05k stars 132 forks source link

Add some more tests to cover more input classes #45

Closed mvo5 closed 9 years ago

mvo5 commented 9 years ago

I looked into using liner in one of my projects and I like it a lot. Playing around with it I noticed that some functions (or input classes) have no tests currently so I looked how much effort it is to add them. Looks like its not much, here is the result of that work. Feel free to ignore given that its just adding some tests.

peterh commented 9 years ago

Thanks for the pull request. More tests is good.

Unfortunately, this breaks tests on Windows.

.\line_test.go:102: s.next undefined (type State has no field or method next) .\line_test.go:102: undefined: nexter .\line_test.go:121: s.next undefined (type State has no field or method next) .\line_test.go:121: undefined: nexter

peterh commented 9 years ago

Closing. Please re-open if you modify the new tests to work on Windows.