Develop

.github/workflows/test.yml-template

@@ -0,0 +1,29 @@
+name: Test
+
+on:
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [20.x]
+
+    steps:
+      - uses: actions/checkout@v2
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v1
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: npm install
+      - run: npm start & sleep 5 && npm test
+      - name: Upload tests report(cypress mochaawesome merged HTML report)
+        if: ${{ always() }}
+        uses: actions/upload-artifact@v2
+        with:
+          name: report
+          path: reports

README.md

@@ -1,5 +1,5 @@
 1. Replace `<your_account>` with your Github username in the link
-    - [DEMO LINK](https://<your_account>.github.io/js_task_parse_list_DOM/)
+    - [DEMO LINK](https://uselessmorning.github.io/js_task_parse_list_DOM/)
 2. Follow [this instructions](https://mate-academy.github.io/layout_task-guideline/)
     - Run `npm run test` command to test your code;
     - Run `npm run test:only -- -n` to run fast test ignoring linter;
@@ -10,14 +10,14 @@
 Hey there! Can you parse data from the list and sort it based on data attributes?
 
 Your task: Sort list by salary in descending order.
-Get an array of employees. Write two functions: 
+Get an array of employees. Write two functions:
 - first, which sorts the list by salary from data attributes
 - second, which returns an array of objects, where objects are employees.
 
 The schema for the employee:
 ```
 {
-  name, 
+  name,
   position,
   salary,
   age

package.json

@@ -22,7 +22,7 @@
     "@linthtml/linthtml": "^0.9.6",
     "@mate-academy/eslint-config": "latest",
     "@mate-academy/linthtml-config": "latest",
-    "@mate-academy/scripts": "^1.8.5",
+    "@mate-academy/scripts": "^2.1.3",
     "@mate-academy/stylelint-config": "latest",
     "@parcel/transformer-sass": "^2.12.0",
     "cypress": "^13.13.0",

src/scripts/main.js

@@ -1,3 +1,39 @@
 'use strict';
 
 // write code here
+
+const toNum = (str) => Number(String(str || '').replace(/[$,]/g, ''));
+
+function sortList(list) {
+  const items = Array.from(list.children);
+
+  items.sort((a, b) => {
+    const salaryA = toNum(a.dataset.salary);
+    const salaryB = toNum(b.dataset.salary);
+
+    const validA = Number.isFinite(salaryA) ? salaryA : 0;
+    const validB = Number.isFinite(salaryB) ? salaryB : 0;
+
+    return validB - validA;
+  });
+
+  list.append(...items);
+}
+
+function getEmployees(list) {
+  const items = Array.from(list.children);
+
+  return items.map((item) => {
+    return {
+      name: item.querySelector('.name').textContent.trim(),
+      position: item.querySelector('.position').textContent.trim(),
+      salary: toNum(item.dataset.salary),
+      age: Number(item.dataset.age),
+    };
+  });
+}
+
+const employeesList = document.querySelector('ul');
+
+sortList(employeesList);
+getEmployees(employeesList);