Reorganize file structure and tests to reduce context issues for AI #20
Conversation
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
- Added missing newlines at the end of several configuration files (.eslintrc.json, .prettierrc, tsconfig.json, jest.config.js, jest.integration.config.js). - Updated README.md to include additional environment variable setup instructions for Azure DevOps. - Improved documentation across various files, ensuring clarity and completeness, particularly in authentication and project setup sections. - Removed deprecated files and tests to streamline the codebase and improve maintainability.
- Updated test files to follow colocation patterns, improving organization and maintainability. - Added tasks to fix specific test files by co-locating or moving individual tests to appropriate files. - Enhanced error handling in authentication tests to provide clearer error messages for token acquisition failures. - Updated mock implementations in tests to ensure accurate behavior and error handling for Azure DevOps API interactions.
- Added tests to handle non-Error objects in catch blocks for both listRepositories and listWorkItems functions, ensuring proper error messages are thrown. - Updated existing tests to convert generic error messages into specific Azure DevOps error types, improving clarity and robustness in error handling. - Refactored mock implementations in authentication tests to simulate various error scenarios, enhancing test coverage and reliability.
…rganization - Deleted outdated test files including server-coverage.test.ts, server-client.test.ts, auth.test.ts, and server-auth.test.ts to streamline the codebase. - Refactored existing tests to enhance clarity and maintainability, ensuring they follow updated patterns and practices. - Improved error handling in authentication tests, ensuring comprehensive coverage for various authentication scenarios. - Updated mock implementations to accurately simulate Azure DevOps API interactions, enhancing test reliability.
This PR completes the test reorganization task by moving tests to follow the co-location pattern, ensuring proper organization and maintainability of the codebase.
…chitecture" and "Vertical Slices" This commit finalizes the restructuring of the codebase to emphasize business domains over technical layers, implementing a feature-based module organization. All tasks related to the refactor have been completed, including the migration of tests to co-locate with their respective implementations, enhancing clarity and maintainability. The old layer-based directories have been removed, and the new structure has been validated with passing tests across all features.
Tiberriver256
changed the title
…ication method This commit simplifies the setup_env.sh script by removing the Personal Access Token (PAT) creation process, which included both automated and manual methods. The script now defaults to using Azure Identity for authentication, streamlining the setup process. Additionally, the .env file has been updated to reflect the new authentication method, enhancing clarity and maintainability.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The bots were struggling...
Attempting to co-locate tests and re-structure file/folders to screaming architecture to reduce cognitive load for the poor puppies