Attention is currently required from: fixeria, laforge, osmith.

pespin uploaded patch set #5 to this change.

View Change

The following approvals got outdated and were removed: Code-Review+1 by laforge, Verified+1 by Jenkins Builder

Introduce tests/osmo-config-merge testsuite

This allows catching current bugs and possible regressions when fixing
them.

For instance, testB2.ok already shows a bug where lots of lines expected
to be merged under the "cs7 instance 0" end up put in an un expected place.
This happens due to comments being used by osmo-config-merge during tree
creation, and will be fixed in a follow-up commit.

Change-Id: Ia172d4c280f4bae1b411ad805cca02355ca7fcc9
---
M .gitignore
M configure.ac
M tests/Makefile.am
A tests/osmo-config-merge/Makefile.am
A tests/osmo-config-merge/atlocal.in
A tests/osmo-config-merge/testA.cfg
A tests/osmo-config-merge/testA.confmerge
A tests/osmo-config-merge/testA.ok
A tests/osmo-config-merge/testB.cfg
A tests/osmo-config-merge/testB.confmerge
A tests/osmo-config-merge/testB.ok
A tests/osmo-config-merge/testB2.cfg
A tests/osmo-config-merge/testB2.confmerge
A tests/osmo-config-merge/testB2.ok
A tests/osmo-config-merge/testsuite.at
15 files changed, 1,079 insertions(+), 0 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/95/40295/5

To view, visit change 40295. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newpatchset
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ia172d4c280f4bae1b411ad805cca02355ca7fcc9
Gerrit-Change-Number: 40295
Gerrit-PatchSet: 5
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Attention: osmith <osmith@sysmocom.de>
Gerrit-Attention: laforge <laforge@osmocom.org>
Gerrit-Attention: fixeria <vyanitskiy@sysmocom.de>