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.

Sign up for GitHub

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

repl: add completion preview #30907

Closed
wants to merge 12 commits into from
Closed

repl: add completion preview #30907

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
19dac98
repl: remove dead code
BridgeAR Dec 10, 2019
c65f3f1
repl: simplify repl autocompletion
BridgeAR Dec 10, 2019
ec47482
repl: simplify code
BridgeAR Dec 10, 2019
43ae689
readline: update ansi-regex
BridgeAR Dec 11, 2019
1426e6c
repl,readline: refactor common code
BridgeAR Dec 11, 2019
42d3590
repl,readline: refactor for simplicity
BridgeAR Dec 11, 2019
ca14259
repl: improve completion
BridgeAR Dec 11, 2019
1cbef42
repl: add completion preview
BridgeAR Dec 11, 2019
707cba4
repl: fix preview bug in case of long lines
BridgeAR Dec 11, 2019
ca4c894
test: add multiple repl preview tests
BridgeAR Dec 13, 2019
0cd4153
fixup: repl: add completion preview
BridgeAR Dec 14, 2019
2138fd7
fixup: test: add multiple repl preview tests
BridgeAR Dec 14, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fixup: test: add multiple repl preview tests
  • Loading branch information
@BridgeAR
BridgeAR committed Dec 14, 2019
commit 2138fd790661d284ba8de799b4c10f27c6092beb
14 changes: 9 additions & 5 deletions test/parallel/test-repl-history-navigation.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,9 +116,9 @@ const tests = [
ENTER,
'veryLongName'.repeat(30),
ENTER,
`${'\u001b[90m \u001b[39m'.repeat(230)} fun`,
`${'\x1B[90m \x1B[39m'.repeat(235)} fun`,
ENTER,
`${' '.repeat(245)} fun`,
`${' '.repeat(236)} fun`,
ENTER
],
expected: [],
Expand All @@ -137,13 +137,17 @@ const tests = [
],
expected: [
prompt,
`${prompt}${' '.repeat(245)} fun`,
`${prompt}${' '.repeat(230)} fun`,
// This exceeds the maximum columns (250):
// Whitespace + prompt + ' // '.length + 'function'.length
// 236 + 2 + 4 + 8
`${prompt}${' '.repeat(236)} fun`,
`${prompt}${' '.repeat(235)} fun`,
' // ction',
' // ction',
`${prompt}${'veryLongName'.repeat(30)}`,
`${prompt}e`,
'\n// RangeError: visible'
'\n// RangeError: visible',
prompt
],
clean: true
},
Expand Down