fixed jvm.additional overriden by docker-entrypoint

[bfeshti]

• Fixed the issue where the custom values set by users for dbms.jvm.additional were being overriden by docker-entrypoint.sh.
• Added a small test for it.

[jennyowen]

Looks good, and if the tests pass I approve.
I do wonder if we have the same problem with NEO4J_wrapper_java_additional?

[bfeshti]

I think we have the same problem with it also,I will fix it. Thanks.

[bfeshti]

Found a new more reliable way to fix it that also includes the NEO4J_wrapper_java_additional

[jennyowen]

Overall this looks good! I only have a minor edit request.

[jennyowen]

this comment applies to all entrypoints.
I've a couple of notes about making this easier for us to maintain.

1. In general if you're doing something in your which looks crazy, it's a good idea to leave a comment as to why you're doing it that way instead of another way. This is because when you come back to your code in 6 months time, you don't waste a load of time wondering whyyyy you did this crazy looking thing (and usually reaching the same conclusion you did originally). What I'm saying is write a comment here about why you're using sed/grep to edit the conf :)

2. Could you make the indentation consistent? It's easier to read that way, and extra indentation makes me worry that some code got deleted around this.

[jennyowen]

great!