Hi! Thank you for your review!
> HI! What makes you think we need additional tab completion tests at all? > If we consult src/bin/psql changes history [0], it can be easily seen > that not every change to tab-complete.in.c is made with a new test. > This is (my own understanding, I may be wrong) because we only need to > test tab-complete patterns that are functionally different, not mirror > all cases from tab-complete.in.c in the TAP test. I agree that not every tab-complete change requires tests, and we don't need to mirror all cases from tab-complete.in.c. I also think tests that provide little benefit compared to their maintenance cost can be omitted. However, since this patch implements a previously unimplemented feature (removing a XXX TODO comment), I believe the primary test cases should be kept to verify the core functionality and prevent regressions. That said, I can reduce the test cases. The current 4 tests could be reduced to 2: One for USING <TAB> (core functionality: table completion) One for USING <table> <TAB> (keyword completion after table) WDYT? Regards, Tatsuya Kawata
