-
Notifications
You must be signed in to change notification settings - Fork 592
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
Add more unit tests #7036
Labels
Comments
Merged
siemiatj
added a commit
that referenced
this issue
Aug 4, 2020
siemiatj
added a commit
that referenced
this issue
Aug 5, 2020
siemiatj
added a commit
that referenced
this issue
Aug 5, 2020
siemiatj
added a commit
that referenced
this issue
Aug 6, 2020
petrican
added a commit
that referenced
this issue
Aug 10, 2020
* Pass mapDispatchToProps obj with the actions * Set proptypes * Proper fomatting on the Loader test * Add hotkeys and keymap mockups and do some cleanups * Test createProcess run * Prettify code * Skel for testing createProcess * #7036 fix modal's test Co-authored-by: Petrica Nanca <petrica.nanca@metasfresh.com> Co-authored-by: siemiatj <kuba@saskla.cz>
petrican
pushed a commit
that referenced
this issue
Aug 11, 2020
* #7036 this should make the unit tests more stable * #7036 reject Promises instead of resolving them on error * - there's no need for the `TableQuickInput` to be connected to the store * #7036 rewrite TableQuickInput tests and generalw cleanup * #7036 fix linter error * #7036 helper for debugging unhandled promises errors * Pass hotkeys and keymap * Fixed the flaky test Co-authored-by: Petrica Nanca <petrica.nanca@metasfresh.com>
petrican
added a commit
that referenced
this issue
Aug 11, 2020
* #7036 this should make the unit tests more stable * #7036 reject Promises instead of resolving them on error * - there's no need for the `TableQuickInput` to be connected to the store * #7036 rewrite TableQuickInput tests and generalw cleanup * #7036 fix linter error * #7036 helper for debugging unhandled promises errors * Pass hotkeys and keymap * Fixed the flaky test * Replaced createWaitForElement in 2nd place * Small comment to trigger rebuild * Jenkins rebuild test 1 * Jenkins rebuild test 2 * Jenkins rebuild test 3 * Jenkins rebuild test 4 * Jenkins rebuild test 5 * Jenkins rebuild test 6 * Jenkins rebuild test 7 * Jenkins rebuild test 8 * Jenkins rebuild test 9 Co-authored-by: siemiatj <kuba@saskla.cz> Co-authored-by: Petrica Nanca <petrica.nanca@metasfresh.com>
siemiatj
added a commit
that referenced
this issue
Nov 4, 2020
* #7036 this should make the unit tests more stable * #7036 reject Promises instead of resolving them on error * - there's no need for the `TableQuickInput` to be connected to the store * #7036 rewrite TableQuickInput tests and generalw cleanup * #7036 fix linter error * #7036 helper for debugging unhandled promises errors * Pass hotkeys and keymap * Fixed the flaky test * Replaced createWaitForElement in 2nd place * Small comment to trigger rebuild * Jenkins rebuild test 1 * Jenkins rebuild test 2 * Jenkins rebuild test 3 * Jenkins rebuild test 4 * Jenkins rebuild test 5 * Jenkins rebuild test 6 * Jenkins rebuild test 7 * Jenkins rebuild test 8 * Jenkins rebuild test 9 Co-authored-by: siemiatj <kuba@saskla.cz> Co-authored-by: Petrica Nanca <petrica.nanca@metasfresh.com> # Conflicts: # frontend/src/__tests__/containers/MasterWindow.test.js # frontend/src/actions/ViewActions.js
metas-ts
added a commit
that referenced
this issue
Nov 16, 2020
* move shipment-sched related WP-processors into one queue (#10151) #10150 (cherry picked from commit fbed9d3) * don't set a contact if the shipment candidate specifies none #10156 * Frontend tests stability (#7121) * #7036 this should make the unit tests more stable * #7036 reject Promises instead of resolving them on error * - there's no need for the `TableQuickInput` to be connected to the store * #7036 rewrite TableQuickInput tests and generalw cleanup * #7036 fix linter error * #7036 helper for debugging unhandled promises errors * Pass hotkeys and keymap * Fixed the flaky test * Replaced createWaitForElement in 2nd place * Small comment to trigger rebuild * Jenkins rebuild test 1 * Jenkins rebuild test 2 * Jenkins rebuild test 3 * Jenkins rebuild test 4 * Jenkins rebuild test 5 * Jenkins rebuild test 6 * Jenkins rebuild test 7 * Jenkins rebuild test 8 * Jenkins rebuild test 9 Co-authored-by: siemiatj <kuba@saskla.cz> Co-authored-by: Petrica Nanca <petrica.nanca@metasfresh.com> # Conflicts: # frontend/src/__tests__/containers/MasterWindow.test.js # frontend/src/actions/ViewActions.js * Blonde monkey hotfix more customer infos2 (#10194) * WIP - also export company name * add to shipmentschedule and keep up-to-date bill location & contact #10187 * add to shipmentschedule and keep up-to-date bill location & contact #10187 * fix unit-tests * fix unit-tests * migrate recent shipment schedules i.e. shipmentschedules that are unprocessed or touched less than three months ago #10187 * unrelated; fix process params #10187 * allow to store the actually forwarded data #10187 * SQL fixes #10187 * SQL fixes #10187 * Q&D workaround non-select-sql-processes * Fixs Bugs #10187 * during-review-fixes #10187 * sql rollout fix * Fix migration-SQL #10187 * found that we used to export the phone- instead mobile-number... ...restoring that for now #10187 * AD_Column reference fix #10187 * tweak export audit item windows #10187 * rename file to avoid file name clashes #10187 * Merge branch 'blonde_monkey_hotfix' into blonde_monkey_uat solved Conflicts: backend/de.metas.business.rest-api-impl/src/main/java/de/metas/rest_api/shipping/ShipmentCandidateAPIService.java backend/de.metas.business.rest-api-impl/src/test/java/de/metas/rest_api/shipping/ShipmentCandidateAPIServiceTest.java backend/de.metas.swat/de.metas.swat.base/src/main/java/de/metas/inoutcandidate/ShipmentScheduleRepository.java * after-merge fixes in the java-code * after-merge fixes in the java-code Co-authored-by: siemiatj <kuba@saskla.cz> Co-authored-by: Petrica Nanca <petrica.nanca@metasfresh.com>
BodorAnnamaria
added a commit
that referenced
this issue
Feb 17, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is this a bug or feature request?
F
What is the current behavior?
We need more unit tests to check if process is called within modal
Which are the steps to reproduce?
run the unit tests suite
What is the expected or desired behavior?
Modal component should call createProcess call
The text was updated successfully, but these errors were encountered: