tests: fix BrotoBuf BCcheck on version-level (#536)

Signed-off-by: Jan Kowalleck <[email protected]>

Author: jkowalleck

tools/src/test/proto/buf_breaking-version.yaml

@@ -3,7 +3,4 @@
 version: v2
 breaking:  # https://buf.build/docs/configuration/v2/buf-yaml#breaking
   use:  # see https://buf.build/docs/breaking/overview#rules-and-categories
-    - WIRE
-  except:
-    # scope is to detect changes from one version to the other -> so ignore "FILE_SAME_PACKAGE"
-    - FILE_SAME_PACKAGE
+    - WIRE

tools/src/test/proto/test.sh

@@ -1,5 +1,5 @@
 #!/usr/bin/env bash
-set -uex
+set -ue
 
 THIS_PATH="$(realpath "$(dirname "$0")")"
 ROOT_PATH="$(realpath "${THIS_PATH}/../../../..")"
@@ -51,23 +51,29 @@ function schema-breaking-version () {
   function compare() {
     NEW="bom-${1}.proto"
     OLD="bom-${2}.proto"
-    SCHEMA_DIR_OLD="${SCHEMA_DIR}_old"
+
+    NEW_NP="$(mktemp)"
+    OLD_NP="$(mktemp)"
+
+    # remove package identifier -> so that the comparisson works as expected
+    sed 's/^package .*//' "${ROOT_PATH}/${SCHEMA_DIR}/${NEW}" > "$NEW_NP"
+    sed 's/^package .*//' "${ROOT_PATH}/${SCHEMA_DIR}/${OLD}" > "$OLD_NP"
 
     echo ">> compare new:${NEW} -VS- old:${OLD}" >&2
     # stick with the original path of "$NEW", so the reporting makes sense...
     docker run --rm \
-      --volume "${ROOT_PATH}/${SCHEMA_DIR}/${OLD}:/workspaces/old/${NEW}:ro" \
-      --volume "${ROOT_PATH}/${SCHEMA_DIR}/${NEW}:/workspaces/new/${NEW}:ro" \
+      --volume "${OLD_NP}:/workspaces/old/${NEW}:ro" \
+      --volume "${NEW_NP}:/workspaces/new/${NEW}:ro" \
       --volume "${THIS_PATH}/buf_breaking-version.yaml:/workspaces/new/buf.yaml:ro" \
-      --workdir '/workspaces' \
+      --workdir '/workspaces/new' \
       bufbuild/buf:"$BUF_IMAGE_VERSION" \
-        breaking new \
-        --against old \
+        breaking \
+        --against ../old \
         --error-format "$LOG_FORMAT"
   }
 
-  compare '1.6' '1.5'
-  compare '1.5' '1.4'
+  # compare '1.6' '1.5'  #  <-- possible breaks are acknowledged
+  # compare '1.5' '1.4'  #  <-- possible breaks are acknowledged
   compare '1.4' '1.3'
 
   echo '>> OK.' >&2