Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QUnit: Extensions refactoring (#11645, #11765, #11843) #11861

Merged
merged 3 commits into from
Feb 4, 2020

Conversation

San4es
Copy link
Contributor

@San4es San4es commented Feb 4, 2020

Cherry-pick #11645, #11765 and #11843.

# Conflicts:
#	testing/.eslintrc
#	testing/tests/DevExpress.viz.sparklines/baseSparklineTooltipEvents.tests.js
* QUnit: Refactor to use one way for ignoring uncleared timers

* QUnit: Get rid of timerIgnoringCheckers.applyUnregister method

* QUnit: timerIgnoringCheckers -> timersDetector.ignoreRules

* QUnit: ignoreRules.needSkip -> ignoreRules.shouldIgnore

* QUnit: Get rid of ignoreAngularBrowserDeferTimer duplication

* QUnit: Refactor isThirdPartyLibraryTimer

* QUnit: Get rid of normalizedTimerInfo

* QUnit: Refactor spyWindowMethods

* QUnit: Consolidate all ignore rules for angular in one place

* QUnit: Log test failure when uncleared timers detected instead of global one

# Conflicts:
#	testing/helpers/qunitExtensions.js
@San4es San4es force-pushed the qunit-extensions-refactoring branch from ddeb9cd to a671162 Compare February 4, 2020 11:25
@San4es San4es force-pushed the qunit-extensions-refactoring branch from a671162 to f6eb06d Compare February 4, 2020 13:11
@San4es San4es marked this pull request as ready for review February 4, 2020 13:35
@San4es San4es merged commit 6c23be9 into DevExpress:19_1 Feb 4, 2020
@San4es San4es deleted the qunit-extensions-refactoring branch February 4, 2020 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants