Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

respect a changed g:go_test_show_name #1641

Merged
merged 1 commit into from
Jan 13, 2018

Conversation

bhcleek
Copy link
Collaborator

@bhcleek bhcleek commented Jan 9, 2018

Respect the current value of g:go_test_show_name on :GoTest runs after
the first one by invalidating the cached errorformat value when the
g:go_test_show_name value is not the same as it was when the errorformat
value was last cached.

Only show the test name for failed tests instead of including the
'--- FAIL: ' prefix and the duration suffix when g:go_test_show_name=1.

Resolve a TODO item in the test errorformat so that the regular
expression used for test errorformat values is more precise.

Remove a bit of testing code that is no longer needed; it was probably
rendered obsolete with some recent changes to how the errorformat value
is created.

Respect the current value of g:go_test_show_name on :GoTest runs after
the first one by invalidating the cached errorformat value when the
g:go_test_show_name value is not the same as it was when the errorformat
value was last cached.

Only show the test name for failed tests instead of including the
'--- FAIL: ' prefix and the duration suffix when g:go_test_show_name=1.

Resolve a TODO item in the test errorformat so that the regular
expression used for test errorformat values is more precise.

Remove a bit of testing code that is no longer needed; it was probably
rendered obsolete with some recent changes to how the errorformat value
is created.
@codecov-io
Copy link

Codecov Report

Merging #1641 into master will increase coverage by 1.52%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1641      +/-   ##
=========================================
+ Coverage   17.77%   19.3%   +1.52%     
=========================================
  Files          57      53       -4     
  Lines        4770    4159     -611     
=========================================
- Hits          848     803      -45     
+ Misses       3922    3356     -566
Flag Coverage Δ
#nvim 14.52% <0%> (+1.25%) ⬆️
#vim74 17.69% <100%> (+1.32%) ⬆️
#vim80 18.46% <100%> (+1.42%) ⬆️
Impacted Files Coverage Δ
autoload/go/test.vim 70.65% <100%> (-1.78%) ⬇️
plugin/go.vim 29.26% <0%> (-4.26%) ⬇️
autoload/go/util.vim 60.51% <0%> (-1.37%) ⬇️
ftplugin/go/commands.vim 0% <0%> (ø) ⬆️
syntax/godebugstacktrace.vim
autoload/go/debug.vim
syntax/godebugvariables.vim
syntax/godebugoutput.vim

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 3ceecc2...31e6c28. Read the comment docs.

@fatih
Copy link
Owner

fatih commented Jan 13, 2018

lgtm

@fatih fatih merged commit 1f675f6 into fatih:master Jan 13, 2018
@bhcleek bhcleek deleted the fix-show-test-name-toggle branch January 13, 2018 09:01
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