Skip to content

Commit

Permalink
Issue 636 - Fixed failing tests that were assuming they would receive…
Browse files Browse the repository at this point in the history
… no documentation target elements when the caret is placed on a word which is prospectively an identifier with the addition of LSPTargetElementEvaluator. The fix, in this case, is to move the caret in the test file contents not to be on a word that would be interpreted as a prospective identifier.
  • Loading branch information
SCWells72 committed Nov 30, 2024
1 parent 9845954 commit 59bea8e
Showing 1 changed file with 8 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -87,8 +87,8 @@ public void testHoverAsMarkdown() {
public void testNoHoverWithEmptyContents() {
assertHover("test.ts",
"""
const s = "";
s.c<caret>harAt(0);
const s = "";<caret>
s.charAt(0);
""",
"""
{
Expand All @@ -100,8 +100,8 @@ public void testNoHoverWithEmptyContents() {

assertHover("test.ts",
"""
const s = "";
s.c<caret>harAt(0);
const s = "";<caret>
s.charAt(0);
""",
"""
{
Expand All @@ -115,8 +115,8 @@ public void testNoHoverWithEmptyContents() {
public void testNoHoverWithInvalidMarkupContent() {
assertHover("test.ts",
"""
const s = "";
s.c<caret>harAt(0);
const s = "";<caret>
s.charAt(0);
""",
"""
{
Expand All @@ -143,8 +143,8 @@ public void testNoHoverWithInvalidMarkupContent() {
public void testNoHoverWithInvalidMarkedString() {
assertHover("test.ts",
"""
const s = "";
s.c<caret>harAt(0);
const s = "";<caret>
s.charAt(0);
""",
"""
{
Expand All @@ -160,5 +160,4 @@ public void testNoHoverWithInvalidMarkedString() {
null
);
}

}

0 comments on commit 59bea8e

Please sign in to comment.