Add test coverage on main script file

[bellmano]

No description provided.

[Copilot]

Pull Request Overview

This PR adds comprehensive test coverage for the main script file using Jest, along with supporting configuration for SonarQube analysis. The purpose is to improve code quality metrics and ensure the countdown timer functionality is properly tested.

• Adds Jest test suite covering the countUpFromTime function with various scenarios including leap years, DST changes, and edge cases
• Configures package.json with Jest setup and test scripts for coverage reporting
• Updates CI/CD pipeline to run tests and generate coverage reports for SonarQube integration

Reviewed Changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
tests/script.test.js	Comprehensive test suite with DOM mocking and various test scenarios for the timer function
src/script.js	Adds CommonJS export for Node.js compatibility in test environment
package.json	Jest configuration with coverage settings and test scripts
sonar-project.properties	SonarQube configuration for source paths and coverage reporting
README.md	Documentation updates with test running instructions and coverage badge
.github/workflows/sonarscan.yml	CI pipeline updates to run tests and generate coverage before SonarQube scan

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 6 out of 8 changed files in this pull request and generated 3 comments.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 6 out of 8 changed files in this pull request and generated 2 comments.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 6 out of 8 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (1)

tests/script.test.js:1

• The assertion expect(exported.countUpFromTime).toBe('function') is incorrect. It should use expect(typeof exported.countUpFromTime).toBe('function') to check the type, similar to line 5.

describe('Main Script Tests', () => {

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Copilot]

Pull Request Overview

Copilot reviewed 6 out of 8 changed files in this pull request and generated 2 comments.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Inconsistent type checking - line 170 expects a number (0) but other lines in the same test convert to Number() first. Either convert to Number() here or use string comparison '0' for consistency.

Suggested change

	expect(mockEl.years.innerHTML).toBe(0);
	expect(Number(mockEl.years.innerHTML)).toBe(0);

[Copilot]

Incorrect assertion - line 13 should use typeof operator like line 5, but line 13 is missing typeof and will fail since the function object is not equal to the string 'function'.