Merge pull request #25 from geoffreyvanwyk/support/test-each-moodle-version

chore(ci): Test each supported Moodle version

Author: geoffreyvanwyk

.github/workflows/build.yml

@@ -59,14 +59,18 @@ jobs:
     name: Molecule
     runs-on: ubuntu-22.04
     strategy:
-      max-parallel: 4
       matrix:
         molecule-scenario:
           - default
           - mariadb
         ubuntu-version:
           - ubuntu2004
           - ubuntu2204
+        moodle-version:
+          - MOODLE_39_STABLE
+          - MOODLE_400_STABLE
+          - MOODLE_401_STABLE
+          - MOODLE_402_STABLE
 
     steps:
       - name: Check out the codebase.
@@ -88,3 +92,4 @@ jobs:
           PY_COLORS: "1"
           ANSIBLE_FORCE_COLOR: "1"
           MOLECULE_DISTRO: ${{ matrix.ubuntu-version }}
+          MOLECULE_MOODLE_VERSION: ${{ matrix.moodle-version }}

molecule/default/molecule.yml

@@ -37,5 +37,6 @@ provisioner:
     host_vars:
       ansible_role_moodle_molecule_${MOLECULE_DISTRO:-ubuntu2204}:
         ansible_user: ubuntu
+        moodle_deploy_version: ${MOLECULE_MOODLE_VERSION:-MOODLE_402_STABLE}
 verifier:
   name: ansible

molecule/mariadb/molecule.yml

@@ -35,5 +35,6 @@ provisioner:
     host_vars:
       ansible_role_moodle_molecule_${MOLECULE_DISTRO:-ubuntu2204}:
         ansible_user: ubuntu
+        moodle_deploy_version: ${MOLECULE_MOODLE_VERSION:-MOODLE_402_STABLE}
 verifier:
   name: ansible

tasks/install_moodle.yml

@@ -65,7 +65,7 @@
       - --adminuser={{ moodle_admin_username }}
       - --adminpass={{ moodle_admin_password }}
       - --adminemail={{ moodle_admin_email }}
-      - --supportemail={{ moodle_support_email }}
+      - "{{ (moodle_deploy_version == 'MOODLE_311_STABLE' or moodle_deploy_version == 'MOODLE_400_STABLE') | ternary(omit, '--supportemail=' + moodle_support_email) }}"
   ansible.builtin.command:
     chdir: "{{ moodle_deploy_destination }}"
     argv: "{{ command_args | reject('equalto', omit) | list }}"

tasks/install_php.yml

@@ -20,6 +20,13 @@
 # @copyright  2023 Geoffrey Bernardo van Wyk (https://geoffreyvanwyk.dev)
 ##
 
+- name: Set PHP version
+  ansible.builtin.set_fact:
+    php_version: "{{
+           '8.1' if (moodle_deploy_version == 'MOODLE_402_STABLE') or (moodle_deploy_version == 'MOODLE_401_STABLE')
+      else '8.0' if (moodle_deploy_version == 'MOODLE_400_STABLE') or (moodle_deploy_version == 'MOODLE_311_STABLE')
+      else '7.4' }}"
+
 - name: Install PHP
   ansible.builtin.include_role:
     name: geoffreyvanwyk.php

vars/main.yml

@@ -21,11 +21,6 @@
 # @see        Understanding Variable Precedence {@link https://docs.ansible.com/ansible/latest/playbook_guide/playbooks_variables.html#understanding-variable-precedence}
 ##
 
-php_version: "{{
-       '8.1' if (moodle_deploy_version == 'MOODLE_402_STABLE') or (moodle_deploy_version == 'MOODLE_401_STABLE')
-  else '8.0' if (moodle_deploy_version == 'MOODLE_400_STABLE') or (moodle_deploy_version == 'MOODLE_311_STABLE')
-  else '7.4' }}"
-
 moodle_instance: "{{ moodle_web_domain + ('' if moodle_web_path == '' else '-' + moodle_web_path) }}"
 
 moodle_cfg_wwwroot: "{{ moodle_web_protocol }}://{{ moodle_web_domain }}/{{ moodle_web_path }}"