Skip to content

updating tests with async_tests#934

Merged
crivetimihai merged 3 commits intomainfrom
update-async-directory
Sep 8, 2025
Merged

updating tests with async_tests#934
crivetimihai merged 3 commits intomainfrom
update-async-directory

Conversation

@shoummu1
Copy link
Copy Markdown
Collaborator

@shoummu1 shoummu1 commented Sep 8, 2025

🐛 Bug Fix: Move async_testing to tests/async

Closes #700

Summary

This PR addresses issue #700 by moving the async_testing directory to tests/async to improve project organization and align with standard testing directory structures.

Changes Made

📁 File Structure Changes

  • Moved all files from async_testing/ to tests/async/
  • Removed the old async_testing directory
  • Preserved all async testing functionality and configurations

⚙️ Configuration Updates

  • Makefile: Updated ASYNC_TEST_DIR variable to point to tests/async
  • MANIFEST.in: Updated package inclusion patterns for new location
  • pyrightconfig.json: Updated exclusion paths for profiles directory
  • GitHub Actions: Updated workflow paths in .github/workflows/asynctest.yml.inactive
  • Plugin Templates: Updated plugin_templates/external/MANIFEST.in.jinja

📦 Package Metadata

  • Regenerated .egg-info directory to reflect new file locations
  • Updated SOURCES.txt to correctly list files in tests/async/
  • Verified no remaining references to old async_testing/ paths

Files Affected

  • async_testing/tests/async/ (7 Python files + config.yaml)
  • Makefile
  • MANIFEST.in
  • pyrightconfig.json
  • .github/workflows/asynctest.yml.inactive
  • plugin_templates/external/MANIFEST.in.jinja
  • mcp_contextforge_gateway.egg-info/SOURCES.txt (regenerated)

Verification

  • ✅ All async test files successfully moved to new location
  • ✅ Package metadata regenerated with correct file paths
  • ✅ No broken references to old async_testing/ directory
  • ✅ Makefile commands updated to use new paths
  • ✅ CI/CD configurations updated

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>
@shoummu1 shoummu1 marked this pull request as ready for review September 8, 2025 06:37
Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
@shoummu1 shoummu1 marked this pull request as draft September 8, 2025 07:34
Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
@crivetimihai crivetimihai marked this pull request as ready for review September 8, 2025 07:44
@crivetimihai crivetimihai merged commit 1a2c3db into main Sep 8, 2025
36 checks passed
@crivetimihai crivetimihai deleted the update-async-directory branch September 8, 2025 07:53
vk-playground pushed a commit to vk-playground/mcp-context-forge that referenced this pull request Sep 14, 2025
* updating tests with async_tests

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

* ignore async in pre-commit

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>

* Add async to makefile

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>

---------

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>
Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
Co-authored-by: Mihai Criveti <crivetimihai@gmail.com>
vk-playground pushed a commit to vk-playground/mcp-context-forge that referenced this pull request Sep 14, 2025
* updating tests with async_tests

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

* ignore async in pre-commit

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>

* Add async to makefile

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>

---------

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>
Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
Co-authored-by: Mihai Criveti <crivetimihai@gmail.com>
vk-playground pushed a commit to vk-playground/mcp-context-forge that referenced this pull request Sep 16, 2025
* updating tests with async_tests

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

* ignore async in pre-commit

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>

* Add async to makefile

Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>

---------

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>
Signed-off-by: Mihai Criveti <crivetimihai@gmail.com>
Co-authored-by: Mihai Criveti <crivetimihai@gmail.com>
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.

[CHORE][TESTING]: Move async_testing to tests/async

2 participants