Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ichizok
Copy link
Contributor

@ichizok ichizok commented Sep 1, 2017

  • Add Test_peek_and_get_char to test_timers
  • Add test_feedinput to feed data to inbuf

I confirmed this test fails with before 8.0.1020.

https://ci.appveyor.com/project/ichizok/vim/build/428
https://travis-ci.org/ichizok/vim/builds/270689255

@codecov-io
Copy link

Codecov Report

Merging #2046 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2046      +/-   ##
==========================================
+ Coverage   74.59%   74.59%   +<.01%     
==========================================
  Files          77       77              
  Lines      126168   126173       +5     
  Branches    28243    28244       +1     
==========================================
+ Hits        94113    94123      +10     
+ Misses      32055    32050       -5
Impacted Files Coverage Δ
src/ui.c 45.03% <ø> (+0.32%) ⬆️
src/evalfunc.c 82.6% <100%> (-0.01%) ⬇️
src/if_xcmdsrv.c 85.18% <0%> (-1.3%) ⬇️
src/gui_gtk_x11.c 47.33% <0%> (-0.06%) ⬇️
src/channel.c 79.43% <0%> (-0.05%) ⬇️
src/ex_cmds2.c 80.08% <0%> (+0.05%) ⬆️
src/gui.c 45.72% <0%> (+0.05%) ⬆️
src/term.c 50.66% <0%> (+0.05%) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9e8dcf9...e35397c. Read the comment docs.

@brammool
Copy link
Contributor

brammool commented Sep 1, 2017 via email

@ichizok
Copy link
Contributor Author

ichizok commented Sep 2, 2017

Sorry, I forgot to add document. I updated commit.

The test should check if the test_feedinput function exists. It is
possible to have the timers feature but not the clientserver feature.
It's safer anyway.

test_feedinput works only when inbuf works, and inbuf works only when
read_from_input_buf is used, i.e. +unix or GUI runnning, therefore I think it is enough to check
has('unix') and has('gui_running').

@brammool brammool closed this in 5e80de3 Sep 3, 2017
@ichizok ichizok deleted the test/feed_input branch September 3, 2017 14:06
adizero pushed a commit to adizero/vim that referenced this pull request May 19, 2018
Problem:    No test for what 8.0.1020 fixes.
Solution:   Add test_feedinput().  Add a test. (Ozaki Kiichi, closes vim#2046)
janlazo added a commit to janlazo/neovim that referenced this pull request Jul 10, 2018
Problem:    No test for what 8.0.1020 fixes.
Solution:   Add test_feedinput().  Add a test. (Ozaki Kiichi, closes vim/vim#2046)
vim/vim@5e80de3
janlazo added a commit to janlazo/neovim that referenced this pull request Jul 20, 2018
Problem:    No test for what 8.0.1020 fixes.
Solution:   Add test_feedinput().  Add a test. (Ozaki Kiichi, closes vim/vim#2046)
vim/vim@5e80de3
janlazo added a commit to janlazo/neovim that referenced this pull request Aug 22, 2018
Problem:    No test for what 8.0.1020 fixes.
Solution:   Add test_feedinput().  Add a test. (Ozaki Kiichi, closes vim/vim#2046)
vim/vim@5e80de3
janlazo added a commit to janlazo/neovim that referenced this pull request Aug 24, 2018
Problem:    No test for what 8.0.1020 fixes.
Solution:   Add test_feedinput().  Add a test. (Ozaki Kiichi, closes vim/vim#2046)
vim/vim@5e80de3
janlazo added a commit to janlazo/neovim that referenced this pull request Sep 4, 2018
Problem:    No test for what 8.0.1020 fixes.
Solution:   Add test_feedinput().  Add a test. (Ozaki Kiichi, closes vim/vim#2046)
vim/vim@5e80de3
timeyyy pushed a commit to timeyyy/neovim that referenced this pull request Sep 23, 2018
Problem:    No test for what 8.0.1020 fixes.
Solution:   Add test_feedinput().  Add a test. (Ozaki Kiichi, closes vim/vim#2046)
vim/vim@5e80de3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants