-
Notifications
You must be signed in to change notification settings - Fork 3
feat: add wrapper-based API testing approach #249
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.
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
json-choi
wants to merge
14
commits into
develop
Choose a base branch
from
241-feature-add-decorator-based-api-testing-support-for-enhanced
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 10 commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
6130974
fix: og_image is not working (#231)
wnghdcjfe ff22da5
fix: resolve issue causing LLM script to not run properly (#235)
wnghdcjfe 8e2bc85
chore(main): release 0.4.1
github-actions[bot] 0acfa8f
fix: ensure header keys are case-insensitive (#251)
PENEKhun fe7d3a2
fix: resolve syntax error in generated openapi json (#253)
PENEKhun 30b9065
fix: functional response validation failure (#246)
PENEKhun 81879ce
feat: add wrapper-based API testing approach (draft/WIP)
json-choi 4d5e8ba
chore: pnpm install
json-choi 172be58
fix: resolve ESLint errors in wrapper tests
json-choi 4ab2a00
feat: add comprehensive documentation for wrapper-based API testing
json-choi 7f6535d
fix: rollback
json-choi b360fd4
Delete .serena/project.yml
json-choi 98714d9
Delete .serena/.gitignore
json-choi 5438398
fix: rollback
json-choi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| {".":"0.4.0"} | ||
| {".":"0.4.1"} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| /cache |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,67 @@ | ||
| # language of the project (csharp, python, rust, java, typescript, go, cpp, or ruby) | ||
| # * For C, use cpp | ||
| # * For JavaScript, use typescript | ||
| # Special requirements: | ||
| # * csharp: Requires the presence of a .sln file in the project folder. | ||
| language: typescript | ||
|
|
||
| # whether to use the project's gitignore file to ignore files | ||
| # Added on 2025-04-07 | ||
| ignore_all_files_in_gitignore: true | ||
| # list of additional paths to ignore | ||
| # same syntax as gitignore, so you can use * and ** | ||
| # Was previously called `ignored_dirs`, please update your config if you are using that. | ||
| # Added (renamed) on 2025-04-07 | ||
| ignored_paths: [] | ||
|
|
||
| # whether the project is in read-only mode | ||
| # If set to true, all editing tools will be disabled and attempts to use them will result in an error | ||
| # Added on 2025-04-18 | ||
| read_only: false | ||
|
|
||
| # list of tool names to exclude. We recommend not excluding any tools, see the readme for more details. | ||
| # Below is the complete list of tools for convenience. | ||
| # To make sure you have the latest list of tools, and to view their descriptions, | ||
| # execute `uv run scripts/print_tool_overview.py`. | ||
| # | ||
| # * `activate_project`: Activates a project by name. | ||
| # * `check_onboarding_performed`: Checks whether project onboarding was already performed. | ||
| # * `create_text_file`: Creates/overwrites a file in the project directory. | ||
| # * `delete_lines`: Deletes a range of lines within a file. | ||
| # * `delete_memory`: Deletes a memory from Serena's project-specific memory store. | ||
| # * `execute_shell_command`: Executes a shell command. | ||
| # * `find_referencing_code_snippets`: Finds code snippets in which the symbol at the given location is referenced. | ||
| # * `find_referencing_symbols`: Finds symbols that reference the symbol at the given location (optionally filtered by type). | ||
| # * `find_symbol`: Performs a global (or local) search for symbols with/containing a given name/substring (optionally filtered by type). | ||
| # * `get_current_config`: Prints the current configuration of the agent, including the active and available projects, tools, contexts, and modes. | ||
| # * `get_symbols_overview`: Gets an overview of the top-level symbols defined in a given file. | ||
| # * `initial_instructions`: Gets the initial instructions for the current project. | ||
| # Should only be used in settings where the system prompt cannot be set, | ||
| # e.g. in clients you have no control over, like Claude Desktop. | ||
| # * `insert_after_symbol`: Inserts content after the end of the definition of a given symbol. | ||
| # * `insert_at_line`: Inserts content at a given line in a file. | ||
| # * `insert_before_symbol`: Inserts content before the beginning of the definition of a given symbol. | ||
| # * `list_dir`: Lists files and directories in the given directory (optionally with recursion). | ||
| # * `list_memories`: Lists memories in Serena's project-specific memory store. | ||
| # * `onboarding`: Performs onboarding (identifying the project structure and essential tasks, e.g. for testing or building). | ||
| # * `prepare_for_new_conversation`: Provides instructions for preparing for a new conversation (in order to continue with the necessary context). | ||
| # * `read_file`: Reads a file within the project directory. | ||
| # * `read_memory`: Reads the memory with the given name from Serena's project-specific memory store. | ||
| # * `remove_project`: Removes a project from the Serena configuration. | ||
| # * `replace_lines`: Replaces a range of lines within a file with new content. | ||
| # * `replace_symbol_body`: Replaces the full definition of a symbol. | ||
| # * `restart_language_server`: Restarts the language server, may be necessary when edits not through Serena happen. | ||
| # * `search_for_pattern`: Performs a search for a pattern in the project. | ||
| # * `summarize_changes`: Provides instructions for summarizing the changes made to the codebase. | ||
| # * `switch_modes`: Activates modes by providing a list of their names | ||
| # * `think_about_collected_information`: Thinking tool for pondering the completeness of collected information. | ||
| # * `think_about_task_adherence`: Thinking tool for determining whether the agent is still on track with the current task. | ||
| # * `think_about_whether_you_are_done`: Thinking tool for determining whether the task is truly completed. | ||
| # * `write_memory`: Writes a named memory (for future reference) to Serena's project-specific memory store. | ||
| excluded_tools: [] | ||
|
|
||
| # initial prompt for the project. It will always be given to the LLM upon activating the project | ||
| # (contrary to the memories, which are loaded on demand). | ||
| initial_prompt: "" | ||
|
|
||
| project_name: "itdoc" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
212 changes: 212 additions & 0 deletions
212
examples/express-ts/src/__tests__/jest/product.wrapper.spec.ts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,212 @@ | ||
| /** | ||
| * Product API Tests using wrapTest wrapper approach | ||
| * | ||
| * This demonstrates the new high-order function wrapping method | ||
| * that automatically captures HTTP requests/responses | ||
| */ | ||
|
|
||
| import { app } from "../../index" | ||
| import { wrapTest, createClient } from "itdoc" | ||
| import { ProductService } from "../../services/productService" | ||
|
|
||
| const apiTest = wrapTest(it) | ||
|
|
||
| const request = createClient.supertest(app) | ||
|
|
||
| describe("Product API - Wrapper Approach", () => { | ||
| beforeEach(() => { | ||
| ProductService.resetProducts() | ||
| }) | ||
| describe("GET /api/products/:id", () => { | ||
| apiTest.withMeta({ | ||
| summary: "Get product by ID", | ||
| tags: ["Products"], | ||
| description: "Retrieves a specific product by its ID", | ||
| })("should return a specific product", async () => { | ||
| const response = await request.get("/api/products/1") | ||
|
|
||
| expect(response.status).toBe(200) | ||
| expect(response.body).toHaveProperty("id", 1) | ||
| expect(response.body).toHaveProperty("name", "Laptop") | ||
| expect(response.body).toHaveProperty("price", 999.99) | ||
| expect(response.body).toHaveProperty("category", "Electronics") | ||
| }) | ||
|
|
||
| apiTest("should return product with different ID", async () => { | ||
| const response = await request.get("/api/products/2") | ||
|
|
||
| expect(response.status).toBe(200) | ||
| expect(response.body).toHaveProperty("id", 2) | ||
| expect(response.body).toHaveProperty("name", "Smartphone") | ||
| }) | ||
| }) | ||
|
|
||
| describe("POST /api/products", () => { | ||
| apiTest.withMeta({ | ||
| summary: "Create new product", | ||
| tags: ["Products", "Create"], | ||
| description: "Creates a new product with the provided information", | ||
| })("should create a new product", async () => { | ||
| const response = await request.post("/api/products").send({ | ||
| name: "Test Product", | ||
| price: 99.99, | ||
| category: "Test Category", | ||
| }) | ||
|
|
||
| expect(response.status).toBe(201) | ||
| expect(response.body).toHaveProperty("id", 3) | ||
| expect(response.body).toHaveProperty("name", "Test Product") | ||
| expect(response.body).toHaveProperty("price", 99.99) | ||
| expect(response.body).toHaveProperty("category", "Test Category") | ||
| }) | ||
|
|
||
| apiTest.withMeta({ | ||
| summary: "Create product with different data", | ||
| tags: ["Products", "Create"], | ||
| })("should create another product", async () => { | ||
| const response = await request.post("/api/products").send({ | ||
| name: "Another Product", | ||
| price: 199.99, | ||
| category: "Another Category", | ||
| }) | ||
|
|
||
| expect(response.status).toBe(201) | ||
| expect(response.body.name).toBe("Another Product") | ||
| }) | ||
| }) | ||
|
|
||
| describe("PUT /api/products/:id", () => { | ||
| apiTest.withMeta({ | ||
| summary: "Update product", | ||
| tags: ["Products", "Update"], | ||
| description: "Updates an existing product with the provided information", | ||
| })("should update a product", async () => { | ||
| const response = await request.put("/api/products/1").send({ | ||
| name: "Updated Product", | ||
| price: 199.99, | ||
| category: "Updated Category", | ||
| }) | ||
|
|
||
| expect(response.status).toBe(200) | ||
| expect(response.body).toHaveProperty("id", 1) | ||
| expect(response.body).toHaveProperty("name", "Updated Product") | ||
| expect(response.body).toHaveProperty("price", 199.99) | ||
| expect(response.body).toHaveProperty("category", "Updated Category") | ||
| }) | ||
|
|
||
| apiTest("should update product with partial data", async () => { | ||
| const response = await request.put("/api/products/2").send({ | ||
| name: "Partially Updated", | ||
| price: 299.99, | ||
| category: "Electronics", | ||
| }) | ||
|
|
||
| expect(response.status).toBe(200) | ||
| expect(response.body.name).toBe("Partially Updated") | ||
| }) | ||
| }) | ||
|
|
||
| describe("Complete product CRUD workflow", () => { | ||
| apiTest.withMeta({ | ||
| summary: "Product CRUD workflow", | ||
| tags: ["Products", "Workflow", "CRUD"], | ||
| description: "Complete create, read, update, delete workflow for products", | ||
| })("should perform complete CRUD operations", async () => { | ||
| const createResponse = await request.post("/api/products").send({ | ||
| name: "Workflow Product", | ||
| price: 149.99, | ||
| category: "Test", | ||
| }) | ||
|
|
||
| expect(createResponse.status).toBe(201) | ||
| const productId = createResponse.body.id | ||
|
|
||
| const getResponse = await request.get(`/api/products/${productId}`) | ||
|
|
||
| expect(getResponse.status).toBe(200) | ||
| expect(getResponse.body.name).toBe("Workflow Product") | ||
|
|
||
| const updateResponse = await request.put(`/api/products/${productId}`).send({ | ||
| name: "Updated Workflow Product", | ||
| price: 179.99, | ||
| category: "Updated Test", | ||
| }) | ||
|
|
||
| expect(updateResponse.status).toBe(200) | ||
| expect(updateResponse.body.name).toBe("Updated Workflow Product") | ||
|
|
||
| const deleteResponse = await request.delete(`/api/products/${productId}`) | ||
|
|
||
| expect(deleteResponse.status).toBe(204) | ||
| }) | ||
| }) | ||
|
|
||
| describe("Product filtering and search", () => { | ||
| apiTest.withMeta({ | ||
| summary: "Filter products by category", | ||
| tags: ["Products", "Filter"], | ||
| })("should filter products with query params", async () => { | ||
| const response = await request | ||
| .get("/api/products/1") | ||
| .query({ category: "Electronics", minPrice: 500 }) | ||
|
|
||
| expect(response.status).toBe(200) | ||
| }) | ||
|
|
||
| apiTest("should search products with multiple params", async () => { | ||
| const response = await request.get("/api/products/1").query({ | ||
| search: "laptop", | ||
| sortBy: "price", | ||
| order: "asc", | ||
| }) | ||
|
|
||
| expect(response.status).toBe(200) | ||
| }) | ||
| }) | ||
|
|
||
| describe("Product API with authentication", () => { | ||
| apiTest.withMeta({ | ||
| summary: "Create product with auth", | ||
| tags: ["Products", "Authentication"], | ||
| })("should create product with authorization header", async () => { | ||
| const response = await request | ||
| .post("/api/products") | ||
| .set("Authorization", "Bearer fake-token-123") | ||
| .send({ | ||
| name: "Authenticated Product", | ||
| price: 299.99, | ||
| category: "Secure", | ||
| }) | ||
|
|
||
| expect(response.status).toBe(201) | ||
| }) | ||
|
|
||
| apiTest("should include custom headers", async () => { | ||
| const response = await request | ||
| .get("/api/products/1") | ||
| .set("Authorization", "Bearer token") | ||
| .set("X-Client-ID", "test-client") | ||
| .set("Accept", "application/json") | ||
|
|
||
| expect(response.status).toBe(200) | ||
| }) | ||
| }) | ||
|
|
||
| describe("DELETE /api/products/:id", () => { | ||
| apiTest.withMeta({ | ||
| summary: "Delete product", | ||
| tags: ["Products", "Delete"], | ||
| description: "Deletes a product by its ID", | ||
| })("should delete a product", async () => { | ||
| const response = await request.delete("/api/products/1") | ||
|
|
||
| expect(response.status).toBe(204) | ||
| }) | ||
|
|
||
| apiTest("should delete another product", async () => { | ||
| const response = await request.delete("/api/products/2") | ||
|
|
||
| expect(response.status).toBe(204) | ||
| }) | ||
| }) | ||
| }) |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.