refactor: extract job filtering from job list

src/pages/JobList/JobList.js

@@ -7,6 +7,7 @@ import { JobTable } from '../../components/JobTable'
 import { LinkButton } from '../../components/LinkButton'
 import { Spinner } from '../../components/Spinner'
 import styles from './JobList.module.css'
+import filterJobs from './filter-jobs'
 
 const infoLink =
     'https://docs.dhis2.org/en/use/user-guides/dhis-core-version-236/maintaining-the-system/scheduling.html'
@@ -30,16 +31,8 @@ const JobList = () => {
         )
     }
 
-    // Filter jobs by the current jobFilter
-    const applyJobFilter = (job) =>
-        job.name.toLowerCase().includes(jobFilter.toLowerCase())
-
-    // Filter jobs depending on the current showSystemJobs
-    const applyShowSystemJobs = (job) =>
-        // Jobs that are configurable are user jobs
-        showSystemJobs ? true : job.configurable
-
-    const jobs = data.filter(applyJobFilter).filter(applyShowSystemJobs)
+    // Apply the current filter settings
+    const jobs = filterJobs({ jobFilter, showSystemJobs, jobs: data })
 
     return (
         <React.Fragment>

src/pages/JobList/filter-jobs.js

@@ -0,0 +1,14 @@
+const filterJobs = ({ jobFilter, showSystemJobs, jobs }) => {
+    // Filter jobs by the current jobFilter
+    const applyJobFilter = (job) =>
+        job.name.toLowerCase().includes(jobFilter.toLowerCase())
+
+    // Filter jobs depending on the current showSystemJobs
+    const applyShowSystemJobs = (job) =>
+        // Jobs that are configurable are user jobs
+        showSystemJobs ? true : job.configurable
+
+    return jobs.filter(applyJobFilter).filter(applyShowSystemJobs)
+}
+
+export default filterJobs

src/pages/JobList/filter-jobs.test.js

@@ -0,0 +1,58 @@
+import filterJobs from './filter-jobs'
+
+describe('filterJobs', () => {
+    it('should filter jobs by the current jobFilter', () => {
+        const jobFilter = 'One'
+        const showSystemJobs = true
+        const expected = { name: 'One', configurable: true }
+        const jobs = [expected, { name: 'Two', configurable: true }]
+
+        expect(filterJobs({ jobFilter, showSystemJobs, jobs })).toEqual([
+            expected,
+        ])
+    })
+
+    it('should ignore job name capitalization', () => {
+        const jobFilter = 'one'
+        const showSystemJobs = true
+        const expected = { name: 'One', configurable: true }
+        const jobs = [expected, { name: 'Two', configurable: true }]
+
+        expect(filterJobs({ jobFilter, showSystemJobs, jobs })).toEqual([
+            expected,
+        ])
+    })
+
+    it('should ignore jobFilter capitalization', () => {
+        const jobFilter = 'One'
+        const showSystemJobs = true
+        const expected = { name: 'one', configurable: true }
+        const jobs = [expected, { name: 'Two', configurable: true }]
+
+        expect(filterJobs({ jobFilter, showSystemJobs, jobs })).toEqual([
+            expected,
+        ])
+    })
+
+    it('should show system jobs and user jobs if showSystemJobs is true', () => {
+        const jobFilter = ''
+        const showSystemJobs = true
+        const jobs = [
+            { name: 'One', configurable: false },
+            { name: 'Two', configurable: true },
+        ]
+
+        expect(filterJobs({ jobFilter, showSystemJobs, jobs })).toEqual(jobs)
+    })
+
+    it('should hide system jobs and show user jobs if showSystemJobs is false', () => {
+        const jobFilter = ''
+        const showSystemJobs = false
+        const expected = { name: 'Two', configurable: true }
+        const jobs = [{ name: 'One', configurable: false }, expected]
+
+        expect(filterJobs({ jobFilter, showSystemJobs, jobs })).toEqual([
+            expected,
+        ])
+    })
+})