PromucFlow_constructor/app/client/src/workers/Evaluation
Vaibhav Chobisa 6a6405227d
fix: clearInterval only clears the 'id' passed as the argument, instead of all the intervals (#29805)
## Description
clearInterval() method is removing all the intervals defined using
setInterval(). This fix will make clearInterval() work as expected, that
is, remove only the intervals whose respective 'id' has been passed as
an argument to it.

#### PR fixes following issue(s)
Fixes #29725 

#### Type of change
- Bug fix (non-breaking change which fixes an issue)

## Testing
Relevant Jest tests added to the file: interval.test.ts
#### How Has This Been Tested?
- [x] Manual
- [x] Jest

#### Test Plan
Tests added to the file: interval.test.ts

## Checklist:
#### Dev activity
- [x] My code follows the style guidelines of this project
- [x] I have performed a self-review of my own code
- [x] I have commented my code, particularly in hard-to-understand areas
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [x] I have added tests that prove my fix is effective or that my
feature works
- [x] New and existing unit tests pass locally with my changes
- [ ] PR is being merged under a feature flag


#### QA activity:
- [ ] [Speedbreak
features](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#speedbreakers-)
have been covered
- [ ] Test plan covers all impacted features and [areas of
interest](https://github.com/appsmithorg/TestSmith/wiki/Guidelines-for-test-plans#areas-of-interest-)
- [ ] Test plan has been peer reviewed by project stakeholders and other
QA members
- [ ] Manually tested functionality on DP
- [ ] We had an implementation alignment call with stakeholders post QA
Round 2
- [ ] Cypress test cases have been added and approved by SDET/manual QA
- [ ] Added `Test Plan Approved` label after Cypress tests were reviewed
- [ ] Added `Test Plan Approved` label after JUnit tests were reviewed


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

- **Bug Fixes**
- Improved interval management by ensuring that `clearInterval` now
correctly removes only the targeted interval, preserving others.
  
- **Tests**
- Expanded test coverage for interval-clearing functionality to validate
the precise removal of intervals.
  
- **Refactor**
- Updated function imports to correctly reference interval clearing
methods, enhancing the accuracy of interval manipulations.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
2023-12-26 15:15:53 +05:30
..
__tests__ chore: Refactoring entity types and updating DS action create permission to fix some bugs on EE (#29573) 2023-12-14 20:14:30 +05:30
dataStore chore: separated tree type into common file to be extended on EE (#27812) 2023-10-10 18:02:17 +05:30
fns fix: clearInterval only clears the 'id' passed as the argument, instead of all the intervals (#29805) 2023-12-26 15:15:53 +05:30
handlers fix: reset widget to default value after setter method (#29151) 2023-12-08 11:16:31 +01:00
JSObject chore: Refactoring entity types and updating DS action create permission to fix some bugs on EE (#29573) 2023-12-14 20:14:30 +05:30
domApis.ts chore: update eslint and dependencies then fix revealed errors (#27908) 2023-10-11 10:14:38 +03:00
errorModifier.ts fix: handle clone errors gracefully (#28451) 2023-10-31 23:59:10 +05:30
evalTreeWithChanges.ts fix: Validation issue in select widget on page reload (#28277) 2023-10-27 18:18:42 +05:30
evaluate.ts feat: Improve error message for typeError (#27795) 2023-10-22 07:46:31 +01:00
evaluation.worker.ts chore: Removes null check in worker request handler to allow null responses (#29070) 2023-11-27 15:00:36 +05:30
evaluationSubstitution.ts chore: upgrade to prettier v2 + enforce import types (#21013)Co-authored-by: Satish Gandham <hello@satishgandham.com> Co-authored-by: Satish Gandham <satish.iitg@gmail.com> 2023-03-16 17:11:47 +05:30
formEval.ts feat: disable spreadsheet entity in case of insert/delete one (#22490) 2023-04-24 10:10:47 +05:30
getEntityForContext.ts chore: create evalContext for module instance CE changes (#29008) 2023-11-22 15:03:00 +05:30
getJSActionForEvalContext.ts chore: create evalContext for module instance CE changes (#29008) 2023-11-22 15:03:00 +05:30
helpers.test.ts feat: Add IFrame widget for Custom EChart and support for custom functions in property pane (#28513) 2023-11-08 19:25:44 +05:30
helpers.ts chore: serialising moment updates #27963 (#29354) 2023-12-11 09:59:46 +05:30
indirectEval.ts feat: Enable fetch API (#17832) 2022-12-01 03:28:58 +05:30
ReplayDSL.ts chore: upgrade to prettier v2 + enforce import types (#21013)Co-authored-by: Satish Gandham <hello@satishgandham.com> Co-authored-by: Satish Gandham <satish.iitg@gmail.com> 2023-03-16 17:11:47 +05:30
replayUtils.ts chore: upgrade to prettier v2 + enforce import types (#21013)Co-authored-by: Satish Gandham <hello@satishgandham.com> Co-authored-by: Satish Gandham <satish.iitg@gmail.com> 2023-03-16 17:11:47 +05:30
setters.ts chore: fixed typescript errors which are thrown in EE because of split (#28009) 2023-10-16 14:53:47 +05:30
SetupDOM.ts feat: Bundle optimization and first load improvements (#21667) 2023-05-11 10:56:03 +05:30
types.ts fix: reset widget to default value after setter method (#29151) 2023-12-08 11:16:31 +01:00
validations.ts feat: Add IFrame widget for Custom EChart and support for custom functions in property pane (#28513) 2023-11-08 19:25:44 +05:30