saveourtool / save-cli

Universal test framework for cli tools [mainly for code analyzers and compilers]
MIT License
42 stars 4 forks source link

NPE in merging configs save.toml files #418

Open Cheshiriks opened 2 years ago

Cheshiriks commented 2 years ago

When you run the fix, plugin NPE is thrown when you try to merge two save.toml files

11111111
nulls commented 2 years ago

can you please provide a code snippet to reproduce this exception?

Also, SaveProperties was refactored, please check this with latest main

petertrr commented 2 years ago

@Cheshiriks which version did you get it with? You should post the whole log (or even attach a log file), because it contains this information in the very beginning

Cheshiriks commented 2 years ago

@Cheshiriks which version did you get it with? You should post the whole log (or even attach a log file), because it contains this information in the very beginning

Yes. Added all log. Tested on latest version save

Cheshiriks commented 2 years ago

can you please provide a code snippet to reproduce this exception?

Also, SaveProperties was refactored, please check this with latest main

Yes.

Parent save.toml

[general]
    execCmd = "java -jar ktlint --disabled_rules=standard -R diktat.jar"
    description = "22 test for diktat (linter and formatter for Kotlin)"
    language = "Kotlin"
    # this is the default value, you don't need to add it explicitly, but can be useful, if you have different pattern:
    expectedWarningsPattern = "// ;warn:?(.*):(\\d*): (.+)"
    timeOutMillis = 30000

child

[general]
    tags = ["fix"]
    description = "Test autofixing of issues discovered by diKTat"

[fix]
    execFlags = "-F"
nulls commented 2 years ago

@Cheshiriks how did you run save-cli (cli args)?

petertrr commented 2 years ago

@nulls it's on the updated screenshot in the first post: save.exe ..\save\test --log all

nulls commented 2 years ago

@nulls it's on the updated screenshot in the first post: save.exe ..\save\test --log all

yeah, thanks. found it!