Skip to content

Refactor test program. - #359

Merged
samchon merged 3 commits into
masterfrom
feat/test
Jul 14, 2025
Merged

Refactor test program.#359
samchon merged 3 commits into
masterfrom
feat/test

Conversation

@samchon

@samchon samchon commented Jul 14, 2025

Copy link
Copy Markdown
Owner

This pull request includes updates to the documentation, refactoring of API simulation logic, and restructuring of test utilities. The changes aim to improve clarity in documentation, simplify simulation logic in API functions, and enhance the modularity and usability of test automation code.

Documentation Updates:

  • Updated docs/ERD.md to replace "Properties" with "Properties as follows:" for better readability and consistency across all entity descriptions. [1] [2] [3] [4] [5]
  • Added descriptions and improved formatting for several entities, such as attachment_files, bbs_articles, and bbs_article_snapshots. [1] [2] [3] [4]

API Simulation Logic Simplification:

  • Simplified simulation logic in src/api/functional/monitors/health/index.ts by removing unnecessary parameters from random and simulate functions.
  • Applied similar simplifications to performance and system API modules, ensuring consistent simulation logic across all monitor APIs. [1] [2] [3] [4]

Test Automation Enhancements:

  • Refactored and modularized test automation utilities by moving TestAutomation to a new test/helpers directory and improving its interface for better usability.
  • Removed the outdated test/TestAutomation.ts file, consolidating its functionality into the new test/helpers/TestAutomation.ts.
  • Updated references to helper modules in the benchmark test to reflect the new directory structure.

@samchon
samchon requested a review from Copilot July 14, 2025 13:01
@samchon samchon self-assigned this Jul 14, 2025
@samchon samchon added the enhancement New feature or request label Jul 14, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 restructures test automation utilities into a new helpers directory, simplifies the simulation logic in several monitor API modules, and enhances the ERD documentation for clarity.

  • Moved and modularized test automation logic into test/helpers, updating imports and integrating a new TestAutomationStdio module in test scripts.
  • Refactored random and simulate functions in src/api/functional/monitors to remove unnecessary parameters and use explicit boolean checks.
  • Updated docs/ERD.md by replacing generic "Properties" headings with "Properties as follows:" and improving entity descriptions.

Reviewed Changes

Copilot reviewed 10 out of 12 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
test/webpack.ts Updated imports to new helper modules and added TestAutomationStdio hooks.
test/index.ts Switched to helper-based imports and integrated stdio-based options and hooks.
test/helpers/TestAutomationStdio.ts Introduced CLI parsing and reporting via stdio for test execution.
test/helpers/TestAutomation.ts Added core execution flow and interface definitions for test automation.
test/benchmark/index.ts Adjusted import paths from internal to helpers.
test/TestAutomation.ts Removed deprecated test automation implementation.
src/api/functional/monitors/*/index.ts Simplified simulation logic by unifying boolean checks and removing extra args.
docs/ERD.md Enhanced formatting and clarified entity property sections.

Comment thread test/helpers/TestAutomationStdio.ts Outdated
Comment thread docs/ERD.md
Comment thread test/webpack.ts
Comment thread test/index.ts
@samchon
samchon merged commit f9c85c7 into master Jul 14, 2025
4 checks passed
@samchon
samchon deleted the feat/test branch July 14, 2025 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants