fix: remove unsafe assertion (#225)

* fix: move assertion separately * fix: fix tests * fix: remove problematic assert
lowlydba · Feb 8, 2024 · c73b88e · c73b88e
1 parent f2f1fd2
commit c73b88e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/ansible-test-windows.yml b/.github/workflows/ansible-test-windows.yml
@@ -40,7 +40,7 @@ jobs:
 
   integration:
     runs-on: ${{ matrix.os }}
-    name: I (Ⓐ${{ matrix.ansible }}+${{ matrix.os }}|grp${{ matrix.group }})
+    name: I (Ⓐ${{ matrix.ansible }}+win-2022|grp${{ matrix.group }})
     strategy:
       fail-fast: false
       matrix:

diff --git a/tests/integration/targets/win_ag_replica/tasks/main.yml b/tests/integration/targets/win_ag_replica/tasks/main.yml
@@ -56,6 +56,10 @@
       lowlydba.sqlserver.availability_group:
       register: ag
 
+    - name: Set ag SqlInstance fact
+      set_fact:
+        ag_sql_instance: "{{ ag.data.SqlInstance }}"
+
     - name: Set replica
       lowlydba.sqlserver.ag_replica:
         session_timeout: 20
@@ -72,7 +76,6 @@
           - result.data.AvailabilityMode == availability_mode
           - result.data.SessionTimeout == 20
           - result.data.FailoverMode == failover_mode
-          - result.data.EndpointUrl == "TCP://{{ ag.data.SqlInstance }}:5022"
           - result is changed
 
   always: