Skip to content

Commit 2326ae4

Browse files
committed
Revert public static property change
Added overload to accept project name. Included param array for libraries. Some other clean up.
1 parent 6a2ea75 commit 2326ae4

File tree

1 file changed

+18
-12
lines changed

1 file changed

+18
-12
lines changed

RubberduckTests/Mocks/MockVbeBuilder.cs

Lines changed: 18 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ namespace RubberduckTests.Mocks
1717
[SuppressMessage("Microsoft.Design", "CA1001")] //CA1001 is complaining about RubberduckTests.Mocks.Windows, which doesn't need to be disposed in this context.
1818
public class MockVbeBuilder
1919
{
20-
public static string TestProjectName { set; get; } = "TestProject1";
21-
public static string TestModuleName { set; get; } = "TestModule1";
20+
public const string TestProjectName = "TestProject1";
21+
public const string TestModuleName = "TestModule1";
2222
private readonly Mock<IVBE> _vbe;
2323
private readonly Mock<IVbeEvents> _vbeEvents;
2424

@@ -177,47 +177,53 @@ public static Mock<IVBE> BuildFromSingleModule(string content, string name, Comp
177177
}
178178

179179
/// <summary>
180-
/// Builds a mock VBE containing multiple standard modules.
180+
/// Builds a mock VBE containing a single "TestProject1" and multiple standard modules.
181181
/// </summary>
182182
public static Mock<IVBE> BuildFromStdModules(params (string name, string content)[] modules)
183183
{
184184
return BuildFromModules(modules.Select(tpl => (tpl.name, tpl.content, ComponentType.StandardModule)));
185185
}
186186

187187
/// <summary>
188-
/// Builds a mock VBE containing one project with multiple modules.
188+
/// Builds a mock VBE containing a single "TestProject1" with multiple modules.
189189
/// </summary>
190190
public static Mock<IVBE> BuildFromModules(params (string name, string content, ComponentType componentType)[] modules)
191191
{
192192
return BuildFromModules((IEnumerable<(string name, string content, ComponentType componentType)>)modules);
193193
}
194194

195195
/// <summary>
196-
/// Builds a mock VBE containing one project with multiple modules.
196+
/// Builds a mock VBE containing a single "TestProject1" with multiple modules.
197197
/// </summary>
198198
public static Mock<IVBE> BuildFromModules(IEnumerable<(string name, string content, ComponentType componentType)> modules)
199199
=> BuildFromModules(modules, Enumerable.Empty<string>());
200200

201201
/// <summary>
202-
/// Builds a mock VBE containing one project with one module and one library.
202+
/// Builds a mock VBE containing a single "TestProject1" with one module and one or more libraries.
203203
/// </summary>
204-
public static Mock<IVBE> BuildFromModules((string name, string content, ComponentType componentType) module, string library)
205-
=> BuildFromModules(new(string, string, ComponentType)[] { module }, new string[] { library });
204+
public static Mock<IVBE> BuildFromModules((string name, string content, ComponentType componentType) module, params string[] libraries)
205+
=> BuildFromModules(new (string, string, ComponentType)[] { module }, libraries);
206206

207207
/// <summary>
208-
/// Builds a mock VBE containing one project with one module and multiple libraries.
208+
/// Builds a mock VBE containing a single "TestProject1" with one module and multiple libraries.
209209
/// </summary>
210210
public static Mock<IVBE> BuildFromModules((string name, string content, ComponentType componentType) module, IEnumerable<string> libraries)
211-
=> BuildFromModules(new(string, string, ComponentType)[] { module }, libraries);
211+
=> BuildFromModules(new (string, string, ComponentType)[] { module }, libraries);
212212

213213
/// <summary>
214-
/// Builds a mock VBE containing one project with multiple modules and libraries.
214+
/// Builds a mock VBE containing a single "TestProject1" with multiple modules and libraries.
215215
/// </summary>
216216
public static Mock<IVBE> BuildFromModules(IEnumerable<(string name, string content, ComponentType componentType)> modules, IEnumerable<string> libraryNames)
217+
=> BuildFromModules(TestProjectName, modules, libraryNames);
218+
219+
/// <summary>
220+
/// Builds a mock VBE containing one project with multiple modules and libraries.
221+
/// </summary>
222+
public static Mock<IVBE> BuildFromModules(string projectName, IEnumerable<(string name, string content, ComponentType componentType)> modules, IEnumerable<string> libraryNames)
217223
{
218224
var vbeBuilder = new MockVbeBuilder();
219225

220-
var builder = vbeBuilder.ProjectBuilder(TestProjectName, ProjectProtection.Unprotected);
226+
var builder = vbeBuilder.ProjectBuilder(projectName, ProjectProtection.Unprotected);
221227
foreach (var (name, content, componentType) in modules)
222228
{
223229
builder.AddComponent(name, componentType, content);

0 commit comments

Comments
 (0)