HpToolsLauncher/Runners/AlmTestSetsRunner.cs
Consider simplifying this complex logical expression. Open
Open
if (string.IsNullOrWhiteSpace(qcServerUrl)
|| (string.IsNullOrWhiteSpace(qcLogin) && !SSOEnabled)
|| string.IsNullOrWhiteSpace(qcDomain)
|| string.IsNullOrWhiteSpace(qcProject)
|| (SSOEnabled && (string.IsNullOrWhiteSpace(qcClientID)
Avoid deeply nested control flow statements. Open
Open
if (!tListIndexName.ToLower().Contains(filterByName.ToLower()) &&
!tListIndexTestName.ToLower().Contains(filterByName.ToLower()))
{
testList.Remove(index);
Avoid deeply nested control flow statements. Open
Open
if (prevRunId == -1)
{
Console.WriteLine("No test runs exist for this test");
continue;
}
Avoid deeply nested control flow statements. Open
Open
if (folder != null)
folderName = folder.Name.Replace(".", "_");
Avoid deeply nested control flow statements. Open
Open
if (testIndex == -1)
{
Console.WriteLine("No test index exist for this test");
}
Avoid too many return
statements within this method. Open
Open
return runDesc;
Avoid too many return
statements within this method. Open
Open
return false;
Avoid too many return
statements within this method. Open
Open
return TestState.Unknown;
Avoid too many return
statements within this method. Open
Open
return null;