Server log report extension

[xsmrcek]

Server log report extension updated

[xsmrcek]

If anyone want to start review feel free :) Except serverLogXmlRep.xslt, I will work on it a little more.

[AmyKosinova]

This sanity check occurs quite often, consider implementing Null Object Design Pattern.

[xsmrcek]

Now it occurs only once and throws assertion error.

[AmyKosinova]

From javadoc of this class:

This class implements steps for testing ssh

All these methods @AfterScenario has nothing to do with testing ssh, rather reporting content of what ssh handler delivers. Therefore this place for such behavior looks inappropriate to me.

[xsmrcek]

I moved a lot of stuff elsewhere :)

[AmyKosinova]

If you insist on lambdas: logContents.forEach((MultiKey k, String v) -> {
Otherwise plain old java would still love you: for (Map.Entry<MultiKey<? extends MultiKey>, String> multiKeyStringEntry : logContents.entrySet()) {
++multiple occurrence

[xsmrcek]

I am using forEach on entrySet now :)

[paveljandejsek]

Changed - even on your sshTemplates merger ultra code 😛

[AmyKosinova]

sshTemplatesArray.entrySet().stream()
            .collect(Collectors.toMap(Map.Entry::getKey, e -> e.getValue().stream().flatMap(Arrays::stream).collect(Collectors.toList())))
            .forEach((k, v) -> sshTemplates.merge(k, v, (v1, v2) -> {
                v2.addAll(v1);
                return v2;
            }));

[xsmrcek]

Thank you

[xsmrcek]

v1, v2 >> list

[paveljandejsek]

Variables renamed + changed to use biconsumer foreach.

[paveljandejsek]

sshTemplatesArray.entrySet().stream() .collect(Collectors.toMap(Map.Entry::getKey, e -> e.getValue().stream().flatMap(Arrays::stream).collect(Collectors.toList()))) .forEach((k, v) -> sshTemplates.merge(k, v, (v1, v2) -> { v2.addAll(v1); return v2; }));

@AmyKosinova