Skip to content

Master do not use alignment for simple tests and more new/old config fixes

This is a follow up to !39908 (closed)

Here I try to handle the alignment errors correctly. Also, many little fixes for differences in old/new configuration (see !39908 (closed) for details about how these were found).

Running full tests, as I'm nervous about breaking something in the trigger.

I have to say I'm not very happy with how I "solved" the problem of failing tests with forceDisableAlignment ... but this is a little tricky. Most likely the 'correct' solution is to separate the conditions algs from the base GeoModel config, so workflows that need this conditions data will need to explicitly configure it. This will touch a lot of code though, so for the moment this seems like a pragmatic fix to try to get this working.

Edited by Edward Moyse

Merge request reports