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

Unit testing output #40

Closed
polluks opened this issue Jun 13, 2023 · 5 comments
Closed

Unit testing output #40

polluks opened this issue Jun 13, 2023 · 5 comments

Comments

@polluks
Copy link
Contributor

polluks commented Jun 13, 2023

Maybe some false positive of neatvi

test/e00.sh: OK
test/e01.sh: OK
test/e02.sh: OK
test/e03.sh: Failed
--- /tmp/.neatvi1	2023-06-14 00:16:03
+++ /tmp/.neatvi2	2023-06-14 00:16:03
@@ -1 +1,2 @@
 abc
+def
OK
test/e04.sh: OK
test/e05.sh: OK
test/e06.sh: Failed
--- /tmp/.neatvi1	2023-06-14 00:16:03
+++ /tmp/.neatvi2	2023-06-14 00:16:03
@@ -1,5 +1,3 @@
 abc
-abc
 def
-def
 ghi
OK
test/e07.sh: OK
test/e08.sh: OK
test/e09.sh: OK
test/e0a.sh: OK
test/e0b.sh: OK
test/e0c.sh: Failed
--- /tmp/.neatvi1	2023-06-14 00:16:03
+++ /tmp/.neatvi2	2023-06-14 00:16:03
@@ -1,9 +1,5 @@
 1
 2
-x
-x
 3
 4
-x
-x
 5
OK
test/e0d.sh: Failed
--- /tmp/.neatvi1	2023-06-14 00:16:03
+++ /tmp/.neatvi2	2023-06-14 00:16:03
@@ -1,7 +1,5 @@
 1
-x
 2
 3
-x
 4
 5
OK
test/e0e.sh: OK
test/e0f.sh: OK
test/e10.sh: Failed
--- /tmp/.neatvi1	2023-06-14 00:16:03
+++ /tmp/.neatvi2	2023-06-14 00:16:03
@@ -1,2 +1,2 @@
-xyz
-xyz
+abc
+def
OK
test/e11.sh: OK
test/e12.sh: Failed
--- /tmp/.neatvi1	2023-06-14 00:16:03
+++ /tmp/.neatvi2	2023-06-14 00:16:03
@@ -1,3 +1,3 @@
 a
-z
+b
 c
OK
test/v00.sh: OK
test/v01.sh: OK
test/v02.sh: OK
test/v03.sh: OK
test/v04.sh: OK
test/v05.sh: OK
test/v06.sh: OK
test/v07.sh: OK
test/v08.sh: OK
test/v09.sh: OK
test/v0a.sh: OK
test/v0b.sh: OK
test/v0c.sh: OK
test/v0d.sh: OK
test/v0e.sh: OK
test/v0f.sh: OK
test/v10.sh: OK
test/v11.sh: OK
test/v12.sh: OK
test/v13.sh: Failed
--- /tmp/.neatvi1	2023-06-14 00:16:03
+++ /tmp/.neatvi2	2023-06-14 00:16:03
@@ -1,3 +1,3 @@
 ABC
-21DEF
-GHI
+1DEF
+2GHI
OK
test/v14.sh: OK
test/v15.sh: OK
test/v16.sh: OK
test/v17.sh: OK
test/v18.sh: OK
test/v19.sh: OK
test/v1a.sh: OK
test/v1b.sh: OK
test/v1c.sh: OK
test/v1d.sh: OK
test/v1e.sh: OK
test/v1f.sh: OK
test/v20.sh: Failed
--- /tmp/.neatvi1	2023-06-14 00:16:03
+++ /tmp/.neatvi2	2023-06-14 00:16:03
@@ -1,2 +1 @@
 aa
-bb
OK
test/v21.sh: OK
test/v22.sh: OK
test/v23.sh: Failed
--- /tmp/.neatvi1	2023-06-14 00:16:03
+++ /tmp/.neatvi2	2023-06-14 00:16:03
@@ -1,5 +1,5 @@
+
 a
 b
 c
-
 a
OK
@kyx0r
Copy link
Owner

kyx0r commented Jun 14, 2023

Thanks, i'll have a look just in case anything is off. Keep in mind some of it may fail intentionally.

@kyx0r
Copy link
Owner

kyx0r commented Jun 14, 2023

e03.sh - fails because :undo is not a command, use :u instead
e06.sh - fails because :yank and :put is not a command, use :ya and :p instead
e0c.sh - fails because :put is not a command, use :p instead
e0d.sh - fails because :put is not a command, use :p instead
e10.sh - fails because :%s/(abc|def)/xyz/ is not properly escaped, use :%s/(abc\\|def)/xyz/ instead
e12.sh - fails because :e +2 range in e command is not implemented, use e $|2 instead. (actually this is some new feature im not aware of)
v13.sh - fails because :set is not a command, use :se instead
v20.sh - something wrong with :g/a/.g/b/d command, improper escape or too many arguments
v23.sh - 1G!}sort fails, this actually may be some old neatvi bug that got carried over or miscalculation of range into pipe command.

@polluks
Copy link
Contributor Author

polluks commented Jun 14, 2023

well done!

@polluks polluks closed this as completed Jun 14, 2023
@kyx0r
Copy link
Owner

kyx0r commented Jun 14, 2023

v12.sh - will not implement
v20.sh - The parser does 1 extra iteration, needs more investigation.
v23.sh - backported.

@kyx0r
Copy link
Owner

kyx0r commented Jun 15, 2023

v20.sh should be fixed now.

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

No branches or pull requests

2 participants