Execute all tests on CI + warnings/errors cleanup

[zrzka]

Till #215 settles down (GH Actions). It's failing now, because #212 isn't merged yet. Once #212 will be merged, Linux & macOS builds will pass. I'll fix crossterm_winapi issues in another PR and then everything will pass.

Fixes #205

[TimonPost]

https://travis-ci.org/TimonPost/crossterm/jobs/584970770#L636

[zrzka]

Going to rebase and will apply #217 changes to this PR, because it's a nightmare to have all these small PRs waiting one for another to pass all the tests.

[zrzka]

@TimonPost can you please enable Auto cancel pull request builds?

[TimonPost]

I checked it, but it is already enabled

[zrzka]

Thanks for checking. I had 4 commits in the queue at one time and I thought that this is not enabled. It must have been Travis CI quirk.

[zrzka]

@TimonPost finally green.

Status

• Rust stable is passing
• Rust nightly is passing
  • Was failing on rustfmt, but it's executed on stable only now
• Zero warnings

Ignored tests & no_run examples

To achieve this, I had to ignore 5 tests and 1 example. All are cursor or resize related, which is expected.

Tests with #[ignore]

• crossterm_terminal/src/terminal/test.rs - winapi_tests::resize_winapi
• crossterm_terminal/src/terminal/test.rs - resize_ansi
• crossterm_winapi/src/console_mode.rs - test::set_get_mode
• crossterm_cursor/src/cursor/test.rs - reset_safe_ansi
• crossterm_cursor/src/cursor/test.rs - goto_ansi

Doc examples with no_run

• crossterm_terminal/src/terminal/terminal.rs - Terminal::exit

Proposal

• Merge this PR to execute everything on CI
• Keep the #[ignore] & no_run
  • Already marked with TODOs
  • Create issue for every single ignored test
    • Like this one

Anyone can run all (+ ignored) tests by adding --ignored (cargo test):

cargo test --all --exclude crossterm_winapi -- --ignored

At least, this whole PR is a progress in a way that everything is executed on the CI now, it passes and ignored tests can be executed locally till there will be a way to make them passing on CI as well.

[TimonPost]

Thanks for your contribution!

Sounds like a good idea, resizing caused weird behavior in the terminal as spoken of before. In my opinion, we should just create one issue for all failing tests, just to keep the things together and have less noise in the issue list.