Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ Examples:
```text
toolbox/text-to-speech/index.html - updated
assets/toolbox/text-to-speech/js/index.js - added
tests/toolbox/text-to-speech/functional.spec.mjs - updated
dev/tests/toolbox/text-to-speech/functional.spec.mjs - updated
```

Do not report:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,25 +7,25 @@ Standardize testing locations and ensure independent tool validation.
## Canonical Test Structure

Page-level Playwright tests:
- tests/playwright/{runtime-page-path}/index.spec.mjs
- dev/tests/playwright/{runtime-page-path}/index.spec.mjs

Tool tests:
- tests/toolbox/{tool-name}/
- dev/tests/toolbox/{tool-name}/

Engine tests:
- tests/engine/{feature-name}/
- dev/tests/engine/{feature-name}/

API tests:
- tests/api/{feature-name}/
- dev/tests/api/{feature-name}/

Server tests:
- tests/server/{feature-name}/
- dev/tests/server/{feature-name}/

Shared JavaScript tests:
- tests/js/shared/
- dev/tests/js/shared/

Regression tests:
- tests/regression/
- dev/tests/regression/

## Rules

Expand All @@ -42,11 +42,11 @@ Regression tests:

## Page-Level Playwright Examples

- `/admin/index.html` -> `tests/playwright/admin/index/index.spec.mjs`
- `/toolbox/sprites/index.html` -> `tests/playwright/toolbox/sprites/index.spec.mjs`
- `/toolbox/game-hub/index.html` -> `tests/playwright/toolbox/game-hub/index.spec.mjs`
- `/toolbox/messages/index.html` -> `tests/playwright/toolbox/messages/index.spec.mjs`
- `/toolbox/game-configuration/index.html` -> `tests/playwright/toolbox/game-configuration/index.spec.mjs`
- `/admin/index.html` -> `dev/tests/playwright/admin/index/index.spec.mjs`
- `/toolbox/sprites/index.html` -> `dev/tests/playwright/toolbox/sprites/index.spec.mjs`
- `/toolbox/game-hub/index.html` -> `dev/tests/playwright/toolbox/game-hub/index.spec.mjs`
- `/toolbox/messages/index.html` -> `dev/tests/playwright/toolbox/messages/index.spec.mjs`
- `/toolbox/game-configuration/index.html` -> `dev/tests/playwright/toolbox/game-configuration/index.spec.mjs`

## Minimum Page-Level Playwright Coverage

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,5 +15,5 @@ Audit Event records describe immutable platform audit metadata for important own
## Validation

- Contract: `src/shared/contracts/auditEventContract.js`
- Test: `tests/shared/AuditEventContract.test.mjs`
- Fixture: `tests/fixtures/audit-events/audit-event-scenarios.json`
- Test: `dev/tests/shared/AuditEventContract.test.mjs`
- Fixture: `dev/tests/fixtures/audit-events/audit-event-scenarios.json`
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,6 @@ Backup Snapshot records must not contain:
Targeted validation lives in:

- `src/shared/contracts/backupSnapshotContract.js`
- `tests/shared/BackupSnapshotContract.test.mjs`
- `tests/fixtures/backup-snapshots/backup-snapshot-scenarios.json`
- `dev/tests/shared/BackupSnapshotContract.test.mjs`
- `dev/tests/fixtures/backup-snapshots/backup-snapshot-scenarios.json`

Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,6 @@ Collaboration Role records describe granted project-scoped access for users, Cre
## Validation

- Contract: `src/shared/contracts/collaborationRoleContract.js`
- Test: `tests/shared/CollaborationRoleContract.test.mjs`
- Fixture: `tests/fixtures/collaboration-roles/collaboration-role-scenarios.json`
- Test: `dev/tests/shared/CollaborationRoleContract.test.mjs`
- Fixture: `dev/tests/fixtures/collaboration-roles/collaboration-role-scenarios.json`

Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,6 @@ Creator Profile records describe the public or private creator identity attached
## Validation

- Contract: `src/shared/contracts/creatorProfileContract.js`
- Test: `tests/shared/CreatorProfileContract.test.mjs`
- Fixture: `tests/fixtures/creator-profiles/creator-profile-scenarios.json`
- Test: `dev/tests/shared/CreatorProfileContract.test.mjs`
- Fixture: `dev/tests/fixtures/creator-profiles/creator-profile-scenarios.json`

Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,6 @@ Identity and permission rules validate database object ownership and access deci

## Validation

- Contract test: `tests/shared/IdentityPermissionsContract.test.mjs`
- Fixture file: `tests/fixtures/identity-permissions/permission-scenarios.json`
- Contract test: `dev/tests/shared/IdentityPermissionsContract.test.mjs`
- Fixture file: `dev/tests/fixtures/identity-permissions/permission-scenarios.json`
- Validation report: `dev/docs_build/dev/reports/identity_permissions_contract_tests_validation.md`
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ Marketplace Transaction Boundary records describe the platform linkage around ma
## Validation

- Contract: `src/shared/contracts/marketplaceTransactionBoundaryContract.js`
- Test: `tests/shared/MarketplaceTransactionBoundaryContract.test.mjs`
- Fixture: `tests/fixtures/marketplace-transaction-boundaries/marketplace-transaction-boundary-scenarios.json`
- Test: `dev/tests/shared/MarketplaceTransactionBoundaryContract.test.mjs`
- Fixture: `dev/tests/fixtures/marketplace-transaction-boundaries/marketplace-transaction-boundary-scenarios.json`
Original file line number Diff line number Diff line change
Expand Up @@ -15,5 +15,5 @@ Moderation Queue records describe admin-only moderation intake metadata for mark
## Validation

- Contract: `src/shared/contracts/moderationQueueContract.js`
- Test: `tests/shared/ModerationQueueContract.test.mjs`
- Fixture: `tests/fixtures/moderation-queues/moderation-queue-scenarios.json`
- Test: `dev/tests/shared/ModerationQueueContract.test.mjs`
- Fixture: `dev/tests/fixtures/moderation-queues/moderation-queue-scenarios.json`
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,5 @@ Notification records describe delivery metadata for platform, project, marketpla
## Validation

- Contract: `src/shared/contracts/notificationContract.js`
- Test: `tests/shared/NotificationContract.test.mjs`
- Fixture: `tests/fixtures/notifications/notification-scenarios.json`
- Test: `dev/tests/shared/NotificationContract.test.mjs`
- Fixture: `dev/tests/fixtures/notifications/notification-scenarios.json`
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,6 @@ Organization records describe a user-owned group identity for collaboration and
## Validation

- Contract: `src/shared/contracts/organizationContract.js`
- Test: `tests/shared/OrganizationContract.test.mjs`
- Fixture: `tests/fixtures/organizations/organization-scenarios.json`
- Test: `dev/tests/shared/OrganizationContract.test.mjs`
- Fixture: `dev/tests/fixtures/organizations/organization-scenarios.json`

Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,6 @@ Project may contain Tool States, Assets, Palettes, Game Manifest, Releases, and

## Validation

- Contract test: `tests/shared/ProjectContract.test.mjs`
- Fixture file: `tests/fixtures/projects/project-scenarios.json`
- Contract test: `dev/tests/shared/ProjectContract.test.mjs`
- Fixture file: `dev/tests/fixtures/projects/project-scenarios.json`
- Validation report: `dev/docs_build/dev/reports/project_contract_tests_validation.md`
Original file line number Diff line number Diff line change
Expand Up @@ -46,5 +46,5 @@ Restore Snapshot records must not contain:
Targeted validation lives in:

- `src/shared/contracts/restoreSnapshotContract.js`
- `tests/shared/RestoreSnapshotContract.test.mjs`
- `tests/fixtures/restore-snapshots/restore-snapshot-scenarios.json`
- `dev/tests/shared/RestoreSnapshotContract.test.mjs`
- `dev/tests/fixtures/restore-snapshots/restore-snapshot-scenarios.json`
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,6 @@ Review Rating records describe user-owned marketplace ratings and optional revie
## Validation

- Contract: `src/shared/contracts/reviewRatingContract.js`
- Test: `tests/shared/ReviewRatingContract.test.mjs`
- Fixture: `tests/fixtures/review-ratings/review-rating-scenarios.json`
- Test: `dev/tests/shared/ReviewRatingContract.test.mjs`
- Fixture: `dev/tests/fixtures/review-ratings/review-rating-scenarios.json`

Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,6 @@ Tool State export is a portable format. The database remains the working system,

## Validation

- Contract test: `tests/shared/ToolStateContract.test.mjs`
- Fixture file: `tests/fixtures/tool-states/tool-state-scenarios.json`
- Contract test: `dev/tests/shared/ToolStateContract.test.mjs`
- Fixture file: `dev/tests/fixtures/tool-states/tool-state-scenarios.json`
- Validation report: `dev/docs_build/dev/reports/tool_state_contract_tests_validation.md`
48 changes: 48 additions & 0 deletions dev/docs_build/dev/reports/PR_26179_OWNER_005-move-tests-to-dev.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
# PR_26179_OWNER_005-move-tests-to-dev

Status: PASS
Team: OWNER
Branch: PR_26179_OWNER_005-move-tests-to-dev
Base branch: PR_26179_OWNER_004-move-governance-workspace
Scope: Move repository test workspace under `dev/tests/`
Updated: 2026-06-27 14:58:31

## Purpose

Move the non-deployable test workspace from root `tests/` to `dev/tests/` and update test commands, Playwright discovery, scripts, and governance references so validation continues from the new path.

## Changes

- Moved tracked `tests/` files to `dev/tests/`.
- Updated `package.json` test scripts and `directories.test` to `dev/tests`.
- Updated `playwright.config.cjs` projects to discover tests under `dev/tests/ui` and `dev/tests/playwright`.
- Updated test runner/helper scripts to reference `dev/tests`.
- Updated Project Instructions and contract standards to reference `dev/tests`.
- Recalculated moved-test relative imports where needed after PR_004 moved governance paths under `dev/`.

## Scope Confirmation

- No production `docs/`, `games/`, `toolbox/`, `account/`, `admin/`, `legal/`, `assets/`, or `src/` product files were moved for this PR.
- No runtime/business logic was moved into `dev/`.
- No Creator-writeable repo folder was introduced.
- This PR is path reorganization for tests only.

## Validation Summary

- Branch validation: PASS, current branch is `PR_26179_OWNER_005-move-tests-to-dev`.
- Tracked root `tests/` count: PASS, 0 tracked files remain.
- Tracked `dev/tests/` count: PASS, 577 tracked files present.
- Root `tests/` directory presence: PASS, absent.
- Old active `tests/` path search: PASS, no active root `tests/` references remain in package/config/scripts/ProjectInstructions except regex literals that intentionally match `dev/tests` paths.
- `git diff --check HEAD -- .`: PASS.
- `package.json` JSON parse: PASS.
- Targeted `node --check` on changed scripts and representative moved tests: PASS.
- Full all-moved-file syntax sweep was attempted and timed out due the large 600-file move; leftover Node validation processes were stopped, then the targeted syntax lane was run and passed.
- `node ./scripts/run-node-test-files.mjs dev/tests/dev-runtime/AdminNotesBoundary.test.mjs dev/tests/tools/DevConsoleIntegration.test.mjs dev/tests/shared/TimeFoundation.test.mjs`: PASS.
- `npm run test:service:api`: PASS, 2 targeted node test files passed.
- `npx playwright test --config=playwright.config.cjs --list`: PASS, discovered Playwright tests under `dev/tests/...`.
- Playwright execution: not run as a full browser lane; this PR changes test location/configuration and discovery, not runtime UI behavior.

## ZIP

Repo-structured ZIP: `tmp/PR_26179_OWNER_005-move-tests-to-dev_delta.zip`
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
# Branch Validation - PR_26179_OWNER_005-move-tests-to-dev

Status: PASS

- Current branch: `PR_26179_OWNER_005-move-tests-to-dev`
- Expected branch: `PR_26179_OWNER_005-move-tests-to-dev`
- Base branch for stacked PR: `PR_26179_OWNER_004-move-governance-workspace`
- Hard stop status: not triggered
- PR_004 dependency: current branch is stacked on the pushed PR_004 head
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
# Manual Validation Notes - PR_26179_OWNER_005-move-tests-to-dev

Status: PASS

- Confirmed current branch is `PR_26179_OWNER_005-move-tests-to-dev`.
- Confirmed no tracked files remain under root `tests/`.
- Confirmed `dev/tests/` is the active test workspace for package scripts and Playwright discovery.
- Confirmed full Playwright execution was not run; discovery validation proved the new configured test root.
- Confirmed ZIP output remains under `tmp/` for this PR because artifact-path relocation is not until PR_26179_OWNER_007.
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
# Requirement Checklist - PR_26179_OWNER_005-move-tests-to-dev

| Requirement | Status | Evidence |
| --- | --- | --- |
| Move `tests/` to `dev/tests/` | PASS | 0 tracked root `tests/` files; 577 tracked `dev/tests/` files. |
| Update `package.json` scripts | PASS | Test scripts reference `dev/tests`. |
| Update Playwright config | PASS | `playwright.config.cjs` discovers under `dev/tests/ui` and `dev/tests/playwright`. |
| Update imports, paths, and test references | PASS | Scripts, ProjectInstructions, and moved test relative paths updated. |
| Keep tests working from new path | PASS | Targeted node tests, service API script, and Playwright discovery passed. |
| Keep product/runtime paths unchanged | PASS | No product runtime files moved by PR_005. |
| Produce required reports | PASS | Report set, changed files, and review diff generated under `dev/docs_build/dev/reports/`. |
| Produce repo-structured ZIP | PASS | `tmp/PR_26179_OWNER_005-move-tests-to-dev_delta.zip` refreshed. |
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
# Validation Lane - PR_26179_OWNER_005-move-tests-to-dev

Status: PASS

## Commands

- `git diff --check HEAD -- .` - PASS
- `node -e "JSON.parse(require('fs').readFileSync('package.json','utf8')); console.log('package-json-ok')"` - PASS
- Targeted `node --check` on changed scripts and representative moved tests - PASS
- `node ./scripts/run-node-test-files.mjs dev/tests/dev-runtime/AdminNotesBoundary.test.mjs dev/tests/tools/DevConsoleIntegration.test.mjs dev/tests/shared/TimeFoundation.test.mjs` - PASS
- `npm run test:service:api` - PASS
- `npx playwright test --config=playwright.config.cjs --list` - PASS

## Notes

The broad all-moved-file syntax sweep timed out because this PR moves hundreds of test files. Targeted syntax checks and targeted execution were used as the validation lane.
Loading
Loading