chore: release v16

.github/workflows/ci.yml

@@ -112,7 +112,7 @@ jobs:
           FRAPPE_BRANCH: ${{ github.event.inputs.branch }}
 
       - name: Run Tests
-        run: cd ~/frappe-bench/ && bench --site test_site run-parallel-tests --app hrms --total-builds ${{ strategy.job-total }} --build-number ${{ matrix.container }}
+        run: cd ~/frappe-bench/ && bench --site test_site run-parallel-tests --app hrms --total-builds ${{ strategy.job-total }} --build-number ${{ matrix.container }} --lightmode
         env:
           TYPE: server
           CAPTURE_COVERAGE: ${{ github.event_name != 'pull_request' }}

.github/workflows/run-individual-tests.yml

@@ -0,0 +1,144 @@
+name: Individual
+
+on:
+  workflow_dispatch:
+
+concurrency:
+  group: server-individual-tests-lightmode-develop
+  cancel-in-progress: true
+
+permissions:
+  contents: read
+
+jobs:
+  discover:
+    runs-on: ubuntu-latest
+    outputs:
+      matrix: ${{ steps.set-matrix.outputs.matrix }}
+    steps:
+    - name: Clone
+      uses: actions/checkout@v6
+    - id: set-matrix
+      run: |
+        # Use grep and find to get the list of test files
+        matrix=$(find . -path '*/test_*.py' | xargs grep -l 'def test_' | sort | awk '{
+            # Remove ./ prefix, file extension, and replace / with .
+            gsub(/^\.\//, "", $0)
+            gsub(/\.py$/, "", $0)
+            gsub(/\//, ".", $0)
+            # Add to array
+            tests[NR] = $0
+        }
+        END {
+            # Start JSON array
+            printf "{\n  \"include\": [\n"
+            # Loop through array and create JSON objects
+            for (i=1; i<=NR; i++) {
+                printf "    {\"test\": \"%s\"}", tests[i]
+                if (i < NR) printf ","
+                printf "\n"
+            }
+            # Close JSON array
+            printf "  ]\n}"
+        }')
+
+        # Output the matrix
+        echo "matrix=$(echo "$matrix" | jq -c)" >> $GITHUB_OUTPUT
+
+        # For debugging (optional)
+        echo "Generated matrix:"
+        echo "$matrix"
+  test:
+    needs: discover
+    runs-on: ubuntu-latest
+    timeout-minutes: 60
+    env:
+      NODE_ENV: "production"
+
+    strategy:
+      fail-fast: false
+      matrix: ${{fromJson(needs.discover.outputs.matrix)}}
+      # max-parallel: 10
+
+    name: Test
+
+    services:
+      mysql:
+        image: mariadb:10.6
+        env:
+          MARIADB_ROOT_PASSWORD: 'root'
+        ports:
+          - 3306:3306
+        options: --health-cmd="mariadb-admin ping" --health-interval=5s --health-timeout=2s --health-retries=3
+
+    steps:
+      - name: Clone
+        uses: actions/checkout@v6
+
+      - name: Setup Python
+        uses: actions/setup-python@v6
+        with:
+          python-version: '3.14'
+
+      - name: Setup Node
+        uses: actions/setup-node@v6
+        with:
+          node-version: 24
+          check-latest: true
+
+      - name: Add to Hosts
+        run: echo "127.0.0.1 test_site" | sudo tee -a /etc/hosts
+
+      - name: Cache pip
+        uses: actions/cache@v4
+        with:
+          path: ~/.cache/pip
+          key: ${{ runner.os }}-pip-${{ hashFiles('**/*requirements.txt', '**/pyproject.toml') }}
+          restore-keys: |
+            ${{ runner.os }}-pip-
+            ${{ runner.os }}-
+
+      - name: Cache node modules
+        uses: actions/cache@v4
+        env:
+          cache-name: cache-node-modules
+        with:
+          path: ~/.npm
+          key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }}
+          restore-keys: |
+            ${{ runner.os }}-build-${{ env.cache-name }}-
+            ${{ runner.os }}-build-
+            ${{ runner.os }}-
+
+      - name: Get yarn cache directory path
+        id: yarn-cache-dir-path
+        run: echo "dir=$(yarn cache dir)" >> $GITHUB_OUTPUT
+
+      - uses: actions/cache@v4
+        id: yarn-cache
+        with:
+          path: ${{ steps.yarn-cache-dir-path.outputs.dir }}
+          key: ${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }}
+          restore-keys: |
+            ${{ runner.os }}-yarn-
+
+      - name: Install
+        run: bash ${GITHUB_WORKSPACE}/.github/helper/install.sh
+        env:
+          DB: mariadb
+          TYPE: server
+          FRAPPE_USER: ${{ github.event.inputs.user }}
+          FRAPPE_BRANCH: ${{ github.event.inputs.branch }}
+
+      - name: Run Tests
+        run: |
+          site_name=$(echo "${{matrix.test}}" | sed -e 's/.*\.\(test_.*$\)/\1/')
+          echo "$site_name"
+          mkdir ~/frappe-bench/sites/$site_name
+          cp -r "${GITHUB_WORKSPACE}/.github/helper/site_config.json" ~/frappe-bench/sites/$site_name/site_config.json
+          cd ~/frappe-bench/
+          bench --site $site_name reinstall --yes
+          bench --site $site_name install-app hrms
+          bench --site $site_name set-config allow_tests true
+          bench --site $site_name run-tests --module ${{ matrix.test }} --lightmode
+

hrms/controllers/tests/test_employee_reminders.py

@@ -4,7 +4,6 @@
 from datetime import timedelta
 
 import frappe
-from frappe.tests import IntegrationTestCase
 from frappe.utils import add_months, getdate
 
 from erpnext.setup.doctype.employee.test_employee import make_employee
@@ -15,16 +14,15 @@
 )
 from hrms.hr.doctype.hr_settings.hr_settings import set_proceed_with_frequency_change
 from hrms.hr.utils import get_holidays_for_employee
+from hrms.tests.utils import HRMSTestSuite
 
 
-class TestEmployeeReminders(IntegrationTestCase):
-	@classmethod
-	def setUpClass(cls):
-		super().setUpClass()
+class TestEmployeeReminders(HRMSTestSuite):
+	def setUp(self):
 		from erpnext.setup.doctype.holiday_list.test_holiday_list import make_holiday_list
 
 		# Create a test holiday list
-		test_holiday_dates = cls.get_test_holiday_dates()
+		test_holiday_dates = self.get_test_holiday_dates()
 		test_holiday_list1 = make_holiday_list(
 			"TestHolidayRemindersList",
 			holiday_dates=[
@@ -45,12 +43,12 @@ def setUpClass(cls):
 		create_holiday_list_assignment("Employee", test_employee.name, test_holiday_list1.name)
 
 		# Attach to class
-		cls.test_employee = test_employee
-		cls.test_holiday_dates = test_holiday_dates
+		self.test_employee = test_employee
+		self.test_holiday_dates = test_holiday_dates
 
 		# Employee without holidays in this month/week
 		test_employee_2 = make_employee("test@empwithoutholiday.io", company="_Test Company")
-		test_employee_2 = frappe.get_doc("Employee", test_employee_2)
+		test_employee_2 = frappe.get_doc("Employee", test_employee_2, company="_Test Company")
 
 		test_holiday_list2 = make_holiday_list(
 			"TestHolidayRemindersList2",
@@ -61,8 +59,12 @@ def setUpClass(cls):
 			to_date=add_months(getdate(), 2),
 		)
 		create_holiday_list_assignment("Employee", test_employee_2.name, test_holiday_list2.name)
-		cls.test_employee_2 = test_employee_2
-		cls.holiday_list_2 = test_holiday_list2
+		self.test_employee_2 = test_employee_2
+		self.holiday_list_2 = test_holiday_list2
+
+		# Clear Email Queue
+		frappe.db.sql("delete from `tabEmail Queue`")
+		frappe.db.sql("delete from `tabEmail Queue Recipient`")
 
 	@classmethod
 	def get_test_holiday_dates(cls):
@@ -76,11 +78,6 @@ def get_test_holiday_dates(cls):
 			today_date + timedelta(weeks=3),
 		]
 
-	def setUp(self):
-		# Clear Email Queue
-		frappe.db.sql("delete from `tabEmail Queue`")
-		frappe.db.sql("delete from `tabEmail Queue Recipient`")
-
 	def test_is_holiday(self):
 		from erpnext.setup.doctype.employee.employee import is_holiday
 

hrms/hr/doctype/appointment_letter/test_appointment_letter.py

@@ -2,8 +2,8 @@
 # See license.txt
 
 # import frappe
-from frappe.tests import IntegrationTestCase
+from hrms.tests.utils import HRMSTestSuite
 
 
-class TestAppointmentLetter(IntegrationTestCase):
+class TestAppointmentLetter(HRMSTestSuite):
 	pass

hrms/hr/doctype/appointment_letter_template/test_appointment_letter_template.py

@@ -2,8 +2,8 @@
 # See license.txt
 
 # import frappe
-from frappe.tests import IntegrationTestCase
+from hrms.tests.utils import HRMSTestSuite
 
 
-class TestAppointmentLetterTemplate(IntegrationTestCase):
+class TestAppointmentLetterTemplate(HRMSTestSuite):
 	pass

hrms/hr/doctype/appraisal/test_appraisal.py

@@ -2,7 +2,6 @@
 # See license.txt
 
 import frappe
-from frappe.tests import IntegrationTestCase
 
 from erpnext.setup.doctype.designation.test_designation import create_designation
 from erpnext.setup.doctype.employee.test_employee import make_employee
@@ -15,9 +14,10 @@
 )
 from hrms.hr.doctype.goal.test_goal import create_goal
 from hrms.tests.test_utils import create_company
+from hrms.tests.utils import HRMSTestSuite
 
 
-class TestAppraisal(IntegrationTestCase):
+class TestAppraisal(HRMSTestSuite):
 	def setUp(self):
 		frappe.db.delete("Goal")
 		frappe.db.delete("Appraisal")
@@ -29,7 +29,6 @@ def setUp(self):
 		engineer = create_designation(designation_name="Engineer")
 		engineer.appraisal_template = self.template.name
 		engineer.save()
-
 		self.employee1 = make_employee(
 			"test_appraisal1@example.com", company=self.company, designation="Engineer"
 		)
@@ -73,7 +72,7 @@ def test_final_score(self):
 		cycle.create_appraisals()
 		appraisal = self.setup_appraisal(cycle)
 
-		self.assertEqual(appraisal.final_score, 3.767)
+		self.assertEqual(appraisal.final_score, 3.77)
 
 	def test_final_score_using_formula(self):
 		cycle = create_appraisal_cycle(designation="Engineer", kra_evaluation_method="Manual Rating")
@@ -88,7 +87,7 @@ def test_final_score_using_formula(self):
 
 		appraisal = self.setup_appraisal(cycle)
 
-		self.assertEqual(appraisal.final_score, 3.767)
+		self.assertEqual(appraisal.final_score, 3.77)
 
 	def setup_appraisal(self, cycle):
 		appraisal = frappe.db.exists("Appraisal", {"appraisal_cycle": cycle.name, "employee": self.employee1})
@@ -106,7 +105,7 @@ def setup_appraisal(self, cycle):
 		appraisal.save()
 
 		# FEEDBACK SCORE
-		reviewer = make_employee("reviewer1@example.com", designation="Engineer")
+		reviewer = make_employee("reviewer1@example.com", designation="Engineer", company=self.company)
 		feedback = create_performance_feedback(
 			self.employee1,
 			reviewer,
@@ -160,8 +159,8 @@ def test_goal_score(self):
 		self.assertEqual(appraisal.appraisal_kra[1].goal_score, 35)
 
 		self.assertEqual(appraisal.goal_score_percentage, 38.75)
-		self.assertEqual(appraisal.total_score, 1.938)
-		self.assertEqual(appraisal.final_score, 0.646)
+		self.assertEqual(appraisal.total_score, 1.94)
+		self.assertEqual(appraisal.final_score, 0.65)
 
 	def test_goal_score_after_parent_goal_change(self):
 		"""

hrms/hr/doctype/appraisal_cycle/test_appraisal_cycle.py

@@ -2,16 +2,16 @@
 # See license.txt
 
 import frappe
-from frappe.tests import IntegrationTestCase
 
 from erpnext.setup.doctype.designation.test_designation import create_designation
 from erpnext.setup.doctype.employee.test_employee import make_employee
 
 from hrms.hr.doctype.appraisal_template.test_appraisal_template import create_appraisal_template
 from hrms.tests.test_utils import create_company
+from hrms.tests.utils import HRMSTestSuite
 
 
-class TestAppraisalCycle(IntegrationTestCase):
+class TestAppraisalCycle(HRMSTestSuite):
 	def setUp(self):
 		company = create_company("_Test Appraisal").name
 		self.template = create_appraisal_template()

hrms/hr/doctype/appraisal_template/test_appraisal_template.py

@@ -2,10 +2,11 @@
 # See license.txt
 
 import frappe
-from frappe.tests import IntegrationTestCase
 
+from hrms.tests.utils import HRMSTestSuite
 
-class TestAppraisalTemplate(IntegrationTestCase):
+
+class TestAppraisalTemplate(HRMSTestSuite):
 	def test_incorrect_weightage_allocation(self):
 		template = create_appraisal_template()
 		template.goals[1].per_weightage = 69.99