Add AGI workflow integration tests

[Bryan-Roe]

Summary

• add test_agi_workflows.py covering AGI chat integration and autonomous file update workflows

Testing

• pytest -q test_agi_workflows.py

---

https://chatgpt.com/codex/tasks/task_e_6886a75ffe088322b319b0abb3b02f1f

Summary by Sourcery

Add end-to-end integration tests for AGI workflows by covering both chat processing and autonomous file update capabilities

New Features:

• Add test_agi_workflows.py with integration tests for NeuralSymbolicAGIIntegration chat and AutonomousFileUpdater file update workflows

Enhancements:

• Introduce load_module helper to dynamically import modules and strip side effects during tests

Tests:

• Implement async pytest functions that stub heavy dependencies and verify chat responses, conversation memory growth, and file creation tasks

[sourcery-ai]

Reviewer's Guide

Add end-to-end integration tests for AGI chat and autonomous file update workflows by introducing a new test module that dynamically loads target scripts, patches heavy dependencies, and runs asyncio-based assertions.

File-Level Changes

Change	Details	Files
Introduce helper for dynamic module loading with optional side-effect stripping	Implement load_module function to import modules by path or strip instance creation lines Use importlib.util and exec to register modules under custom names	test_agi_workflows.py
Add run_chat_integration test for AGI chat pipeline	Load and instantiate NeuralSymbolicAGIIntegration Stub _create_neural_model, _create_symbolic_reasoner, and _create_knowledge_graph Replace processing methods with lightweight async stubs Assert initialization, response content, agent_type, and memory growth	test_agi_workflows.py
Add run_file_update test for autonomous file updater	Load file update system module with strip_instance to avoid init side effects Manually construct AutonomousFileUpdater instance and set internal attributes Execute a FileUpdateTask to create a file in a temporary directory Assert task success and file existence	test_agi_workflows.py
Combine async tests into a single pytest entrypoint	Define test_agi_workflow function that runs both async workflows using asyncio.run Leverage pytest tmp_path fixture for workspace isolation	test_agi_workflows.py

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it. You can also reply to a
  review comment with @sourcery-ai issue to create an issue from it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time. You can also comment
  @sourcery-ai title on the pull request to (re-)generate the title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time exactly where you
  want it. You can also comment @sourcery-ai summary on the pull request to
  (re-)generate the summary at any time.
• Generate reviewer's guide: Comment @sourcery-ai guide on the pull
  request to (re-)generate the reviewer's guide at any time.
• Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
  pull request to resolve all Sourcery comments. Useful if you've already
  addressed all the comments and don't want to see them anymore.
• Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
  request to dismiss all existing Sourcery reviews. Especially useful if you
  want to start fresh with a new review - don't forget to comment
  @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[Copilot]

Pull Request Overview

This PR adds integration tests for AGI workflows by creating a comprehensive test file that validates AGI chat integration and autonomous file update systems. The tests use mocking and stubbing techniques to test core functionality without running expensive operations.

• Adds integration tests for AGI chat functionality with neural-symbolic processing
• Adds integration tests for autonomous file update workflows
• Implements utility functions for dynamic module loading with selective code stripping

Comments suppressed due to low confidence (1)

test_agi_workflows.py:93

• The test function combines multiple unrelated test scenarios (chat integration and file updates) into a single test. This makes it harder to identify which specific functionality failed. Consider splitting into separate test functions for better isolation and debugging.

def test_agi_workflow(tmp_path):

[sourcery-ai]

Hey @Bryan-Roe - I've reviewed your changes - here's some feedback:

• The strip_instance approach using string replacement is brittle—consider refactoring the module to accept a configuration flag or using import hooks / dependency injection for cleaner side-effect control.
• Hard-coded file paths like '09-agi-development/agi_chat_integration.py' may break in different environments; use pathlib with project root detection or import the module directly instead of relying on literal paths.
• Rather than calling asyncio.run inside a synchronous test, you could leverage pytest-asyncio to declare async test functions and simplify the event loop management.

Prompt for AI Agents

Please address the comments from this code review:
## Overall Comments
- The strip_instance approach using string replacement is brittle—consider refactoring the module to accept a configuration flag or using import hooks / dependency injection for cleaner side-effect control.
- Hard-coded file paths like '09-agi-development/agi_chat_integration.py' may break in different environments; use pathlib with project root detection or import the module directly instead of relying on literal paths.
- Rather than calling asyncio.run inside a synchronous test, you could leverage pytest-asyncio to declare async test functions and simplify the event loop management.

## Individual Comments

### Comment 1
<location> `test_agi_workflows.py:87` </location>
<code_context>
+    import logging
+    updater.logger = logging.getLogger("AutonomousFileUpdaterTest")
+
+    task = FileUpdateTask(str(tmp_dir / 'demo.txt'), 'create', content='sample', backup=False)
+    success = await updater.execute_file_task(task)
+    assert success
+    assert (tmp_dir / 'demo.txt').exists()
+
+
</code_context>

<issue_to_address>
Missing tests for file update edge cases (e.g., update, delete, backup, restricted files)

Please add tests for 'update', 'delete', backup, and restricted file scenarios to ensure full coverage of AutonomousFileUpdater operations.
</issue_to_address>

<suggested_fix>
<<<<<<< SEARCH
    task = FileUpdateTask(str(tmp_dir / 'demo.txt'), 'create', content='sample', backup=False)
    success = await updater.execute_file_task(task)
    assert success
    assert (tmp_dir / 'demo.txt').exists()


=======
    # Test file creation
    task = FileUpdateTask(str(tmp_dir / 'demo.txt'), 'create', content='sample', backup=False)
    success = await updater.execute_file_task(task)
    assert success
    assert (tmp_dir / 'demo.txt').exists()

    # Test file update
    update_task = FileUpdateTask(str(tmp_dir / 'demo.txt'), 'update', content='updated content', backup=False)
    update_success = await updater.execute_file_task(update_task)
    assert update_success
    with open(tmp_dir / 'demo.txt', 'r') as f:
        assert f.read() == 'updated content'

    # Test file delete
    delete_task = FileUpdateTask(str(tmp_dir / 'demo.txt'), 'delete', backup=False)
    delete_success = await updater.execute_file_task(delete_task)
    assert delete_success
    assert not (tmp_dir / 'demo.txt').exists()

    # Test file creation with backup
    task_with_backup = FileUpdateTask(str(tmp_dir / 'demo2.txt'), 'create', content='backup test', backup=True)
    backup_success = await updater.execute_file_task(task_with_backup)
    assert backup_success
    assert (tmp_dir / 'demo2.txt').exists()
    # Now update with backup enabled and check backup exists
    update_with_backup = FileUpdateTask(str(tmp_dir / 'demo2.txt'), 'update', content='backup updated', backup=True)
    update_backup_success = await updater.execute_file_task(update_with_backup)
    assert update_backup_success
    backup_files = list(updater.backup_path.glob('demo2.txt*'))
    assert len(backup_files) > 0

    # Test restricted file scenario
    restricted_file = tmp_dir / 'restricted.txt'
    restricted_file.write_text('restricted')
    updater.restricted_files = [str(restricted_file)]
    restricted_task = FileUpdateTask(str(restricted_file), 'update', content='should not update', backup=False)
    restricted_success = await updater.execute_file_task(restricted_task)
    assert not restricted_success
    # File content should remain unchanged
    with open(restricted_file, 'r') as f:
        assert f.read() == 'restricted'
    updater.restricted_files = []
>>>>>>> REPLACE

</suggested_fix>

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}