Fix[IT]: also execute tests that do not have a consistency mark

Signed-off-by: JEAN-LOUIS "JL" LEROY <[email protected]>

Author: jll63

.github/workflows/build.yaml

@@ -163,7 +163,7 @@ jobs:
       matrix:
         mode: ["legacy_mode", "fsm_mode"]
         cluster: ["single", "multi"]
-        consistency: ["eventual_consistency", "strong_consistency"]
+        consistency: ["eventual_consistency", "strong_consistency", "(not eventual_consistency and not strong_consistency)"]
       fail-fast: false
     runs-on: ubuntu-latest
     needs: build_ubuntu
@@ -255,7 +255,7 @@ jobs:
             deps
             /opt/bb/include
           key: cache-${{ github.sha }}
-      
+
       - name: Setup core_pattern
         run: |
           sudo mkdir /cores
@@ -267,7 +267,7 @@ jobs:
           pip install -r ${{ github.workspace }}/src/python/requirements.txt
           cd src/python
           python3 -m blazingmq.dev.fuzztest --broker-dir ${{ github.workspace }}/build/blazingmq/src/applications/bmqbrkr --request ${{ matrix.request }}
-      
+
       - name: Print core information
         if: failure()
         run: |

src/integration-tests/test_admin_command_routing.py

@@ -32,7 +32,7 @@
 import multiprocessing.pool
 
 
-def test_primary_rerouting(multi_node: Cluster, domain_urls: tc.DomainUrls) -> None:
+def test_primary_rerouting(multi_node: Cluster) -> None:
     """
     Test: commands intended only for primary node are automatically routed to
           primary node and response it sent back when executed from non-primary
@@ -52,8 +52,6 @@ def test_primary_rerouting(multi_node: Cluster, domain_urls: tc.DomainUrls) -> N
     # TODO Skip admin command routing tests until admin command routing is re-enabled
     return
 
-    du = domain_urls
-
     admin = AdminClient()
 
     # find the first node which is not a known leader
@@ -115,7 +113,7 @@ def test_primary_rerouting(multi_node: Cluster, domain_urls: tc.DomainUrls) -> N
     admin.stop()
 
 
-def test_cluster_rerouting(multi_node: Cluster, domain_urls: tc.DomainUrls) -> None:
+def test_cluster_rerouting(multi_node: Cluster) -> None:
     """
     Test: commands intended for cluster are routed to all nodes in the cluster
           regardless of the node the command is initially sent to
@@ -136,8 +134,6 @@ def test_cluster_rerouting(multi_node: Cluster, domain_urls: tc.DomainUrls) -> N
     # TODO Skip admin command routing tests until admin command routing is re-enabled
     return
 
-    du = domain_urls
-
     admin = AdminClient()
 
     node = multi_node.nodes()[0]
@@ -190,7 +186,7 @@ def test_cluster_rerouting(multi_node: Cluster, domain_urls: tc.DomainUrls) -> N
     admin.stop()
 
 
-def test_multi_response_encoding(multi_node: Cluster, domain_urls: tc.DomainUrls):
+def test_multi_response_encoding(multi_node: Cluster):
     """
     Test: JSON encoding options work with multiple responses (when routing to
           multiple nodes)
@@ -209,8 +205,6 @@ def test_multi_response_encoding(multi_node: Cluster, domain_urls: tc.DomainUrls
     # TODO Skip admin command routing tests until admin command routing is re-enabled
     return
 
-    du = domain_urls
-
     def is_compact(json_str: str) -> bool:
         return "    " not in json_str