Skip to content

Commit

Permalink
Merge pull request #167 from foretagsplatsen/LHF/F7319261160__test_ex…
Browse files Browse the repository at this point in the history
…periment_with_vitest_in_both_yaem_and_widgets

F7319261160: Experiment with vitest
  • Loading branch information
ValentinaVasile committed Apr 23, 2024
2 parents 682d8d4 + ae3bf69 commit a42bd14
Show file tree
Hide file tree
Showing 10 changed files with 705 additions and 1,596 deletions.
9 changes: 7 additions & 2 deletions .eslintrc.cjs
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,15 @@ module.exports = {
ignoreExports: [
`${__dirname}/.eslintrc.cjs`,
"index.js",
"babel.config.cjs",
"jest.config.mjs",
"vitest.config.js",
],
},
],
},
settings: {
"import/resolver": {
exports: {},
node: true,
},
},
};
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -27,4 +27,4 @@ jobs:
cache: 'yarn'
- run: yarn install --frozen-lockfile
- run: yarn run lint
- run: yarn run test
- run: yarn run test:run
19 changes: 0 additions & 19 deletions babel.config.cjs

This file was deleted.

17 changes: 0 additions & 17 deletions jest.config.mjs

This file was deleted.

12 changes: 6 additions & 6 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,15 +10,15 @@
"license": "MIT",
"scripts": {
"lint": "eslint --report-unused-disable-directives .",
"test": "jest"
"test:run": "vitest",
"test:open": "vitest --ui"
},
"devDependencies": {
"@babel/core": "^7.24.4",
"@babel/plugin-transform-modules-commonjs": "^7.24.1",
"@foretagsplatsen/eslint-plugin": "^7.0.1",
"@jest/globals": "^29.7.0",
"@vitest/coverage-v8": "^1.5.0",
"@vitest/ui": "^1.5.0",
"eslint": "^8.57.0",
"jest": "^29.7.0",
"jest-extended": "^4.0.2"
"eslint-import-resolver-exports": "^1.0.0-beta.5",
"vitest": "^1.5.0"
}
}
8 changes: 2 additions & 6 deletions test/.eslintrc.cjs
Original file line number Diff line number Diff line change
@@ -1,19 +1,15 @@
/* eslint-env node */

module.exports = {
extends: ["plugin:@foretagsplatsen/jest"],
// extends: ["plugin:@foretagsplatsen/jest"],
rules: {
"import/no-unused-modules": [
"error",
{
unusedExports: true,
missingExports: true,
// List of files not exporting anything:
ignoreExports: [
`${__dirname}/.eslintrc.cjs`,
`${__dirname}/setup.js`,
"**/*.test.js",
],
ignoreExports: [`${__dirname}/.eslintrc.cjs`, "**/*.test.js"],
},
],
},
Expand Down
26 changes: 13 additions & 13 deletions test/events.test.js
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
import event from "../src/event.js";
import eventCategory from "../src/eventCategory.js";
import events from "../src/eventManager.js";
import { describe, expect, it, jest } from "@jest/globals";
import { expect, describe, it, vi } from "vitest";

describe("events", () => {
it("bind callback to event", () => {
// Arrange: an event
let anEvent = event();
let spy = jest.fn();
let spy = vi.fn();

// Act: bind a callback
anEvent.register(spy);
Expand All @@ -22,8 +22,8 @@ describe("events", () => {
it("bind multiple callbacks to an event", () => {
// Arrange: an event
let anEvent = event();
let spy1 = jest.fn();
let spy2 = jest.fn();
let spy1 = vi.fn();
let spy2 = vi.fn();

// Act: bind two callbacks and trigger event
anEvent.register(spy1);
Expand All @@ -40,7 +40,7 @@ describe("events", () => {
// Arrange: an event
let anEvent = event();

let spy = jest.fn();
let spy = vi.fn();

// Act: bind two callbacks and trigger event
anEvent.register(spy);
Expand All @@ -56,7 +56,7 @@ describe("events", () => {
// Arrange: an event
let anEvent = event();

let spy = jest.fn();
let spy = vi.fn();

// Act: bind two callbacks and trigger event
anEvent.register(() => {
Expand All @@ -76,8 +76,8 @@ describe("events", () => {
it("trigger pass values to callbacks", () => {
// Arrange: an event
let anEvent = event();
let spy1 = jest.fn();
let spy2 = jest.fn();
let spy1 = vi.fn();
let spy2 = vi.fn();

// Act: bind two callbacks and trigger event
anEvent.register(spy1);
Expand All @@ -93,7 +93,7 @@ describe("events", () => {
it("un-Bind callback using unregister", () => {
// Arrange: an event
let anEvent = event();
let spy = jest.fn();
let spy = vi.fn();

// bind a callback
let eventBinding = anEvent.register(spy);
Expand All @@ -109,7 +109,7 @@ describe("events", () => {
it("un-Bind callback using unbind", () => {
// Arrange: an event
let anEvent = event();
let spy = jest.fn();
let spy = vi.fn();

// bind a callback
let eventBinding = anEvent.register(spy);
Expand All @@ -125,7 +125,7 @@ describe("events", () => {
it("bind and trigger callback only once using registerOnce", () => {
// Arrange: an event
let anEvent = event();
let spy = jest.fn();
let spy = vi.fn();

// Act: bind a callback
anEvent.registerOnce(spy);
Expand Down Expand Up @@ -192,7 +192,7 @@ describe("events", () => {
// Arrange: an event
let someEvents = eventCategory();
let anEvent = someEvents.createEvent("namedEvent");
let spy = jest.fn();
let spy = vi.fn();

// bind a callback
let eventBinding = someEvents.register("namedEvent", spy);
Expand All @@ -209,7 +209,7 @@ describe("events", () => {
// Arrange: an event
let someEvents = eventCategory();
let anEvent = someEvents.createEvent("namedEvent");
let spy = jest.fn();
let spy = vi.fn();

// Act: bind a callback
someEvents.registerOnce("namedEvent", spy);
Expand Down
5 changes: 0 additions & 5 deletions test/setup.js

This file was deleted.

12 changes: 12 additions & 0 deletions vitest.config.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
import { defineConfig } from "vitest/config";

export default defineConfig({
test: {
restoreMocks: true,

coverage: {
enabled: true,
include: ["src/**"],
},
},
});

0 comments on commit a42bd14

Please sign in to comment.