Skip to content

Commit e2a97de

Browse files
committed
Merge pull request #560 from hoanhtien/FixBraceCompletionTests
Fix BraceCompletionTests.
2 parents cac24a7 + f4a243a commit e2a97de

1 file changed

Lines changed: 11 additions & 9 deletions

File tree

Nodejs/Tests/Core.UI/BraceCompletion.cs

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -14,14 +14,12 @@
1414
//
1515
//*********************************************************//
1616

17-
using System;
18-
using System.IO;
19-
using Microsoft.VisualStudio.Language.Intellisense;
17+
using Microsoft.NodejsTools.Project;
2018
using Microsoft.VisualStudio.TestTools.UnitTesting;
2119
using TestUtilities;
2220
using TestUtilities.SharedProject;
23-
using TestUtilities.UI;
24-
21+
using TestUtilities.UI;
22+
2523
namespace Microsoft.Nodejs.Tests.UI {
2624
[TestClass]
2725
public class BraceCompletion : NodejsProjectTest {
@@ -48,7 +46,8 @@ public static void DoDeployment(TestContext context) {
4846
[HostType("VSTestHost")]
4947
public void BraceCompletionsBasic() {
5048
using (var solution = BasicProject.Generate().ToVs()) {
51-
var server = solution.OpenItem("Require", "SomeFolder", "baz.js");
49+
string folderName = NodejsFolderNode.AppendLabel("SomeFolder", FolderContentType.Node);
50+
var server = solution.OpenItem("Require", folderName, "baz.js");
5251

5352
// Test '('
5453
Keyboard.Type("(");
@@ -63,7 +62,8 @@ public void BraceCompletionsBasic() {
6362
[HostType("VSTestHost")]
6463
public void BraceCompletionsE2E() {
6564
using (var solution = BasicProject.Generate().ToVs()) {
66-
var server = solution.OpenItem("Require", "SomeFolder", "baz.js");
65+
string folderName = NodejsFolderNode.AppendLabel("SomeFolder", FolderContentType.Node);
66+
var server = solution.OpenItem("Require", folderName, "baz.js");
6767

6868
// Test '('
6969
Keyboard.Type("require(");
@@ -101,7 +101,8 @@ public void BraceCompletionsE2E() {
101101
[HostType("VSTestHost")]
102102
public void BraceCompletionDoesNotOccurInCommentsOrLiterals() {
103103
using (var solution = BasicProject.Generate().ToVs()) {
104-
var server = solution.OpenItem("Require", "SomeFolder", "baz.js");
104+
string folderName = NodejsFolderNode.AppendLabel("SomeFolder", FolderContentType.Node);
105+
var server = solution.OpenItem("Require", folderName, "baz.js");
105106

106107
// Verify that a comment gets no completions brace causes nothing if it has no start.
107108
Keyboard.Type("// '");
@@ -123,7 +124,8 @@ public void BraceCompletionDoesNotOccurInCommentsOrLiterals() {
123124
[HostType("VSTestHost")]
124125
public void BraceCompletionTerminatesOnEscapedQuoteInLiteralAndDoesntStartAgain() {
125126
using (var solution = BasicProject.Generate().ToVs()) {
126-
var server = solution.OpenItem("Require", "SomeFolder", "baz.js");
127+
string folderName = NodejsFolderNode.AppendLabel("SomeFolder", FolderContentType.Node);
128+
var server = solution.OpenItem("Require", folderName, "baz.js");
127129

128130
// Verify that when adding an escaped double quote to a literal that we don't try to start
129131
// a new brace completion on close of the string literal.

0 commit comments

Comments
 (0)