Skip to content

feat tests#23

Merged
mvhenten merged 6 commits intomainfrom
feat-tests
Dec 3, 2025
Merged

feat tests#23
mvhenten merged 6 commits intomainfrom
feat-tests

Conversation

@mvhenten
Copy link
Owner

@mvhenten mvhenten commented Dec 3, 2025

  • Move demo to test
  • fix(ci): ignore build/dist
  • fix(ci): update biome config
  • fix(ci): fix formatting
  • Remove eslint config; we don't use eslint
  • feat(ci): adding unit tests

Copilot AI review requested due to automatic review settings December 3, 2025 09:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR transitions the project from a demo-based structure to a proper test-driven approach, while also modernizing the tooling configuration. It removes the demo directory in favor of a dedicated test suite, updates Biome to version 2.3.8 with corresponding configuration changes, and removes the unused ESLint configuration.

Key Changes

  • Introduces comprehensive unit tests for entity generation and decorator functionality
  • Updates Biome from 1.9.4 to 2.3.8 with modernized configuration (VCS integration, assist actions)
  • Removes demo directory and replaces it with a test directory containing TypeSpec test fixtures

Reviewed changes

Copilot reviewed 12 out of 15 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
test/tspconfig.yaml New test configuration for TypeSpec compilation
test/main.tsp TypeSpec test fixtures with entity and model definitions
test/entities.test.js Comprehensive unit tests for Job and Person entities, covering attributes, decorators, and indexes
src/lib.ts Import order formatting adjustment
src/index.ts Export order formatting adjustment
src/emitter.ts Updated biome-ignore comment explanation
src/decorators/$entity.ts Removed unused NumericLiteral import
package.json Updated test scripts and Biome version to 2.3.8
package-lock.json Updated Biome dependencies to 2.3.8
biome.json Migrated to Biome 2.x configuration schema with VCS support and assist actions
.gitignore Added build and dist directories (dist was already present)
eslint.config.js Removed entire ESLint configuration
demo/* Removed demo directory files (tspconfig.yaml, package.json, index.ts)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mvhenten mvhenten merged commit bb07161 into main Dec 3, 2025
3 checks passed
@mvhenten mvhenten deleted the feat-tests branch December 3, 2025 09:57
mvhenten added a commit that referenced this pull request Dec 3, 2025
* Move demo to test

* fix(ci): ignore build/dist

* fix(ci): update biome config

* fix(ci): fix formatting

* Remove eslint config; we don't use eslint

* feat(ci): adding unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants