public void RealFile_Load_CheckRulesAndParametersDictionarySerialization()
{
// Arrange
var testLogger = new TestLogger(logToConsole: true);
var dir = CreateTestSpecificDirectory();
var filePath1 = Path.Combine(dir, "settings.json");
const string validSettingsData = @"{
'sonarlint.rules': {
'xxx:xxx': {
'Level': 'On',
'Parameters': {
'key1': 'value1'
}
},
'xxx:yyy': {
'level': 'off',
'parameters': null
},
'xxx:zzz': {
'level': 'off'
}
}
}";
File.WriteAllText(filePath1, validSettingsData);
var testSubject = new RulesSettingsSerializer(new FileSystem(), testLogger);
// 1. Load from disc
var loadedSettings = testSubject.SafeLoad(filePath1);
loadedSettings.Should().NotBeNull();
loadedSettings.Rules.Should().NotBeNull();
loadedSettings.Rules.Count.Should().Be(3);
// Check loaded data
// 1. Rules dictionary: lookup should be case-insensitive
loadedSettings.Rules.TryGetValue("xxx:xxx", out var rulesConfig).Should().BeTrue();
loadedSettings.Rules.TryGetValue("XXX:XXX", out rulesConfig).Should().BeTrue();
// 2. Parameters dictionary: lookup should be case-insensitive
rulesConfig.Parameters.TryGetValue("key1", out var paramValue).Should().BeTrue();
// BUG - we want it to be case-insenstive, but it isn't
// rulesConfig.Parameters.TryGetValue("KEY1", out paramValue).Should().BeTrue();
paramValue.Should().Be("value1");
// 3. Parameters dictionary: explicitly null in file -> null on deserialization
loadedSettings.Rules["xxx:yyy"].Parameters.Should().BeNull();
// 4. Parameters dictionary: missing parameters setting in dictionary -> null on deserialization
loadedSettings.Rules["xxx:zzz"].Parameters.Should().BeNull();
}
private UserSettings SafeLoadUserSettings(string filePath, ILogger logger)
{
var settings = serializer.SafeLoad(filePath);
if (settings == null)
{
logger.WriteLine(AnalysisStrings.Settings_UsingDefaultSettings);
settings = new RulesSettings();
}
return(new UserSettings(settings));
}
private RulesSettings FindConnectedModeSettings(string languageKey, BindingConfiguration binding)
{
var language = Language.GetLanguageFromLanguageKey(languageKey);
Debug.Assert(language != null, $"Unknown language key: {languageKey}");
if (language != null)
{
var filePath = solutionInfoProvider.CalculateSolutionSonarQubeRuleSetFilePath(binding.Project.ProjectKey, language, binding.Mode);
var settings = serializer.SafeLoad(filePath);
return(settings);
}
return(null);
}
private RulesSettings FindConnectedModeSettings(string languageKey, BindingConfiguration binding)
{
var language = Language.GetLanguageFromLanguageKey(languageKey);
Debug.Assert(language != null, $"Unknown language key: {languageKey}");
if (language != null)
{
var filePath = binding.BuildPathUnderConfigDirectory(language.FileSuffixAndExtension);
var settings = serializer.SafeLoad(filePath);
return(settings);
}
return(null);
}
public void Load_CriticalError_IsNotSquashed()
{
// Arrange
var fileSystemMock = new Mock <IFileSystem>();
fileSystemMock.Setup(x => x.File.Exists("settings.file")).Returns(true);
fileSystemMock.Setup(x => x.File.ReadAllText("settings.file")).Throws(new System.StackOverflowException("critical custom error message"));
var logger = new TestLogger(logToConsole: true);
var testSubject = new RulesSettingsSerializer(fileSystemMock.Object, logger);
// Act
Action act = () => testSubject.SafeLoad("settings.file");
// Assert
act.Should().ThrowExactly <StackOverflowException>().And.Message.Should().Be("critical custom error message");
logger.AssertPartialOutputStringDoesNotExist("critical custom error message");
}
public void Load_NonCriticalError_IsSquashed_AndNullReturned()
{
// Arrange
var fileSystemMock = new Mock <IFileSystem>();
fileSystemMock.Setup(x => x.File.Exists("settings.file")).Returns(true);
fileSystemMock.Setup(x => x.File.ReadAllText("settings.file")).Throws(new System.InvalidOperationException("custom error message"));
var logger = new TestLogger(logToConsole: true);
var testSubject = new RulesSettingsSerializer(fileSystemMock.Object, logger);
// Act
var result = testSubject.SafeLoad("settings.file");
// Assert
result.Should().BeNull();
logger.AssertPartialOutputStringExists("custom error message");
}
public void Load_MissingFile_NullReturned()
{
// Arrange
var fileSystemMock = new Mock <IFileSystem>();
fileSystemMock.Setup(x => x.File.Exists("settings.file")).Returns(false);
var logger = new TestLogger(logToConsole: true);
var testSubject = new RulesSettingsSerializer(fileSystemMock.Object, logger);
// Act
var result = testSubject.SafeLoad("settings.file");
// Assert
result.Should().BeNull();
fileSystemMock.Verify(x => x.File.ReadAllText(It.IsAny <string>()), Times.Never);
}
private RulesSettings LoadSettings(string filePath)
{
var serializer = new RulesSettingsSerializer(new FileSystem(), new TestLogger());
return(serializer.SafeLoad(filePath));
}
public void RealFile_RoundTripLoadAndSave_WithFirstCapsPropertyNames()
{
// Note: the JSON serializer is configured to save property names
// to lower-case by default i.e. "level", "severity" etc.
// However, it should tolerate loading files with first-caps names
// e.g. "Level", "Parameters" etc.
// This snippet uses upper-case "Level", "Severity" and "Parameters"
// to check they are processed correctly.
// Arrange
var testLogger = new TestLogger(logToConsole: true);
var dir = CreateTestSpecificDirectory();
var filePath1 = Path.Combine(dir, "settings.json");
var filePath2 = Path.Combine(dir, "settings.json.txt");
const string validSettingsData = @"{
'UnknownData' : 'will be dropped on save',
'sonarlint.rules': {
'typescript:S2685': {
'Level': 'On',
'Parameters': {
'key1': 'value1'
},
'Severity': 'Critical'
},
'xxx:yyy': {
'level': 'off',
'parameters': {
'key2': 'value2',
'key3': 'value3'
},
'severity': 'Blocker'
}
},
'More UnknownData' : 'will also be dropped on save',
}";
File.WriteAllText(filePath1, validSettingsData);
var testSubject = new RulesSettingsSerializer(new FileSystem(), testLogger);
// 1. Load from disc
var loadedSettings = testSubject.SafeLoad(filePath1);
loadedSettings.Should().NotBeNull();
loadedSettings.Rules.Should().NotBeNull();
loadedSettings.Rules.Count.Should().Be(2);
// Check loaded data
loadedSettings.Rules["typescript:S2685"].Level.Should().Be(RuleLevel.On);
loadedSettings.Rules["xxx:yyy"].Level.Should().Be(RuleLevel.Off);
loadedSettings.Rules["typescript:S2685"].Parameters.Should().NotBeNull();
loadedSettings.Rules["typescript:S2685"].Parameters["key1"].Should().Be("value1");
loadedSettings.Rules["xxx:yyy"].Parameters.Should().NotBeNull();
loadedSettings.Rules["xxx:yyy"].Parameters["key2"].Should().Be("value2");
loadedSettings.Rules["xxx:yyy"].Parameters["key3"].Should().Be("value3");
loadedSettings.Rules["typescript:S2685"].Severity.Should().Be(IssueSeverity.Critical);
loadedSettings.Rules["xxx:yyy"].Severity.Should().Be(IssueSeverity.Blocker);
// 2. Save and reload
testSubject.SafeSave(filePath2, loadedSettings);
File.Exists(filePath2).Should().BeTrue();
var reloadedSettings = testSubject.SafeLoad(filePath2);
TestContext.AddResultFile(filePath2);
reloadedSettings.Should().NotBeNull();
reloadedSettings.Rules.Should().NotBeNull();
reloadedSettings.Rules.Count.Should().Be(2);
// Check loaded data
reloadedSettings.Rules["typescript:S2685"].Level.Should().Be(RuleLevel.On);
reloadedSettings.Rules["xxx:yyy"].Level.Should().Be(RuleLevel.Off);
loadedSettings.Rules["typescript:S2685"].Parameters.Should().NotBeNull();
loadedSettings.Rules["typescript:S2685"].Parameters["key1"].Should().Be("value1");
loadedSettings.Rules["xxx:yyy"].Parameters.Should().NotBeNull();
loadedSettings.Rules["xxx:yyy"].Parameters["key2"].Should().Be("value2");
loadedSettings.Rules["xxx:yyy"].Parameters["key3"].Should().Be("value3");
}
public void RealFile_RoundTripSaveAndLoad()
{
// Arrange
var testLogger = new TestLogger(logToConsole: true);
var dir = CreateTestSpecificDirectory();
var filePath = Path.Combine(dir, "settings.txt");
var settings = new RulesSettings
{
Rules = new Dictionary <string, RuleConfig>
{
{ "repo1:key1", new RuleConfig {
Level = RuleLevel.Off, Severity = IssueSeverity.Major
} },
{ "repo1:key2", new RuleConfig {
Level = RuleLevel.On, Severity = IssueSeverity.Blocker
} },
{ "repox:keyy",
new RuleConfig
{
Level = RuleLevel.On,
Parameters = new Dictionary <string, string>
{
{ "key1", "value1" },
{ "key2", "value2" }
}
} }
}
};
var testSubject = new RulesSettingsSerializer(new FileSystem(), testLogger);
// Act: save and reload
testSubject.SafeSave(filePath, settings);
File.Exists(filePath).Should().BeTrue();
var reloadedSettings = testSubject.SafeLoad(filePath);
TestContext.AddResultFile(filePath);
reloadedSettings.Should().NotBeNull();
reloadedSettings.Rules.Should().NotBeNull();
reloadedSettings.Rules.Count.Should().Be(3);
// Check loaded data
reloadedSettings.Rules["repo1:key1"].Level.Should().Be(RuleLevel.Off);
reloadedSettings.Rules["repo1:key2"].Level.Should().Be(RuleLevel.On);
reloadedSettings.Rules["repox:keyy"].Level.Should().Be(RuleLevel.On);
reloadedSettings.Rules["repo1:key1"].Severity.Should().Be(IssueSeverity.Major);
reloadedSettings.Rules["repo1:key2"].Severity.Should().Be(IssueSeverity.Blocker);
reloadedSettings.Rules["repox:keyy"].Severity.Should().BeNull();
reloadedSettings.Rules["repo1:key1"].Parameters.Should().BeNull();
reloadedSettings.Rules["repo1:key2"].Parameters.Should().BeNull();
var rulexParams = reloadedSettings.Rules["repox:keyy"].Parameters;
rulexParams.Should().NotBeNull();
rulexParams.Keys.Should().BeEquivalentTo("key1", "key2");
rulexParams["key1"].Should().Be("value1");
rulexParams["key2"].Should().Be("value2");
}