Skip to content

test: add SessionManager unit coverage#18

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.

Sign up for GitHub

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

Open
edgepillar wants to merge 1 commit into
base:main
Choose a base branch
from
Open

test: add SessionManager unit coverage #18

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion CLAUDE.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -38,6 +38,8 @@ npm run dev # Web dev server (localhost:5173)
npm run dev:extension # Extension watch build
npm run build # Web production build → dist/
npm run build:extension # Extension production build → dist-extension/
npm test # Run Vitest unit suite
npm run typecheck # Type-check with vue-tsc
npm run lint # ESLint (TypeScript + Vue)
npm run format # Prettier
```
Expand DownExpand Up@@ -71,4 +73,5 @@ npm run format # Prettier

## Testing

No automated test suite. TypeScript strict mode is the primary correctness check.
Vitest provides focused unit coverage for `SessionManager`. Validate changes with `npm test`,
`npm run typecheck`, and `npm run lint`.
4 changes: 3 additions & 1 deletion README.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -71,12 +71,14 @@ Sending a transaction requires the active wallet to be unlocked — if it's lock
| `npm run build` | Production build of the web app → `dist/` |
| `npm run build:extension` | Production build of the extension → `dist-extension/` |
| `npm run preview` | Preview the production web build |
| `npm test` | Run the Vitest unit suite |
| `npm run typecheck` | Type-check with `vue-tsc` |
| `npm run lint` | Run ESLint |
| `npm run lint:fix` | Run ESLint with autofix |
| `npm run format` | Format with Prettier |

> There is no automated test suite. TypeScript strict mode, `typecheck`, and `lint` are the correctness checks.
> Automated tests currently cover `SessionManager`. Type checking and linting remain correctness
> checks.

## Tech Stack

Expand Down
3 changes: 2 additions & 1 deletion docs/ARCHITECTURE.md
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -155,6 +155,7 @@ Shared constants live in `src/config.ts` — the default node URL (`wss://node.z
## Conventions

- **No semicolons, single quotes, 100-char width** (`.prettierrc.json`).
- **Strict TypeScript** is the primary correctness check; there is no automated test suite (`npm run typecheck` + `npm run lint`).
- **Validation** uses Vitest for focused `SessionManager` unit coverage (`npm test`), strict
TypeScript (`npm run typecheck`), and ESLint (`npm run lint`).
- **PascalCase** for components, **camelCase** for services and functions.
- **No Pinia/Vuex** — shared state is the module-level reactive state inside composables.
Loading