apply geometry checks to all stages in elementsFullHistory requests

[tyrasd]

Description

Updates addEntityToOutput method to check geometry filters not only in the deprecated "simpleFeatureTypes" code path.

Corresponding issue

Closes #109

Checklist

• My code follows the code-style rules, and I have checked on the static analyses
• I have commented my code
• I have written javadoc (required for public methods)
• I have added sufficient unit and API tests
• I have made corresponding changes to the documentation
• I have updated the CHANGELOG.md
• I have adjusted the examples in the check-ohsome-api script or created an issue in the corresponding repository. More Information here.

[tyrasd]

strictly speaking, cd57e8a is unrelated to #109, but I thought it might still be nice to have all defined tests to be executed on jenkins 😉

[KowatschFab]

strictly speaking, cd57e8a is unrelated to #109, but I thought it might still be nice to have all defined tests to be executed on jenkins 😉

You mean jenkins only executes tests that have explicitly Test written in the method name..? :O

[tyrasd]

You mean jenkins only executes tests that have explicitly Test written in the method name..? :O

Oh, forget what I wrote. I miscounted the tests that should be executed and falsely remembered that test methods should have the same naming scheme as the test classes (see https://stackoverflow.com/a/6178629/1627467). Actually, this doesn't apply to the method names, so it was already working fine and didn't need a fix. I would say we still keep the changes from cd57e8a (now b3d5112), since it makes all test method have a uniform naming scheme.

[KowatschFab]

Please check the comments.

[KowatschFab]

You mean jenkins only executes tests that have explicitly Test written in the method name..? :O

Oh, forget what I wrote. I miscounted the tests that should be executed and falsely remembered that test methods should have the same naming scheme as the test classes (see https://stackoverflow.com/a/6178629/1627467). Actually, this doesn't apply to the method names, so it was already working fine and didn't need a fix. I would say we still keep the changes from cd57e8a (now b3d5112), since it makes all test method have a uniform naming scheme.

Ok I see.. Since it's only important for the classes to follow the specific format, my idea was to remove then the superfluous Test at the end of every method. That's why I did not add it to the new tests. And I did not want to change all test names within that branch. Should have created an issue for that I guess so it would not get forgotten..