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.

Sign up for GitHub

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

Jump to bottom

DataCollector check with only Uri in Runsettings #2177

Merged
merged 10 commits into from Oct 1, 2019
Merged

DataCollector check with only Uri in Runsettings #2177

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
bb8e581
Removed friendly name null check
Sep 17, 2019
f663737
removed redundant code and added tests for the datacollection manager
Sep 18, 2019
91e2ea1
added warning message if friendlyname is incorrect
Sep 18, 2019
ac485d5
Given prority to Uri
Sep 25, 2019
dc7ed89
changed naming in resources
Sep 26, 2019
6d6cd6e
added tests and verified behavior
Sep 30, 2019
59d58c7
Merge branch 'master' into FriendlyNameErrorInDataCollectors
hvinett Oct 1, 2019
58a0a41
changed member variables to functional variables
Oct 1, 2019
01907a4
Merge branch 'FriendlyNameErrorInDataCollectors' of https://github.co…
Oct 1, 2019
0463219
added datacollectorsettings as member varaible again
Oct 1, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
6 changes: 5 additions & 1 deletion src/Microsoft.TestPlatform.Common/DataCollection/DataCollectionManager.cs
View file
Open in desktop
Expand Up @@ -422,7 +422,11 @@ private void LoadAndInitialize(DataCollectorSettings dataCollectorSettings, stri
// Look up the extension and initialize it if one is found.
var extensionManager = this.DataCollectorExtensionManager;
var dataCollectorUri = string.Empty;
this.TryGetUriFromFriendlyName(dataCollectorSettings.FriendlyName, out dataCollectorUri);

if (!this.TryGetUriFromFriendlyName(dataCollectorSettings.FriendlyName, out dataCollectorUri))
hvinett marked this conversation as resolved.
Show resolved Hide resolved
{
dataCollectorUri = dataCollectorSettings.Uri.ToString();
}

DataCollector dataCollector = null;
if (!string.IsNullOrWhiteSpace(dataCollectorUri))
Expand Down
6 changes: 5 additions & 1 deletion src/Microsoft.TestPlatform.CrossPlatEngine/DataCollection/DataCollectionLauncherFactory.cs
View file
Open in desktop
Expand Up @@ -28,7 +28,11 @@ internal static IDataCollectionLauncher GetDataCollectorLauncher(IProcessHelper
var dataCollectionRunSettings = XmlRunSettingsUtilities.GetDataCollectionRunSettings(settingsXml);
foreach (var dataCollectorSettings in dataCollectionRunSettings.DataCollectorSettingsList)
{
if (dataCollectorSettings.FriendlyName.ToLower().Equals("event log"))
if (dataCollectorSettings.FriendlyName!=null && dataCollectorSettings.FriendlyName.ToLower().Equals("event log"))
hvinett marked this conversation as resolved.
Show resolved Hide resolved
{
return new DefaultDataCollectionLauncher();
}
if(dataCollectorSettings.Uri!=null && dataCollectorSettings.Uri.ToString().ToLower().Equals(@"datacollector://Microsoft/EventLog/2.0"))
{
return new DefaultDataCollectionLauncher();
}
Expand Down
6 changes: 0 additions & 6 deletions src/Microsoft.TestPlatform.ObjectModel/DataCollector/DataCollectorSettings.cs
View file
Open in desktop
Expand Up @@ -201,12 +201,6 @@ internal static DataCollectorSettings FromXml(XmlReader reader)

}

if (string.IsNullOrWhiteSpace(settings.FriendlyName))
{
throw new SettingsException(
String.Format(CultureInfo.CurrentCulture, Resources.Resources.MissingDataCollectorAttributes, "FriendlyName"));
}

reader.Read();
if (!empty)
{
Expand Down