From 1b132ddaa2734fc43a9c172407fc968cfeec4a24 Mon Sep 17 00:00:00 2001 From: Jake Leahy Date: Mon, 10 Jul 2023 16:34:10 +1000 Subject: Fix nimsuggest not showing suggestions for imported tuples (#22241) * Add tests Also test if exported all tuple fields works. This seems like a hacky solution so will try and dive further to find a better solution * Always suggest tuple fields if it passes the filter If the tuple we are accessing is in scope then all the fields will also be in scope * Update tests so line numbers are correct --- nimsuggest/tests/module_20265.nim | 6 ++++++ nimsuggest/tests/t20265_1.nim | 8 ++++++++ nimsuggest/tests/t20265_2.nim | 8 ++++++++ 3 files changed, 22 insertions(+) create mode 100644 nimsuggest/tests/module_20265.nim create mode 100644 nimsuggest/tests/t20265_1.nim create mode 100644 nimsuggest/tests/t20265_2.nim (limited to 'nimsuggest') diff --git a/nimsuggest/tests/module_20265.nim b/nimsuggest/tests/module_20265.nim new file mode 100644 index 000000000..24b7d10c9 --- /dev/null +++ b/nimsuggest/tests/module_20265.nim @@ -0,0 +1,6 @@ +type A* = tuple + a: int + b: int + +var x*: A = (a: 2, b: 10) +var y* = (a: 2, b: 10) diff --git a/nimsuggest/tests/t20265_1.nim b/nimsuggest/tests/t20265_1.nim new file mode 100644 index 000000000..553b3d545 --- /dev/null +++ b/nimsuggest/tests/t20265_1.nim @@ -0,0 +1,8 @@ +discard """ +$nimsuggest --tester $file +>sug $1 +sug;;skField;;a;;int;;*module_20265.nim;;6;;10;;"";;100;;None +sug;;skField;;b;;int;;*module_20265.nim;;6;;16;;"";;100;;None +""" +import module_20265 +y.#[!]# diff --git a/nimsuggest/tests/t20265_2.nim b/nimsuggest/tests/t20265_2.nim new file mode 100644 index 000000000..33edf2d9a --- /dev/null +++ b/nimsuggest/tests/t20265_2.nim @@ -0,0 +1,8 @@ +discard """ +$nimsuggest --tester $file +>sug $1 +sug;;skField;;a;;int;;*module_20265.nim;;2;;2;;"";;100;;None +sug;;skField;;b;;int;;*module_20265.nim;;3;;2;;"";;100;;None +""" +import module_20265 +x.#[!]# -- cgit 1.4.1-2-gfad0