Skip to content

Commit

Permalink
chore: remove usage of rsvp promises
Browse files Browse the repository at this point in the history
  • Loading branch information
anehx committed Sep 6, 2024
1 parent 7a57146 commit 194e42d
Show file tree
Hide file tree
Showing 3 changed files with 39 additions and 42 deletions.
1 change: 0 additions & 1 deletion packages/form-builder/tests/helpers/promise.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { later } from "@ember/runloop";
import { Promise } from "rsvp";

export function resolveAfter(ms) {
return new Promise((resolve, reject) => {
Expand Down
37 changes: 18 additions & 19 deletions packages/form-builder/tests/unit/utils/and-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
* Copied and updated from nucleartide/ember-changeset-hofs
*/
import { module, test } from "qunit";
import { resolve } from "rsvp";

import and from "@projectcaluma/ember-form-builder/utils/and";
import { resolveAfter, rejectAfter } from "dummy/tests/helpers/promise";
Expand Down Expand Up @@ -154,21 +153,21 @@ module("Unit | Utility | and", function () {

{
const validators1 = [
() => resolve("first error"),
() => resolve("second error"),
() => resolve("third error"),
() => Promise.resolve("first error"),
() => Promise.resolve("second error"),
() => Promise.resolve("third error"),
];

const validators2 = [
() => resolve("fourth error"),
() => resolve("fifth error"),
() => resolve("sixth error"),
() => Promise.resolve("fourth error"),
() => Promise.resolve("fifth error"),
() => Promise.resolve("sixth error"),
];

const validators3 = [
() => resolve("seventh error"),
() => resolve("eighth error"),
() => resolve("ninth error"),
() => Promise.resolve("seventh error"),
() => Promise.resolve("eighth error"),
() => Promise.resolve("ninth error"),
];

const validationFn = and(
Expand All @@ -181,21 +180,21 @@ module("Unit | Utility | and", function () {

{
const validators1 = [
() => resolve(true),
() => resolve(true),
() => resolve(true),
() => Promise.resolve(true),
() => Promise.resolve(true),
() => Promise.resolve(true),
];

const validators2 = [
() => resolve(true),
() => resolve("leeroy jenkins"),
() => resolve(true),
() => Promise.resolve(true),
() => Promise.resolve("leeroy jenkins"),
() => Promise.resolve(true),
];

const validators3 = [
() => resolve(true),
() => resolve(true),
() => resolve(true),
() => Promise.resolve(true),
() => Promise.resolve(true),
() => Promise.resolve(true),
];

const validationFn = and(
Expand Down
43 changes: 21 additions & 22 deletions packages/form-builder/tests/unit/utils/or-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
* Copied and updated from nucleartide/ember-changeset-hofs
*/
import { module, test } from "qunit";
import { resolve } from "rsvp";

import or from "@projectcaluma/ember-form-builder/utils/or";
import { resolveAfter, rejectAfter } from "dummy/tests/helpers/promise";
Expand Down Expand Up @@ -173,9 +172,9 @@ module("Unit | Utility | or", function () {

test("should return the last error if all validators return errors", async function (assert) {
const validators = [
() => resolve("first error"),
() => resolve("second error"),
() => resolve("third error"),
() => Promise.resolve("first error"),
() => Promise.resolve("second error"),
() => Promise.resolve("third error"),
];

const validationFn = or(...validators);
Expand All @@ -187,21 +186,21 @@ module("Unit | Utility | or", function () {

{
const validators1 = [
() => resolve("first error"),
() => resolve("second error"),
() => resolve("third error"),
() => Promise.resolve("first error"),
() => Promise.resolve("second error"),
() => Promise.resolve("third error"),
];

const validators2 = [
() => resolve("fourth error"),
() => resolve("fifth error"),
() => resolve("sixth error"),
() => Promise.resolve("fourth error"),
() => Promise.resolve("fifth error"),
() => Promise.resolve("sixth error"),
];

const validators3 = [
() => resolve("seventh error"),
() => resolve("eighth error"),
() => resolve("ninth error"),
() => Promise.resolve("seventh error"),
() => Promise.resolve("eighth error"),
() => Promise.resolve("ninth error"),
];

const validationFn = or(
Expand All @@ -214,21 +213,21 @@ module("Unit | Utility | or", function () {

{
const validators1 = [
() => resolve("first error"),
() => resolve("second error"),
() => resolve("third error"),
() => Promise.resolve("first error"),
() => Promise.resolve("second error"),
() => Promise.resolve("third error"),
];

const validators2 = [
() => resolve("fourth error"),
() => resolve(true), // derp
() => resolve("sixth error"),
() => Promise.resolve("fourth error"),
() => Promise.resolve(true), // derp
() => Promise.resolve("sixth error"),
];

const validators3 = [
() => resolve("seventh error"),
() => resolve("eighth error"),
() => resolve("ninth error"),
() => Promise.resolve("seventh error"),
() => Promise.resolve("eighth error"),
() => Promise.resolve("ninth error"),
];

const validationFn = or(
Expand Down

0 comments on commit 194e42d

Please sign in to comment.