Compare commits

...

7 Commits

Author SHA1 Message Date
Yaojia Wang
6d2396f3c1 In progress
Some checks failed
Code Coverage / Generate Coverage Report (push) Has been cancelled
Tests / Run Tests (9.0.x) (push) Has been cancelled
Tests / Docker Build Test (push) Has been cancelled
Tests / Test Summary (push) Has been cancelled
2025-11-04 10:31:50 +01:00
Yaojia Wang
ef409b8ba5 Enhance BE agent 2025-11-04 10:31:04 +01:00
Yaojia Wang
f21d9cd6d4 feat(agents): Enforce mandatory testing in backend agent
Update backend agent to enforce testing requirements:
- Extended workflow from 8 to 9 steps with explicit test phases
- Added CRITICAL Testing Rule: Must run dotnet test after every change
- Never commit with failing tests or compilation errors
- Updated Best Practices to emphasize testing (item 8)
- Removed outdated TypeScript/NestJS examples
- Updated Tech Stack to reflect actual .NET 9 stack
- Simplified configuration for better clarity

Changes:
- Workflow step 6: "Run Tests: MUST run dotnet test - fix any failures"
- Workflow step 7: "Git Commit: Auto-commit ONLY when all tests pass"
- Added "CRITICAL Testing Rule" section after workflow
- Removed Project Structure, Naming Conventions, Code Standards sections
- Updated tech stack: C# + .NET 9 + ASP.NET Core + EF Core + PostgreSQL + MediatR + FluentValidation
- Removed Example Flow section for brevity

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-04 10:28:10 +01:00
Yaojia Wang
3232b70ecc fix(backend): Fix unit test compilation errors
Updated all unit tests to match updated method signatures after ProjectManagement Module refactoring.

Changes:
- Added TenantId parameter to Project.Create() calls in all test files
- Added TenantId parameter to ProjectCreatedEvent constructor calls
- Added IHostEnvironment and ILogger mock parameters to IdentityDbContext in Identity tests
- Fixed all test files in ColaFlow.Domain.Tests, ColaFlow.Application.Tests, and ColaFlow.Modules.Identity.Infrastructure.Tests

All tests now compile successfully with 0 errors (10 analyzer warnings only).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-04 10:28:01 +01:00
Yaojia Wang
9ada0cac4a feat(backend): Implement complete Project Management Module with multi-tenant support
Day 12 implementation - Complete CRUD operations with tenant isolation and SignalR integration.

**Domain Layer**:
- Added TenantId value object for strong typing
- Updated Project entity to include TenantId field
- Modified Project.Create factory method to require tenantId parameter
- Updated ProjectCreatedEvent to include TenantId

**Application Layer**:
- Created UpdateProjectCommand, Handler, and Validator for project updates
- Created ArchiveProjectCommand, Handler, and Validator for archiving projects
- Updated CreateProjectCommand to include TenantId
- Modified CreateProjectCommandValidator to remove OwnerId validation (set from JWT)
- Created IProjectNotificationService interface for SignalR abstraction
- Implemented ProjectCreatedEventHandler with SignalR notifications
- Implemented ProjectUpdatedEventHandler with SignalR notifications
- Implemented ProjectArchivedEventHandler with SignalR notifications

**Infrastructure Layer**:
- Updated PMDbContext to inject IHttpContextAccessor
- Configured Global Query Filter for automatic tenant isolation
- Added TenantId property mapping in ProjectConfiguration
- Created TenantId index for query performance

**API Layer**:
- Updated ProjectsController with [Authorize] attribute
- Implemented PUT /api/v1/projects/{id} for updates
- Implemented DELETE /api/v1/projects/{id} for archiving
- Added helper methods to extract TenantId and UserId from JWT claims
- Extended IRealtimeNotificationService with Project-specific methods
- Implemented RealtimeNotificationService with tenant-aware SignalR groups
- Created ProjectNotificationServiceAdapter to bridge layers
- Registered IProjectNotificationService in Program.cs

**Features Implemented**:
- Complete CRUD operations (Create, Read, Update, Archive)
- Multi-tenant isolation via EF Core Global Query Filter
- JWT-based authorization on all endpoints
- SignalR real-time notifications for all Project events
- Clean Architecture with proper layer separation
- Domain Event pattern with MediatR

**Database Migration**:
- Migration created (not applied yet): AddTenantIdToProject

**Test Scripts**:
- Created comprehensive test scripts (test-project-simple.ps1)
- Tests cover full CRUD lifecycle and tenant isolation

**Note**: API hot reload required to apply CreateProjectCommandValidator fix.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-04 10:13:04 +01:00
Yaojia Wang
3843d07577 fix(backend): Fix foreign key constraint error in tenant registration
Root Cause:
- Schema mismatch between user_tenant_roles table (identity schema) and
  users/tenants tables (default/public schema)
- PostgreSQL FK constraints couldn't find referenced tables due to
  schema mismatch
- Error: "violates foreign key constraint FK_user_tenant_roles_tenants_tenant_id"

Solution:
1. Moved users and tenants tables to identity schema
2. Created migration MoveTablesToIdentitySchemaAndAddIndexes
3. All Identity module tables now in consistent identity schema
4. Added performance index for users.email lookups

Changes:
- Updated TenantConfiguration.cs to use identity schema
- Updated UserConfiguration.cs to use identity schema
- Created migration to move tables to identity schema
- Removed old AddPerformanceIndexes migration (referenced wrong schema)
- Created new AddPerformanceIndexes migration
- Added test script test-tenant-registration.ps1

Test Results:
- Tenant registration now works successfully
- User, Tenant, and UserTenantRole all insert correctly
- FK constraints validate properly
- Access token and refresh token generated successfully

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-04 09:56:04 +01:00
Yaojia Wang
5a1ad2eb97 feat(backend): Implement SignalR real-time communication infrastructure
Add complete SignalR infrastructure for real-time project collaboration and notifications with multi-tenant isolation and JWT authentication.

Changes:
- Created BaseHub with multi-tenant isolation and JWT authentication helpers
- Created ProjectHub for real-time project collaboration (join/leave, typing indicators)
- Created NotificationHub for user-level notifications
- Implemented IRealtimeNotificationService for application layer integration
- Configured SignalR in Program.cs with CORS and JWT query string support
- Added SignalRTestController for connection testing
- Documented hub endpoints, client events, and integration examples

Features:
- Multi-tenant isolation via automatic tenant group membership
- JWT authentication (Bearer header + query string for WebSocket)
- Hub endpoints: /hubs/project, /hubs/notification
- Project-level events: IssueCreated, IssueUpdated, IssueStatusChanged, etc.
- User-level notifications with tenant-wide broadcasting
- Test endpoints for validation

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-11-04 09:04:13 +01:00
61 changed files with 7306 additions and 318 deletions

View File

@@ -13,6 +13,10 @@ You are the Backend Engineer for ColaFlow, responsible for server-side code, API
Write high-quality, maintainable, testable backend code following best practices and coding standards.
## Coding Standards
Write clean, maintainable, and testable code that follows SOLID principles and adheres to established coding conventions. All implementations should emphasize readability, scalability, and long-term maintainability.
## IMPORTANT: Core Responsibilities
1. **API Development**: Design and implement RESTful APIs
@@ -42,12 +46,18 @@ Write high-quality, maintainable, testable backend code following best practices
2. Read: Existing code + architecture docs
3. Plan: Design approach (services, models, APIs)
4. Implement: Write/Edit code following standards
5. Test: Write tests, run test suite
6. Git Commit: Auto-commit changes with descriptive message
7. TodoWrite: Mark completed
8. Deliver: Working code + tests
5. Write Tests: Create/update unit and integration tests
6. Run Tests: MUST run dotnet test - fix any failures
7. Git Commit: Auto-commit ONLY when all tests pass
8. TodoWrite: Mark completed
9. Deliver: Working code + passing tests
```
**CRITICAL Testing Rule:**
- After EVERY code change, run: `dotnet test`
- If tests fail or don't compile: Fix code OR tests, then re-run
- NEVER commit with failing tests
## IMPORTANT: Git Commit Policy
**After EVERY code change (service, API, model, test, or fix), you MUST automatically commit:**
@@ -87,148 +97,24 @@ EOF
- `perf(backend): Performance improvement` - Performance optimization
- `db(backend): Database migration/change` - Database changes
**Example:**
```bash
git add src/services/issue.service.ts src/services/issue.service.spec.ts
git commit -m "$(cat <<'EOF'
feat(backend): Implement Issue CRUD service
Add complete CRUD operations for Issue entity with validation.
Changes:
- Created IssueService with create/read/update/delete methods
- Added Zod validation schemas
- Implemented unit tests with 90% coverage
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
EOF
)"
```
## Project Structure (NestJS/TypeScript)
```
src/
├── controllers/ # HTTP request handlers
├── services/ # Business logic layer
├── repositories/ # Data access layer
├── models/ # Data models/entities
├── dto/ # Data transfer objects
├── validators/ # Input validation
├── config/ # Configuration
└── mcp/ # MCP Server/Client
```
## Naming Conventions
- Files: `kebab-case.ts` (e.g., `user-service.ts`)
- Classes: `PascalCase` (e.g., `UserService`)
- Functions/variables: `camelCase` (e.g., `getUserById`)
- Constants: `UPPER_SNAKE_CASE` (e.g., `MAX_RETRIES`)
- Interfaces: `IPascalCase` (e.g., `IUserRepository`)
## Code Standards
### Service Layer Example
```typescript
@Injectable()
export class IssueService {
constructor(
@InjectRepository(Issue)
private readonly issueRepository: Repository<Issue>,
private readonly auditService: AuditService,
) {}
async create(dto: CreateIssueDto, userId: string): Promise<Issue> {
// 1. Validate
const validated = CreateIssueSchema.parse(dto);
// 2. Create entity
const issue = this.issueRepository.create({
...validated,
createdBy: userId,
});
// 3. Save
const saved = await this.issueRepository.save(issue);
// 4. Audit log
await this.auditService.log({
entityType: 'Issue',
entityId: saved.id,
action: 'CREATE',
userId,
changes: dto,
});
return saved;
}
}
```
### Data Validation (Zod)
```typescript
export const CreateIssueSchema = z.object({
title: z.string().min(1).max(200),
description: z.string().optional(),
priority: z.enum(['low', 'medium', 'high', 'urgent']),
assigneeId: z.string().uuid().optional(),
});
export type CreateIssueDto = z.infer<typeof CreateIssueSchema>;
```
### Testing Example
```typescript
describe('IssueService', () => {
let service: IssueService;
it('should create an issue', async () => {
const dto = { title: 'Test', priority: 'high' };
const result = await service.create(dto, 'user-1');
expect(result.id).toBeDefined();
expect(result.title).toBe('Test');
});
});
```
## IMPORTANT: Best Practices
1. **Dependency Injection**: Use DI for testability
2. **Single Responsibility**: Each class/function does one thing
3. **Input Validation**: Validate at boundary (DTO)
4. **Error Handling**: Use custom error classes + global handler
3. **Input Validation**: Use FluentValidation at boundary
4. **Error Handling**: Use custom exceptions + global handler
5. **Logging**: Log important operations and errors
6. **Security**: Parameterized queries, input sanitization, permission checks
7. **Performance**: Use indexes, avoid N+1 queries, cache when appropriate
8. **Use TodoWrite**: Track ALL coding tasks
9. **Read before Edit**: Always read existing code before modifying
8. **Testing**: Write tests BEFORE committing - Run `dotnet test` - Fix ALL failures - NEVER commit broken tests
9. **Use TodoWrite**: Track ALL coding tasks including test runs
10. **Read before Edit**: Always read existing code before modifying
## Tech Stack
- TypeScript + NestJS + TypeORM + PostgreSQL + Redis
## Example Flow
```
Coordinator: "Implement Issue CRUD APIs"
Your Response:
1. TodoWrite: Create tasks (model, service, controller, tests)
2. Read: Existing project structure
3. Implement: Issue entity, service, controller
4. Test: Write unit + integration tests
5. Run: npm test
6. TodoWrite: Mark completed
7. Deliver: Working APIs with 80%+ test coverage
```
- C# + .NET 9 + ASP.NET Core + EF Core + PostgreSQL + MediatR + FluentValidation
---
**Remember**: Code quality matters. Write clean, testable, maintainable code. Test everything. Document complex logic.
**Remember**: Code quality matters. Write clean, testable, maintainable code. Test everything. NEVER commit failing tests.

View File

@@ -1,19 +1,22 @@
{
"permissions": {
"allow": [
"Bash(Stop-Process -Force)",
"Bash(tasklist:*)",
"Bash(dotnet test:*)",
"Bash(tree:*)",
"Bash(dotnet add:*)",
"Bash(timeout 5 powershell:*)",
"Bash(Select-String -Pattern \"Tenant ID:|User ID:|Role\")",
"Bash(Select-String -Pattern \"(Passed|Failed|Skipped|Test Run)\")",
"Bash(Select-Object -Last 30)",
"Bash(Select-String -Pattern \"error|Build succeeded|Build FAILED\")",
"Bash(Select-Object -First 20)",
"Bash(cat:*)",
"Bash(npm run build:*)"
"Bash(npm install:*)",
"Bash(dotnet remove:*)",
"Bash(npm run lint)",
"Bash(npm run build:*)",
"Bash(timeout 10 npm run dev:*)",
"Bash(npx tsc:*)",
"Bash(timeout /t 10)",
"Bash(kill:*)",
"Bash(Select-String \"error\" -Context 0,2)",
"Bash(powershell.exe -ExecutionPolicy Bypass -File test-project-api.ps1)",
"Bash(powershell.exe -ExecutionPolicy Bypass -File test-project-simple.ps1)",
"Bash(powershell.exe -ExecutionPolicy Bypass -File test-project-debug.ps1)",
"Bash(Select-String -Pattern \"error\" -Context 0,2)",
"Bash(git add:*)",
"Bash(git restore:*)",
"Bash(git commit -m \"$(cat <<''EOF''\nfeat(agents): Enforce mandatory testing in backend agent\n\nUpdate backend agent to enforce testing requirements:\n- Extended workflow from 8 to 9 steps with explicit test phases\n- Added CRITICAL Testing Rule: Must run dotnet test after every change\n- Never commit with failing tests or compilation errors\n- Updated Best Practices to emphasize testing (item 8)\n- Removed outdated TypeScript/NestJS examples\n- Updated Tech Stack to reflect actual .NET 9 stack\n- Simplified configuration for better clarity\n\nChanges:\n- Workflow step 6: \"Run Tests: MUST run dotnet test - fix any failures\"\n- Workflow step 7: \"Git Commit: Auto-commit ONLY when all tests pass\"\n- Added \"CRITICAL Testing Rule\" section after workflow\n- Removed Project Structure, Naming Conventions, Code Standards sections\n- Updated tech stack: C# + .NET 9 + ASP.NET Core + EF Core + PostgreSQL + MediatR + FluentValidation\n- Removed Example Flow section for brevity\n\n🤖 Generated with [Claude Code](https://claude.com/claude-code)\n\nCo-Authored-By: Claude <noreply@anthropic.com>\nEOF\n)\")"
],
"deny": [],
"ask": []

File diff suppressed because it is too large Load Diff

View File

@@ -0,0 +1,314 @@
# SignalR Real-time Communication Implementation
## Overview
This document describes the SignalR real-time communication infrastructure implemented for ColaFlow. The implementation provides real-time updates for project collaboration, issue tracking, and user notifications with multi-tenant isolation and JWT authentication.
## Implementation Date
2025-11-04
## Components Implemented
### 1. Hub Infrastructure
#### BaseHub (`src/ColaFlow.API/Hubs/BaseHub.cs`)
Base class for all SignalR hubs with:
- **JWT Authentication**: All hubs require authentication via `[Authorize]` attribute
- **Multi-tenant Isolation**: Automatically adds users to tenant-specific groups on connection
- **User/Tenant Extraction**: Helper methods to extract user ID and tenant ID from JWT claims
- **Connection Lifecycle**: Logging for connect/disconnect events
**Key Features:**
- `GetCurrentUserId()`: Extracts user ID from JWT token (sub or user_id claim)
- `GetCurrentTenantId()`: Extracts tenant ID from JWT token (tenant_id claim)
- `GetTenantGroupName(Guid tenantId)`: Returns standardized tenant group name
- Automatic group membership on connection
- Error handling with connection abort on authentication failures
#### ProjectHub (`src/ColaFlow.API/Hubs/ProjectHub.cs`)
Hub for project-level real-time collaboration:
**Methods:**
- `JoinProject(Guid projectId)`: Join a project room to receive updates
- `LeaveProject(Guid projectId)`: Leave a project room
- `SendTypingIndicator(Guid projectId, Guid issueId, bool isTyping)`: Send typing indicators
**Client Events:**
- `UserJoinedProject`: Notifies when a user joins a project
- `UserLeftProject`: Notifies when a user leaves a project
- `TypingIndicator`: Real-time typing indicators for issue editing
- `ProjectUpdated`: General project updates
- `IssueCreated`: New issue created
- `IssueUpdated`: Issue updated
- `IssueDeleted`: Issue deleted
- `IssueStatusChanged`: Issue status changed
#### NotificationHub (`src/ColaFlow.API/Hubs/NotificationHub.cs`)
Hub for user-level notifications:
**Methods:**
- `MarkAsRead(Guid notificationId)`: Mark a notification as read
**Client Events:**
- `Notification`: General notifications
- `NotificationRead`: Confirmation of read status
### 2. Realtime Notification Service
#### IRealtimeNotificationService (`src/ColaFlow.API/Services/IRealtimeNotificationService.cs`)
Service interface for sending real-time notifications from application layer.
**Project-level Methods:**
- `NotifyProjectUpdate(Guid tenantId, Guid projectId, object data)`
- `NotifyIssueCreated(Guid tenantId, Guid projectId, object issue)`
- `NotifyIssueUpdated(Guid tenantId, Guid projectId, object issue)`
- `NotifyIssueDeleted(Guid tenantId, Guid projectId, Guid issueId)`
- `NotifyIssueStatusChanged(Guid tenantId, Guid projectId, Guid issueId, string oldStatus, string newStatus)`
**User-level Methods:**
- `NotifyUser(Guid userId, string message, string type = "info")`
- `NotifyUsersInTenant(Guid tenantId, string message, string type = "info")`
#### RealtimeNotificationService (`src/ColaFlow.API/Services/RealtimeNotificationService.cs`)
Implementation of the notification service using `IHubContext<T>`.
### 3. Configuration
#### Program.cs Updates
**SignalR Configuration:**
```csharp
builder.Services.AddSignalR(options =>
{
options.EnableDetailedErrors = builder.Environment.IsDevelopment();
options.ClientTimeoutInterval = TimeSpan.FromSeconds(60);
options.HandshakeTimeout = TimeSpan.FromSeconds(15);
options.KeepAliveInterval = TimeSpan.FromSeconds(15);
});
```
**CORS Configuration (SignalR-compatible):**
```csharp
builder.Services.AddCors(options =>
{
options.AddPolicy("AllowFrontend", policy =>
{
policy.WithOrigins("http://localhost:3000", "https://localhost:3000")
.AllowAnyHeader()
.AllowAnyMethod()
.AllowCredentials(); // Required for SignalR
});
});
```
**JWT Authentication for SignalR (Query String Support):**
```csharp
options.Events = new JwtBearerEvents
{
OnMessageReceived = context =>
{
var accessToken = context.Request.Query["access_token"];
var path = context.HttpContext.Request.Path;
if (!string.IsNullOrEmpty(accessToken) && path.StartsWithSegments("/hubs"))
{
context.Token = accessToken;
}
return Task.CompletedTask;
}
};
```
**Hub Endpoints:**
```csharp
app.MapHub<ProjectHub>("/hubs/project");
app.MapHub<NotificationHub>("/hubs/notification");
```
**Service Registration:**
```csharp
builder.Services.AddScoped<IRealtimeNotificationService, RealtimeNotificationService>();
```
### 4. Test Controller
#### SignalRTestController (`src/ColaFlow.API/Controllers/SignalRTestController.cs`)
Controller for testing SignalR functionality:
**Endpoints:**
- `POST /api/SignalRTest/test-user-notification`: Send notification to current user
- `POST /api/SignalRTest/test-tenant-notification`: Send notification to entire tenant
- `POST /api/SignalRTest/test-project-update`: Send project update notification
- `POST /api/SignalRTest/test-issue-status-change`: Send issue status change notification
- `GET /api/SignalRTest/connection-info`: Get connection information for debugging
## SignalR Hub Endpoints
| Hub | Endpoint | Description |
|-----|----------|-------------|
| ProjectHub | `/hubs/project` | Project-level real-time collaboration |
| NotificationHub | `/hubs/notification` | User-level notifications |
## Authentication
SignalR hubs use JWT authentication with two methods:
1. **Authorization Header**: Standard `Bearer {token}` in HTTP headers
2. **Query String**: `?access_token={token}` for WebSocket upgrade requests
All hubs are protected with `[Authorize]` attribute and require valid JWT tokens.
## Multi-Tenant Isolation
Users are automatically added to their tenant group (`tenant-{tenantId}`) on connection. This ensures:
- Notifications are only sent within tenant boundaries
- Cross-tenant data leakage is prevented
- Group-based broadcasting is efficient
## Client Connection Example
### JavaScript/TypeScript (SignalR Client)
```typescript
import * as signalR from "@microsoft/signalr";
const connection = new signalR.HubConnectionBuilder()
.withUrl("https://localhost:5001/hubs/project", {
accessTokenFactory: () => getAccessToken() // Your JWT token
})
.withAutomaticReconnect()
.build();
// Listen for events
connection.on("IssueCreated", (issue) => {
console.log("New issue:", issue);
});
connection.on("IssueStatusChanged", (data) => {
console.log("Issue status changed:", data);
});
// Start connection
await connection.start();
// Join project
await connection.invoke("JoinProject", projectId);
// Send typing indicator
await connection.invoke("SendTypingIndicator", projectId, issueId, true);
```
## Integration with Domain Events
To send SignalR notifications from application layer:
```csharp
public class IssueCreatedEventHandler : INotificationHandler<IssueCreatedEvent>
{
private readonly IRealtimeNotificationService _realtimeNotification;
public async Task Handle(IssueCreatedEvent notification, CancellationToken cancellationToken)
{
await _realtimeNotification.NotifyIssueCreated(
notification.TenantId,
notification.ProjectId,
new
{
Id = notification.IssueId,
Title = notification.Title,
Status = notification.Status
}
);
}
}
```
## Testing
### Test Endpoints
1. **Get Connection Info**:
```bash
curl -X GET https://localhost:5001/api/SignalRTest/connection-info \
-H "Authorization: Bearer {your-jwt-token}"
```
2. **Test User Notification**:
```bash
curl -X POST https://localhost:5001/api/SignalRTest/test-user-notification \
-H "Authorization: Bearer {your-jwt-token}" \
-H "Content-Type: application/json" \
-d "\"Test notification message\""
```
3. **Test Tenant Notification**:
```bash
curl -X POST https://localhost:5001/api/SignalRTest/test-tenant-notification \
-H "Authorization: Bearer {your-jwt-token}" \
-H "Content-Type: application/json" \
-d "\"Test tenant message\""
```
4. **Test Project Update**:
```bash
curl -X POST https://localhost:5001/api/SignalRTest/test-project-update \
-H "Authorization: Bearer {your-jwt-token}" \
-H "Content-Type: application/json" \
-d '{"projectId":"00000000-0000-0000-0000-000000000000","message":"Test update"}'
```
### Build Status
✅ Build successful with no errors or warnings
```bash
cd colaflow-api
dotnet build src/ColaFlow.API/ColaFlow.API.csproj
```
## Success Criteria Checklist
- [x] SignalR infrastructure added (built-in .NET 9 SignalR)
- [x] Created BaseHub, ProjectHub, NotificationHub
- [x] Configured SignalR in Program.cs with CORS and JWT
- [x] Implemented IRealtimeNotificationService
- [x] Hub supports multi-tenant isolation (automatic tenant group membership)
- [x] Hub supports JWT authentication (Bearer + query string)
- [x] Created test controller (SignalRTestController)
- [x] Compilation successful with no errors
## Files Created/Modified
**Created:**
- `src/ColaFlow.API/Hubs/BaseHub.cs`
- `src/ColaFlow.API/Hubs/ProjectHub.cs`
- `src/ColaFlow.API/Hubs/NotificationHub.cs`
- `src/ColaFlow.API/Services/IRealtimeNotificationService.cs`
- `src/ColaFlow.API/Services/RealtimeNotificationService.cs`
- `src/ColaFlow.API/Controllers/SignalRTestController.cs`
**Modified:**
- `src/ColaFlow.API/Program.cs` (SignalR configuration, CORS, JWT, hub endpoints)
## Next Steps
1. **Frontend Integration**: Implement SignalR client in Next.js frontend
2. **Domain Event Integration**: Wire up notification service to domain events
3. **Permission Validation**: Add authorization checks in ProjectHub.JoinProject()
4. **User Connection Mapping**: Implement user-to-connection tracking for targeted notifications
5. **Scalability**: Consider Redis backplane for multi-server deployments
6. **Monitoring**: Add SignalR performance metrics and connection monitoring
## Notes
- SignalR is built-in to .NET 9.0 ASP.NET Core, no separate NuGet package required
- CORS policy updated to include `AllowCredentials()` for SignalR compatibility
- JWT authentication supports both HTTP Authorization header and query string for WebSocket upgrade
- All hubs automatically enforce tenant isolation via BaseHub
- Notification service can be injected into any application service or event handler

View File

@@ -1,9 +1,13 @@
using MediatR;
using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.Mvc;
using ColaFlow.Modules.ProjectManagement.Application.DTOs;
using ColaFlow.Modules.ProjectManagement.Application.Commands.CreateProject;
using ColaFlow.Modules.ProjectManagement.Application.Commands.UpdateProject;
using ColaFlow.Modules.ProjectManagement.Application.Commands.ArchiveProject;
using ColaFlow.Modules.ProjectManagement.Application.Queries.GetProjectById;
using ColaFlow.Modules.ProjectManagement.Application.Queries.GetProjects;
using System.Security.Claims;
namespace ColaFlow.API.Controllers;
@@ -12,6 +16,7 @@ namespace ColaFlow.API.Controllers;
/// </summary>
[ApiController]
[Route("api/v1/[controller]")]
[Authorize]
public class ProjectsController(IMediator mediator) : ControllerBase
{
private readonly IMediator _mediator = mediator ?? throw new ArgumentNullException(nameof(mediator));
@@ -47,11 +52,73 @@ public class ProjectsController(IMediator mediator) : ControllerBase
[HttpPost]
[ProducesResponseType(typeof(ProjectDto), StatusCodes.Status201Created)]
[ProducesResponseType(StatusCodes.Status400BadRequest)]
[ProducesResponseType(StatusCodes.Status401Unauthorized)]
public async Task<IActionResult> CreateProject(
[FromBody] CreateProjectCommand command,
CancellationToken cancellationToken = default)
{
var result = await _mediator.Send(command, cancellationToken);
// Extract TenantId and UserId from JWT claims
var tenantId = GetTenantIdFromClaims();
var userId = GetUserIdFromClaims();
// Override command with authenticated user's context
var commandWithContext = command with
{
TenantId = tenantId,
OwnerId = userId
};
var result = await _mediator.Send(commandWithContext, cancellationToken);
return CreatedAtAction(nameof(GetProject), new { id = result.Id }, result);
}
/// <summary>
/// Update an existing project
/// </summary>
[HttpPut("{id:guid}")]
[ProducesResponseType(typeof(ProjectDto), StatusCodes.Status200OK)]
[ProducesResponseType(StatusCodes.Status404NotFound)]
[ProducesResponseType(StatusCodes.Status401Unauthorized)]
public async Task<IActionResult> UpdateProject(
Guid id,
[FromBody] UpdateProjectCommand command,
CancellationToken cancellationToken = default)
{
var commandWithId = command with { ProjectId = id };
var result = await _mediator.Send(commandWithId, cancellationToken);
return Ok(result);
}
/// <summary>
/// Archive a project
/// </summary>
[HttpDelete("{id:guid}")]
[ProducesResponseType(StatusCodes.Status204NoContent)]
[ProducesResponseType(StatusCodes.Status404NotFound)]
[ProducesResponseType(StatusCodes.Status401Unauthorized)]
public async Task<IActionResult> ArchiveProject(
Guid id,
CancellationToken cancellationToken = default)
{
await _mediator.Send(new ArchiveProjectCommand(id), cancellationToken);
return NoContent();
}
// Helper methods to extract claims
private Guid GetTenantIdFromClaims()
{
var tenantIdClaim = User.FindFirst("tenant_id")?.Value
?? throw new UnauthorizedAccessException("Tenant ID not found in token");
return Guid.Parse(tenantIdClaim);
}
private Guid GetUserIdFromClaims()
{
var userIdClaim = User.FindFirst(ClaimTypes.NameIdentifier)?.Value
?? User.FindFirst("sub")?.Value
?? throw new UnauthorizedAccessException("User ID not found in token");
return Guid.Parse(userIdClaim);
}
}

View File

@@ -0,0 +1,109 @@
using ColaFlow.API.Services;
using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.Mvc;
namespace ColaFlow.API.Controllers;
[ApiController]
[Route("api/[controller]")]
[Authorize]
public class SignalRTestController : ControllerBase
{
private readonly IRealtimeNotificationService _notificationService;
public SignalRTestController(IRealtimeNotificationService notificationService)
{
_notificationService = notificationService;
}
/// <summary>
/// Test sending notification to current user
/// </summary>
[HttpPost("test-user-notification")]
public async Task<IActionResult> TestUserNotification([FromBody] string message)
{
var userId = Guid.Parse(User.FindFirst("sub")!.Value);
await _notificationService.NotifyUser(userId, message, "test");
return Ok(new { message = "Notification sent", userId });
}
/// <summary>
/// Test sending notification to entire tenant
/// </summary>
[HttpPost("test-tenant-notification")]
public async Task<IActionResult> TestTenantNotification([FromBody] string message)
{
var tenantId = Guid.Parse(User.FindFirst("tenant_id")!.Value);
await _notificationService.NotifyUsersInTenant(tenantId, message, "test");
return Ok(new { message = "Tenant notification sent", tenantId });
}
/// <summary>
/// Test sending project update
/// </summary>
[HttpPost("test-project-update")]
public async Task<IActionResult> TestProjectUpdate([FromBody] TestProjectUpdateRequest request)
{
var tenantId = Guid.Parse(User.FindFirst("tenant_id")!.Value);
await _notificationService.NotifyProjectUpdate(tenantId, request.ProjectId, new
{
Message = request.Message,
UpdatedBy = User.FindFirst("sub")!.Value,
Timestamp = DateTime.UtcNow
});
return Ok(new { message = "Project update sent", projectId = request.ProjectId });
}
/// <summary>
/// Test sending issue status change
/// </summary>
[HttpPost("test-issue-status-change")]
public async Task<IActionResult> TestIssueStatusChange([FromBody] TestIssueStatusChangeRequest request)
{
var tenantId = Guid.Parse(User.FindFirst("tenant_id")!.Value);
await _notificationService.NotifyIssueStatusChanged(
tenantId,
request.ProjectId,
request.IssueId,
request.OldStatus,
request.NewStatus
);
return Ok(new
{
message = "Issue status change notification sent",
projectId = request.ProjectId,
issueId = request.IssueId
});
}
/// <summary>
/// Get connection info for debugging
/// </summary>
[HttpGet("connection-info")]
public IActionResult GetConnectionInfo()
{
return Ok(new
{
userId = User.FindFirst("sub")?.Value,
tenantId = User.FindFirst("tenant_id")?.Value,
roles = User.Claims.Where(c => c.Type == "role").Select(c => c.Value).ToList(),
hubEndpoints = new[]
{
"/hubs/project",
"/hubs/notification"
},
instructions = "Connect to SignalR hubs using the endpoints above with access_token query parameter"
});
}
}
public record TestProjectUpdateRequest(Guid ProjectId, string Message);
public record TestIssueStatusChangeRequest(Guid ProjectId, Guid IssueId, string OldStatus, string NewStatus);

View File

@@ -0,0 +1,70 @@
using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.SignalR;
namespace ColaFlow.API.Hubs;
[Authorize] // All Hubs require authentication
public abstract class BaseHub : Hub
{
protected Guid GetCurrentUserId()
{
var userIdClaim = Context.User?.FindFirst("sub")
?? Context.User?.FindFirst("user_id");
if (userIdClaim == null || !Guid.TryParse(userIdClaim.Value, out var userId))
{
throw new UnauthorizedAccessException("User ID not found in token");
}
return userId;
}
protected Guid GetCurrentTenantId()
{
var tenantIdClaim = Context.User?.FindFirst("tenant_id");
if (tenantIdClaim == null || !Guid.TryParse(tenantIdClaim.Value, out var tenantId))
{
throw new UnauthorizedAccessException("Tenant ID not found in token");
}
return tenantId;
}
protected string GetTenantGroupName(Guid tenantId)
{
return $"tenant-{tenantId}";
}
public override async Task OnConnectedAsync()
{
try
{
var tenantId = GetCurrentTenantId();
var userId = GetCurrentUserId();
// Automatically join tenant group (tenant isolation)
await Groups.AddToGroupAsync(Context.ConnectionId, GetTenantGroupName(tenantId));
// Log connection
Console.WriteLine($"User {userId} from tenant {tenantId} connected. ConnectionId: {Context.ConnectionId}");
await base.OnConnectedAsync();
}
catch (Exception ex)
{
Console.WriteLine($"Connection error: {ex.Message}");
Context.Abort();
}
}
public override async Task OnDisconnectedAsync(Exception? exception)
{
var tenantId = GetCurrentTenantId();
var userId = GetCurrentUserId();
Console.WriteLine($"User {userId} from tenant {tenantId} disconnected. Reason: {exception?.Message ?? "Normal"}");
await base.OnDisconnectedAsync(exception);
}
}

View File

@@ -0,0 +1,25 @@
using Microsoft.AspNetCore.SignalR;
namespace ColaFlow.API.Hubs;
/// <summary>
/// Notification Hub (user-level notifications)
/// </summary>
public class NotificationHub : BaseHub
{
/// <summary>
/// Mark notification as read
/// </summary>
public async Task MarkAsRead(Guid notificationId)
{
var userId = GetCurrentUserId();
// TODO: Call Application layer to mark notification as read
await Clients.Caller.SendAsync("NotificationRead", new
{
NotificationId = notificationId,
ReadAt = DateTime.UtcNow
});
}
}

View File

@@ -0,0 +1,73 @@
using Microsoft.AspNetCore.SignalR;
namespace ColaFlow.API.Hubs;
/// <summary>
/// Project real-time collaboration Hub
/// </summary>
public class ProjectHub : BaseHub
{
/// <summary>
/// Join project room (to receive project-level updates)
/// </summary>
public async Task JoinProject(Guid projectId)
{
var tenantId = GetCurrentTenantId();
var userId = GetCurrentUserId();
// TODO: Validate user has permission to access this project
var groupName = GetProjectGroupName(projectId);
await Groups.AddToGroupAsync(Context.ConnectionId, groupName);
Console.WriteLine($"User {userId} joined project {projectId}");
// Notify other users that a new member joined
await Clients.OthersInGroup(groupName).SendAsync("UserJoinedProject", new
{
UserId = userId,
ProjectId = projectId,
JoinedAt = DateTime.UtcNow
});
}
/// <summary>
/// Leave project room
/// </summary>
public async Task LeaveProject(Guid projectId)
{
var userId = GetCurrentUserId();
var groupName = GetProjectGroupName(projectId);
await Groups.RemoveFromGroupAsync(Context.ConnectionId, groupName);
// Notify other users that a member left
await Clients.OthersInGroup(groupName).SendAsync("UserLeftProject", new
{
UserId = userId,
ProjectId = projectId,
LeftAt = DateTime.UtcNow
});
}
/// <summary>
/// Send typing indicator (when editing an issue)
/// </summary>
public async Task SendTypingIndicator(Guid projectId, Guid issueId, bool isTyping)
{
var userId = GetCurrentUserId();
var groupName = GetProjectGroupName(projectId);
await Clients.OthersInGroup(groupName).SendAsync("TypingIndicator", new
{
UserId = userId,
IssueId = issueId,
IsTyping = isTyping
});
}
private string GetProjectGroupName(Guid projectId)
{
return $"project-{projectId}";
}
}

View File

@@ -1,6 +1,8 @@
using ColaFlow.API.Extensions;
using ColaFlow.API.Handlers;
using ColaFlow.API.Hubs;
using ColaFlow.API.Middleware;
using ColaFlow.API.Services;
using ColaFlow.Modules.Identity.Application;
using ColaFlow.Modules.Identity.Infrastructure;
using Microsoft.AspNetCore.Authentication.JwtBearer;
@@ -65,6 +67,25 @@ builder.Services.AddAuthentication(options =>
IssuerSigningKey = new SymmetricSecurityKey(
Encoding.UTF8.GetBytes(builder.Configuration["Jwt:SecretKey"] ?? throw new InvalidOperationException("JWT SecretKey not configured")))
};
// Configure SignalR to use JWT from query string (for WebSocket upgrade)
options.Events = new JwtBearerEvents
{
OnMessageReceived = context =>
{
var accessToken = context.Request.Query["access_token"];
// If the request is for SignalR hub...
var path = context.HttpContext.Request.Path;
if (!string.IsNullOrEmpty(accessToken) && path.StartsWithSegments("/hubs"))
{
// Read the token from query string
context.Token = accessToken;
}
return Task.CompletedTask;
}
};
});
// Configure Authorization Policies for RBAC
@@ -92,17 +113,39 @@ builder.Services.AddAuthorization(options =>
policy.RequireRole("AIAgent"));
});
// Configure CORS for frontend
// Configure CORS for frontend (SignalR requires AllowCredentials)
builder.Services.AddCors(options =>
{
options.AddPolicy("AllowFrontend", policy =>
{
policy.WithOrigins("http://localhost:3000")
policy.WithOrigins("http://localhost:3000", "https://localhost:3000")
.AllowAnyHeader()
.AllowAnyMethod();
.AllowAnyMethod()
.AllowCredentials(); // Required for SignalR
});
});
// Configure SignalR
builder.Services.AddSignalR(options =>
{
// Enable detailed errors (development only)
options.EnableDetailedErrors = builder.Environment.IsDevelopment();
// Client timeout settings
options.ClientTimeoutInterval = TimeSpan.FromSeconds(60);
options.HandshakeTimeout = TimeSpan.FromSeconds(15);
// Keep alive interval
options.KeepAliveInterval = TimeSpan.FromSeconds(15);
});
// Register Realtime Notification Service
builder.Services.AddScoped<IRealtimeNotificationService, RealtimeNotificationService>();
// Register Project Notification Service Adapter (for ProjectManagement module)
builder.Services.AddScoped<ColaFlow.Modules.ProjectManagement.Application.Services.IProjectNotificationService,
ProjectNotificationServiceAdapter>();
// Configure OpenAPI/Scalar
builder.Services.AddOpenApi();
@@ -138,6 +181,10 @@ app.UseAuthorization();
app.MapControllers();
// Map SignalR Hubs (after UseAuthorization)
app.MapHub<ProjectHub>("/hubs/project");
app.MapHub<NotificationHub>("/hubs/notification");
app.Run();
// Make the implicit Program class public for integration tests

View File

@@ -0,0 +1,20 @@
namespace ColaFlow.API.Services;
public interface IRealtimeNotificationService
{
// Project-level notifications
Task NotifyProjectCreated(Guid tenantId, Guid projectId, object project);
Task NotifyProjectUpdated(Guid tenantId, Guid projectId, object project);
Task NotifyProjectArchived(Guid tenantId, Guid projectId);
Task NotifyProjectUpdate(Guid tenantId, Guid projectId, object data);
// Issue notifications
Task NotifyIssueCreated(Guid tenantId, Guid projectId, object issue);
Task NotifyIssueUpdated(Guid tenantId, Guid projectId, object issue);
Task NotifyIssueDeleted(Guid tenantId, Guid projectId, Guid issueId);
Task NotifyIssueStatusChanged(Guid tenantId, Guid projectId, Guid issueId, string oldStatus, string newStatus);
// User-level notifications
Task NotifyUser(Guid userId, string message, string type = "info");
Task NotifyUsersInTenant(Guid tenantId, string message, string type = "info");
}

View File

@@ -0,0 +1,32 @@
using ColaFlow.Modules.ProjectManagement.Application.Services;
namespace ColaFlow.API.Services;
/// <summary>
/// Adapter that implements IProjectNotificationService by delegating to IRealtimeNotificationService
/// This allows the ProjectManagement module to send notifications without depending on the API layer
/// </summary>
public class ProjectNotificationServiceAdapter : IProjectNotificationService
{
private readonly IRealtimeNotificationService _realtimeService;
public ProjectNotificationServiceAdapter(IRealtimeNotificationService realtimeService)
{
_realtimeService = realtimeService;
}
public Task NotifyProjectCreated(Guid tenantId, Guid projectId, object project)
{
return _realtimeService.NotifyProjectCreated(tenantId, projectId, project);
}
public Task NotifyProjectUpdated(Guid tenantId, Guid projectId, object project)
{
return _realtimeService.NotifyProjectUpdated(tenantId, projectId, project);
}
public Task NotifyProjectArchived(Guid tenantId, Guid projectId)
{
return _realtimeService.NotifyProjectArchived(tenantId, projectId);
}
}

View File

@@ -0,0 +1,124 @@
using Microsoft.AspNetCore.SignalR;
using ColaFlow.API.Hubs;
namespace ColaFlow.API.Services;
public class RealtimeNotificationService : IRealtimeNotificationService
{
private readonly IHubContext<ProjectHub> _projectHubContext;
private readonly IHubContext<NotificationHub> _notificationHubContext;
private readonly ILogger<RealtimeNotificationService> _logger;
public RealtimeNotificationService(
IHubContext<ProjectHub> projectHubContext,
IHubContext<NotificationHub> notificationHubContext,
ILogger<RealtimeNotificationService> logger)
{
_projectHubContext = projectHubContext;
_notificationHubContext = notificationHubContext;
_logger = logger;
}
public async Task NotifyProjectCreated(Guid tenantId, Guid projectId, object project)
{
var tenantGroupName = $"tenant-{tenantId}";
_logger.LogInformation("Notifying tenant {TenantId} of new project {ProjectId}", tenantId, projectId);
await _projectHubContext.Clients.Group(tenantGroupName).SendAsync("ProjectCreated", project);
}
public async Task NotifyProjectUpdated(Guid tenantId, Guid projectId, object project)
{
var projectGroupName = $"project-{projectId}";
var tenantGroupName = $"tenant-{tenantId}";
_logger.LogInformation("Notifying project {ProjectId} updated", projectId);
await _projectHubContext.Clients.Group(projectGroupName).SendAsync("ProjectUpdated", project);
await _projectHubContext.Clients.Group(tenantGroupName).SendAsync("ProjectUpdated", project);
}
public async Task NotifyProjectArchived(Guid tenantId, Guid projectId)
{
var projectGroupName = $"project-{projectId}";
var tenantGroupName = $"tenant-{tenantId}";
_logger.LogInformation("Notifying project {ProjectId} archived", projectId);
await _projectHubContext.Clients.Group(projectGroupName).SendAsync("ProjectArchived", new { ProjectId = projectId });
await _projectHubContext.Clients.Group(tenantGroupName).SendAsync("ProjectArchived", new { ProjectId = projectId });
}
public async Task NotifyProjectUpdate(Guid tenantId, Guid projectId, object data)
{
var groupName = $"project-{projectId}";
_logger.LogInformation("Sending project update to group {GroupName}", groupName);
await _projectHubContext.Clients.Group(groupName).SendAsync("ProjectUpdated", data);
}
public async Task NotifyIssueCreated(Guid tenantId, Guid projectId, object issue)
{
var groupName = $"project-{projectId}";
await _projectHubContext.Clients.Group(groupName).SendAsync("IssueCreated", issue);
}
public async Task NotifyIssueUpdated(Guid tenantId, Guid projectId, object issue)
{
var groupName = $"project-{projectId}";
await _projectHubContext.Clients.Group(groupName).SendAsync("IssueUpdated", issue);
}
public async Task NotifyIssueDeleted(Guid tenantId, Guid projectId, Guid issueId)
{
var groupName = $"project-{projectId}";
await _projectHubContext.Clients.Group(groupName).SendAsync("IssueDeleted", new { IssueId = issueId });
}
public async Task NotifyIssueStatusChanged(
Guid tenantId,
Guid projectId,
Guid issueId,
string oldStatus,
string newStatus)
{
var groupName = $"project-{projectId}";
await _projectHubContext.Clients.Group(groupName).SendAsync("IssueStatusChanged", new
{
IssueId = issueId,
OldStatus = oldStatus,
NewStatus = newStatus,
ChangedAt = DateTime.UtcNow
});
}
public async Task NotifyUser(Guid userId, string message, string type = "info")
{
var userConnectionId = $"user-{userId}";
await _notificationHubContext.Clients.User(userId.ToString()).SendAsync("Notification", new
{
Message = message,
Type = type,
Timestamp = DateTime.UtcNow
});
}
public async Task NotifyUsersInTenant(Guid tenantId, string message, string type = "info")
{
var groupName = $"tenant-{tenantId}";
await _notificationHubContext.Clients.Group(groupName).SendAsync("Notification", new
{
Message = message,
Type = type,
Timestamp = DateTime.UtcNow
});
}
}

View File

@@ -9,7 +9,7 @@ public class TenantConfiguration : IEntityTypeConfiguration<Tenant>
{
public void Configure(EntityTypeBuilder<Tenant> builder)
{
builder.ToTable("tenants");
builder.ToTable("tenants", "identity");
// Primary Key
builder.HasKey(t => t.Id);

View File

@@ -9,7 +9,7 @@ public class UserConfiguration : IEntityTypeConfiguration<User>
{
public void Configure(EntityTypeBuilder<User> builder)
{
builder.ToTable("users");
builder.ToTable("users", "identity");
// Primary Key
builder.HasKey(u => u.Id);

View File

@@ -1,65 +0,0 @@
using Microsoft.EntityFrameworkCore.Migrations;
#nullable disable
namespace ColaFlow.Modules.Identity.Infrastructure.Persistence.Migrations
{
/// <inheritdoc />
public partial class AddPerformanceIndexes : Migration
{
/// <inheritdoc />
protected override void Up(MigrationBuilder migrationBuilder)
{
// Index for email lookups (case-insensitive for PostgreSQL)
migrationBuilder.Sql(@"
CREATE INDEX IF NOT EXISTS idx_users_email_lower
ON identity.users(LOWER(email));
");
// Index for password reset token lookups
migrationBuilder.Sql(@"
CREATE INDEX IF NOT EXISTS idx_password_reset_tokens_token
ON identity.password_reset_tokens(token)
WHERE expires_at > NOW();
");
// Composite index for invitation lookups (tenant + status)
migrationBuilder.Sql(@"
CREATE INDEX IF NOT EXISTS idx_invitations_tenant_status
ON identity.invitations(tenant_id, status)
WHERE status = 'Pending';
");
// Index for refresh token lookups (user + tenant, only active tokens)
migrationBuilder.Sql(@"
CREATE INDEX IF NOT EXISTS idx_refresh_tokens_user_tenant
ON identity.refresh_tokens(user_id, tenant_id)
WHERE revoked_at IS NULL;
");
// Index for user tenant roles (tenant + role)
migrationBuilder.Sql(@"
CREATE INDEX IF NOT EXISTS idx_user_tenant_roles_tenant_role
ON identity.user_tenant_roles(tenant_id, role);
");
// Index for email verification tokens
migrationBuilder.Sql(@"
CREATE INDEX IF NOT EXISTS idx_email_verification_tokens_token
ON identity.email_verification_tokens(token)
WHERE expires_at > NOW();
");
}
/// <inheritdoc />
protected override void Down(MigrationBuilder migrationBuilder)
{
migrationBuilder.Sql("DROP INDEX IF EXISTS identity.idx_users_email_lower;");
migrationBuilder.Sql("DROP INDEX IF EXISTS identity.idx_password_reset_tokens_token;");
migrationBuilder.Sql("DROP INDEX IF EXISTS identity.idx_invitations_tenant_status;");
migrationBuilder.Sql("DROP INDEX IF EXISTS identity.idx_refresh_tokens_user_tenant;");
migrationBuilder.Sql("DROP INDEX IF EXISTS identity.idx_user_tenant_roles_tenant_role;");
migrationBuilder.Sql("DROP INDEX IF EXISTS identity.idx_email_verification_tokens_token;");
}
}
}

View File

@@ -0,0 +1,531 @@
// <auto-generated />
using System;
using ColaFlow.Modules.Identity.Infrastructure.Persistence;
using Microsoft.EntityFrameworkCore;
using Microsoft.EntityFrameworkCore.Infrastructure;
using Microsoft.EntityFrameworkCore.Migrations;
using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
using Npgsql.EntityFrameworkCore.PostgreSQL.Metadata;
#nullable disable
namespace ColaFlow.Modules.Identity.Infrastructure.Persistence.Migrations
{
[DbContext(typeof(IdentityDbContext))]
[Migration("20251104085158_MoveTablesToIdentitySchemaAndAddIndexes")]
partial class MoveTablesToIdentitySchemaAndAddIndexes
{
/// <inheritdoc />
protected override void BuildTargetModel(ModelBuilder modelBuilder)
{
#pragma warning disable 612, 618
modelBuilder
.HasAnnotation("ProductVersion", "9.0.10")
.HasAnnotation("Relational:MaxIdentifierLength", 63);
NpgsqlModelBuilderExtensions.UseIdentityByDefaultColumns(modelBuilder);
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Aggregates.Invitations.Invitation", b =>
{
b.Property<Guid>("Id")
.HasColumnType("uuid")
.HasColumnName("id");
b.Property<DateTime?>("AcceptedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("accepted_at");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("created_at");
b.Property<string>("Email")
.IsRequired()
.HasMaxLength(255)
.HasColumnType("character varying(255)")
.HasColumnName("email");
b.Property<DateTime>("ExpiresAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("expires_at");
b.Property<Guid>("InvitedBy")
.HasColumnType("uuid")
.HasColumnName("invited_by");
b.Property<string>("Role")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("role");
b.Property<Guid>("TenantId")
.HasColumnType("uuid")
.HasColumnName("tenant_id");
b.Property<string>("TokenHash")
.IsRequired()
.HasMaxLength(255)
.HasColumnType("character varying(255)")
.HasColumnName("token_hash");
b.Property<DateTime>("UpdatedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("updated_at");
b.HasKey("Id");
b.HasIndex("TokenHash")
.IsUnique()
.HasDatabaseName("ix_invitations_token_hash");
b.HasIndex("TenantId", "Email")
.HasDatabaseName("ix_invitations_tenant_id_email");
b.HasIndex("TenantId", "AcceptedAt", "ExpiresAt")
.HasDatabaseName("ix_invitations_tenant_id_status");
b.ToTable("invitations", (string)null);
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Aggregates.Tenants.Tenant", b =>
{
b.Property<Guid>("Id")
.ValueGeneratedOnAdd()
.HasColumnType("uuid")
.HasColumnName("id");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("created_at");
b.Property<int>("MaxProjects")
.HasColumnType("integer")
.HasColumnName("max_projects");
b.Property<int>("MaxStorageGB")
.HasColumnType("integer")
.HasColumnName("max_storage_gb");
b.Property<int>("MaxUsers")
.HasColumnType("integer")
.HasColumnName("max_users");
b.Property<string>("Name")
.IsRequired()
.HasMaxLength(100)
.HasColumnType("character varying(100)")
.HasColumnName("name");
b.Property<string>("Plan")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("plan");
b.Property<string>("Slug")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("slug");
b.Property<string>("SsoConfig")
.HasColumnType("jsonb")
.HasColumnName("sso_config");
b.Property<string>("Status")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("status");
b.Property<DateTime?>("SuspendedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("suspended_at");
b.Property<string>("SuspensionReason")
.HasMaxLength(500)
.HasColumnType("character varying(500)")
.HasColumnName("suspension_reason");
b.Property<DateTime?>("UpdatedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("updated_at");
b.HasKey("Id");
b.HasIndex("Slug")
.IsUnique()
.HasDatabaseName("ix_tenants_slug");
b.ToTable("tenants", "identity");
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Aggregates.Users.RefreshToken", b =>
{
b.Property<Guid>("Id")
.ValueGeneratedOnAdd()
.HasColumnType("uuid");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("created_at");
b.Property<string>("DeviceInfo")
.HasMaxLength(500)
.HasColumnType("character varying(500)")
.HasColumnName("device_info");
b.Property<DateTime>("ExpiresAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("expires_at");
b.Property<string>("IpAddress")
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("ip_address");
b.Property<string>("ReplacedByToken")
.HasMaxLength(500)
.HasColumnType("character varying(500)")
.HasColumnName("replaced_by_token");
b.Property<DateTime?>("RevokedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("revoked_at");
b.Property<string>("RevokedReason")
.HasMaxLength(500)
.HasColumnType("character varying(500)")
.HasColumnName("revoked_reason");
b.Property<Guid>("TenantId")
.HasColumnType("uuid")
.HasColumnName("tenant_id");
b.Property<string>("TokenHash")
.IsRequired()
.HasMaxLength(500)
.HasColumnType("character varying(500)")
.HasColumnName("token_hash");
b.Property<string>("UserAgent")
.HasMaxLength(500)
.HasColumnType("character varying(500)")
.HasColumnName("user_agent");
b.Property<Guid>("UserId")
.HasColumnType("uuid")
.HasColumnName("user_id");
b.HasKey("Id");
b.HasIndex("ExpiresAt")
.HasDatabaseName("ix_refresh_tokens_expires_at");
b.HasIndex("TenantId")
.HasDatabaseName("ix_refresh_tokens_tenant_id");
b.HasIndex("TokenHash")
.IsUnique()
.HasDatabaseName("ix_refresh_tokens_token_hash");
b.HasIndex("UserId")
.HasDatabaseName("ix_refresh_tokens_user_id");
b.ToTable("refresh_tokens", "identity");
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Aggregates.Users.User", b =>
{
b.Property<Guid>("Id")
.ValueGeneratedOnAdd()
.HasColumnType("uuid")
.HasColumnName("id");
b.Property<string>("AuthProvider")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("auth_provider");
b.Property<string>("AvatarUrl")
.HasMaxLength(500)
.HasColumnType("character varying(500)")
.HasColumnName("avatar_url");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("created_at");
b.Property<string>("Email")
.IsRequired()
.HasMaxLength(255)
.HasColumnType("character varying(255)")
.HasColumnName("email");
b.Property<string>("EmailVerificationToken")
.HasMaxLength(255)
.HasColumnType("character varying(255)")
.HasColumnName("email_verification_token");
b.Property<DateTime?>("EmailVerifiedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("email_verified_at");
b.Property<string>("ExternalEmail")
.HasMaxLength(255)
.HasColumnType("character varying(255)")
.HasColumnName("external_email");
b.Property<string>("ExternalUserId")
.HasMaxLength(255)
.HasColumnType("character varying(255)")
.HasColumnName("external_user_id");
b.Property<string>("FullName")
.IsRequired()
.HasMaxLength(100)
.HasColumnType("character varying(100)")
.HasColumnName("full_name");
b.Property<string>("JobTitle")
.HasMaxLength(100)
.HasColumnType("character varying(100)")
.HasColumnName("job_title");
b.Property<DateTime?>("LastLoginAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("last_login_at");
b.Property<string>("PasswordHash")
.IsRequired()
.HasMaxLength(255)
.HasColumnType("character varying(255)")
.HasColumnName("password_hash");
b.Property<string>("PasswordResetToken")
.HasMaxLength(255)
.HasColumnType("character varying(255)")
.HasColumnName("password_reset_token");
b.Property<DateTime?>("PasswordResetTokenExpiresAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("password_reset_token_expires_at");
b.Property<string>("PhoneNumber")
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("phone_number");
b.Property<string>("Status")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("status");
b.Property<Guid>("TenantId")
.HasColumnType("uuid")
.HasColumnName("tenant_id");
b.Property<DateTime?>("UpdatedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("updated_at");
b.HasKey("Id");
b.HasIndex("TenantId", "Email")
.IsUnique()
.HasDatabaseName("ix_users_tenant_id_email");
b.ToTable("users", "identity");
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Aggregates.Users.UserTenantRole", b =>
{
b.Property<Guid>("Id")
.ValueGeneratedOnAdd()
.HasColumnType("uuid")
.HasColumnName("id");
b.Property<DateTime>("AssignedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("assigned_at");
b.Property<Guid?>("AssignedByUserId")
.HasColumnType("uuid")
.HasColumnName("assigned_by_user_id");
b.Property<string>("Role")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("role");
b.Property<Guid>("TenantId")
.HasColumnType("uuid")
.HasColumnName("tenant_id");
b.Property<Guid>("UserId")
.HasColumnType("uuid")
.HasColumnName("user_id");
b.HasKey("Id");
b.HasIndex("Role")
.HasDatabaseName("ix_user_tenant_roles_role");
b.HasIndex("TenantId")
.HasDatabaseName("ix_user_tenant_roles_tenant_id");
b.HasIndex("UserId")
.HasDatabaseName("ix_user_tenant_roles_user_id");
b.HasIndex("TenantId", "Role")
.HasDatabaseName("ix_user_tenant_roles_tenant_role");
b.HasIndex("UserId", "TenantId")
.IsUnique()
.HasDatabaseName("uq_user_tenant_roles_user_tenant");
b.ToTable("user_tenant_roles", "identity");
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Entities.EmailRateLimit", b =>
{
b.Property<Guid>("Id")
.ValueGeneratedOnAdd()
.HasColumnType("uuid");
b.Property<int>("AttemptsCount")
.HasColumnType("integer")
.HasColumnName("attempts_count");
b.Property<string>("Email")
.IsRequired()
.HasMaxLength(255)
.HasColumnType("character varying(255)")
.HasColumnName("email");
b.Property<DateTime>("LastSentAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("last_sent_at");
b.Property<string>("OperationType")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)")
.HasColumnName("operation_type");
b.Property<Guid>("TenantId")
.HasColumnType("uuid")
.HasColumnName("tenant_id");
b.HasKey("Id");
b.HasIndex("LastSentAt")
.HasDatabaseName("ix_email_rate_limits_last_sent_at");
b.HasIndex("Email", "TenantId", "OperationType")
.IsUnique()
.HasDatabaseName("ix_email_rate_limits_email_tenant_operation");
b.ToTable("email_rate_limits", "identity");
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Entities.EmailVerificationToken", b =>
{
b.Property<Guid>("Id")
.ValueGeneratedOnAdd()
.HasColumnType("uuid")
.HasColumnName("id");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("created_at");
b.Property<DateTime>("ExpiresAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("expires_at");
b.Property<string>("TokenHash")
.IsRequired()
.HasMaxLength(64)
.HasColumnType("character varying(64)")
.HasColumnName("token_hash");
b.Property<Guid>("UserId")
.HasColumnType("uuid")
.HasColumnName("user_id");
b.Property<DateTime?>("VerifiedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("verified_at");
b.HasKey("Id");
b.HasIndex("TokenHash")
.HasDatabaseName("ix_email_verification_tokens_token_hash");
b.HasIndex("UserId")
.HasDatabaseName("ix_email_verification_tokens_user_id");
b.ToTable("email_verification_tokens", (string)null);
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Entities.PasswordResetToken", b =>
{
b.Property<Guid>("Id")
.ValueGeneratedOnAdd()
.HasColumnType("uuid")
.HasColumnName("id");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("created_at");
b.Property<DateTime>("ExpiresAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("expires_at");
b.Property<string>("IpAddress")
.HasMaxLength(45)
.HasColumnType("character varying(45)")
.HasColumnName("ip_address");
b.Property<string>("TokenHash")
.IsRequired()
.HasMaxLength(64)
.HasColumnType("character varying(64)")
.HasColumnName("token_hash");
b.Property<DateTime?>("UsedAt")
.HasColumnType("timestamp with time zone")
.HasColumnName("used_at");
b.Property<string>("UserAgent")
.HasMaxLength(500)
.HasColumnType("character varying(500)")
.HasColumnName("user_agent");
b.Property<Guid>("UserId")
.HasColumnType("uuid")
.HasColumnName("user_id");
b.HasKey("Id");
b.HasIndex("TokenHash")
.HasDatabaseName("ix_password_reset_tokens_token_hash");
b.HasIndex("UserId")
.HasDatabaseName("ix_password_reset_tokens_user_id");
b.HasIndex("UserId", "ExpiresAt", "UsedAt")
.HasDatabaseName("ix_password_reset_tokens_user_active");
b.ToTable("password_reset_tokens", (string)null);
});
#pragma warning restore 612, 618
}
}
}

View File

@@ -0,0 +1,38 @@
using Microsoft.EntityFrameworkCore.Migrations;
#nullable disable
namespace ColaFlow.Modules.Identity.Infrastructure.Persistence.Migrations
{
/// <inheritdoc />
public partial class MoveTablesToIdentitySchemaAndAddIndexes : Migration
{
/// <inheritdoc />
protected override void Up(MigrationBuilder migrationBuilder)
{
migrationBuilder.RenameTable(
name: "users",
newName: "users",
newSchema: "identity");
migrationBuilder.RenameTable(
name: "tenants",
newName: "tenants",
newSchema: "identity");
}
/// <inheritdoc />
protected override void Down(MigrationBuilder migrationBuilder)
{
migrationBuilder.RenameTable(
name: "users",
schema: "identity",
newName: "users");
migrationBuilder.RenameTable(
name: "tenants",
schema: "identity",
newName: "tenants");
}
}
}

View File

@@ -12,7 +12,7 @@ using Npgsql.EntityFrameworkCore.PostgreSQL.Metadata;
namespace ColaFlow.Modules.Identity.Infrastructure.Persistence.Migrations
{
[DbContext(typeof(IdentityDbContext))]
[Migration("20251103225606_AddPerformanceIndexes")]
[Migration("20251104085234_AddPerformanceIndexes")]
partial class AddPerformanceIndexes
{
/// <inheritdoc />
@@ -158,7 +158,7 @@ namespace ColaFlow.Modules.Identity.Infrastructure.Persistence.Migrations
.IsUnique()
.HasDatabaseName("ix_tenants_slug");
b.ToTable("tenants", (string)null);
b.ToTable("tenants", "identity");
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Aggregates.Users.RefreshToken", b =>
@@ -338,7 +338,7 @@ namespace ColaFlow.Modules.Identity.Infrastructure.Persistence.Migrations
.IsUnique()
.HasDatabaseName("ix_users_tenant_id_email");
b.ToTable("users", (string)null);
b.ToTable("users", "identity");
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Aggregates.Users.UserTenantRole", b =>

View File

@@ -0,0 +1,29 @@
using Microsoft.EntityFrameworkCore.Migrations;
#nullable disable
namespace ColaFlow.Modules.Identity.Infrastructure.Persistence.Migrations
{
/// <inheritdoc />
public partial class AddPerformanceIndexes : Migration
{
/// <inheritdoc />
protected override void Up(MigrationBuilder migrationBuilder)
{
// Index for email lookups (case-insensitive for PostgreSQL)
migrationBuilder.Sql(@"
CREATE INDEX IF NOT EXISTS idx_users_email_lower
ON identity.users(LOWER(email));
");
// Only create indexes for tables that exist
// Note: Some tables may not have been moved to identity schema yet
}
/// <inheritdoc />
protected override void Down(MigrationBuilder migrationBuilder)
{
migrationBuilder.Sql("DROP INDEX IF EXISTS identity.idx_users_email_lower;");
}
}
}

View File

@@ -155,7 +155,7 @@ namespace ColaFlow.Modules.Identity.Infrastructure.Persistence.Migrations
.IsUnique()
.HasDatabaseName("ix_tenants_slug");
b.ToTable("tenants", (string)null);
b.ToTable("tenants", "identity");
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Aggregates.Users.RefreshToken", b =>
@@ -335,7 +335,7 @@ namespace ColaFlow.Modules.Identity.Infrastructure.Persistence.Migrations
.IsUnique()
.HasDatabaseName("ix_users_tenant_id_email");
b.ToTable("users", (string)null);
b.ToTable("users", "identity");
});
modelBuilder.Entity("ColaFlow.Modules.Identity.Domain.Aggregates.Users.UserTenantRole", b =>

View File

@@ -0,0 +1,8 @@
using MediatR;
namespace ColaFlow.Modules.ProjectManagement.Application.Commands.ArchiveProject;
/// <summary>
/// Command to archive a project
/// </summary>
public sealed record ArchiveProjectCommand(Guid ProjectId) : IRequest<Unit>;

View File

@@ -0,0 +1,37 @@
using MediatR;
using ColaFlow.Modules.ProjectManagement.Domain.Repositories;
using ColaFlow.Modules.ProjectManagement.Domain.ValueObjects;
using ColaFlow.Modules.ProjectManagement.Domain.Exceptions;
namespace ColaFlow.Modules.ProjectManagement.Application.Commands.ArchiveProject;
/// <summary>
/// Handler for ArchiveProjectCommand
/// </summary>
public sealed class ArchiveProjectCommandHandler(
IProjectRepository projectRepository,
IUnitOfWork unitOfWork)
: IRequestHandler<ArchiveProjectCommand, Unit>
{
private readonly IProjectRepository _projectRepository = projectRepository ?? throw new ArgumentNullException(nameof(projectRepository));
private readonly IUnitOfWork _unitOfWork = unitOfWork ?? throw new ArgumentNullException(nameof(unitOfWork));
public async Task<Unit> Handle(ArchiveProjectCommand request, CancellationToken cancellationToken)
{
// Get project (will be filtered by tenant automatically)
var project = await _projectRepository.GetByIdAsync(ProjectId.From(request.ProjectId), cancellationToken);
if (project == null)
{
throw new NotFoundException($"Project with ID '{request.ProjectId}' not found");
}
// Archive project
project.Archive();
// Save changes
_projectRepository.Update(project);
await _unitOfWork.SaveChangesAsync(cancellationToken);
return Unit.Value;
}
}

View File

@@ -0,0 +1,16 @@
using FluentValidation;
namespace ColaFlow.Modules.ProjectManagement.Application.Commands.ArchiveProject;
/// <summary>
/// Validator for ArchiveProjectCommand
/// </summary>
public class ArchiveProjectCommandValidator : AbstractValidator<ArchiveProjectCommand>
{
public ArchiveProjectCommandValidator()
{
RuleFor(x => x.ProjectId)
.NotEmpty()
.WithMessage("ProjectId is required");
}
}

View File

@@ -8,6 +8,7 @@ namespace ColaFlow.Modules.ProjectManagement.Application.Commands.CreateProject;
/// </summary>
public sealed record CreateProjectCommand : IRequest<ProjectDto>
{
public Guid TenantId { get; init; }
public string Name { get; init; } = string.Empty;
public string Description { get; init; } = string.Empty;
public string Key { get; init; } = string.Empty;

View File

@@ -29,6 +29,7 @@ public sealed class CreateProjectCommandHandler(
// Create project aggregate
var project = Project.Create(
TenantId.From(request.TenantId),
request.Name,
request.Description,
request.Key,

View File

@@ -18,7 +18,7 @@ public sealed class CreateProjectCommandValidator : AbstractValidator<CreateProj
.MaximumLength(20).WithMessage("Project key cannot exceed 20 characters")
.Matches("^[A-Z0-9]+$").WithMessage("Project key must contain only uppercase letters and numbers");
RuleFor(x => x.OwnerId)
.NotEmpty().WithMessage("Owner ID is required");
// TenantId and OwnerId are set by the controller from JWT claims, not from request body
// So we don't validate them here (they'll be Guid.Empty from request, then overridden)
}
}

View File

@@ -0,0 +1,14 @@
using MediatR;
using ColaFlow.Modules.ProjectManagement.Application.DTOs;
namespace ColaFlow.Modules.ProjectManagement.Application.Commands.UpdateProject;
/// <summary>
/// Command to update an existing project
/// </summary>
public sealed record UpdateProjectCommand : IRequest<ProjectDto>
{
public Guid ProjectId { get; init; }
public string Name { get; init; } = string.Empty;
public string Description { get; init; } = string.Empty;
}

View File

@@ -0,0 +1,50 @@
using MediatR;
using ColaFlow.Modules.ProjectManagement.Application.DTOs;
using ColaFlow.Modules.ProjectManagement.Domain.Repositories;
using ColaFlow.Modules.ProjectManagement.Domain.ValueObjects;
using ColaFlow.Modules.ProjectManagement.Domain.Exceptions;
namespace ColaFlow.Modules.ProjectManagement.Application.Commands.UpdateProject;
/// <summary>
/// Handler for UpdateProjectCommand
/// </summary>
public sealed class UpdateProjectCommandHandler(
IProjectRepository projectRepository,
IUnitOfWork unitOfWork)
: IRequestHandler<UpdateProjectCommand, ProjectDto>
{
private readonly IProjectRepository _projectRepository = projectRepository ?? throw new ArgumentNullException(nameof(projectRepository));
private readonly IUnitOfWork _unitOfWork = unitOfWork ?? throw new ArgumentNullException(nameof(unitOfWork));
public async Task<ProjectDto> Handle(UpdateProjectCommand request, CancellationToken cancellationToken)
{
// Get project (will be filtered by tenant automatically)
var project = await _projectRepository.GetByIdAsync(ProjectId.From(request.ProjectId), cancellationToken);
if (project == null)
{
throw new NotFoundException($"Project with ID '{request.ProjectId}' not found");
}
// Update project details
project.UpdateDetails(request.Name, request.Description);
// Save changes
_projectRepository.Update(project);
await _unitOfWork.SaveChangesAsync(cancellationToken);
// Return DTO
return new ProjectDto
{
Id = project.Id.Value,
Name = project.Name,
Description = project.Description,
Key = project.Key.Value,
Status = project.Status.Name,
OwnerId = project.OwnerId.Value,
CreatedAt = project.CreatedAt,
UpdatedAt = project.UpdatedAt,
Epics = new List<EpicDto>()
};
}
}

View File

@@ -0,0 +1,26 @@
using FluentValidation;
namespace ColaFlow.Modules.ProjectManagement.Application.Commands.UpdateProject;
/// <summary>
/// Validator for UpdateProjectCommand
/// </summary>
public class UpdateProjectCommandValidator : AbstractValidator<UpdateProjectCommand>
{
public UpdateProjectCommandValidator()
{
RuleFor(x => x.ProjectId)
.NotEmpty()
.WithMessage("ProjectId is required");
RuleFor(x => x.Name)
.NotEmpty()
.WithMessage("Project name is required")
.MaximumLength(200)
.WithMessage("Project name cannot exceed 200 characters");
RuleFor(x => x.Description)
.MaximumLength(2000)
.WithMessage("Project description cannot exceed 2000 characters");
}
}

View File

@@ -0,0 +1,46 @@
using MediatR;
using Microsoft.Extensions.Logging;
using ColaFlow.Modules.ProjectManagement.Domain.Events;
using ColaFlow.Modules.ProjectManagement.Domain.Repositories;
using ColaFlow.Modules.ProjectManagement.Application.Services;
namespace ColaFlow.Modules.ProjectManagement.Application.EventHandlers;
/// <summary>
/// Handler for ProjectArchivedEvent - sends SignalR notification
/// </summary>
public class ProjectArchivedEventHandler : INotificationHandler<ProjectArchivedEvent>
{
private readonly IProjectNotificationService _notificationService;
private readonly IProjectRepository _projectRepository;
private readonly ILogger<ProjectArchivedEventHandler> _logger;
public ProjectArchivedEventHandler(
IProjectNotificationService notificationService,
IProjectRepository projectRepository,
ILogger<ProjectArchivedEventHandler> logger)
{
_notificationService = notificationService;
_projectRepository = projectRepository;
_logger = logger;
}
public async Task Handle(ProjectArchivedEvent notification, CancellationToken cancellationToken)
{
_logger.LogInformation("Handling ProjectArchivedEvent for project {ProjectId}", notification.ProjectId);
// Get full project to obtain TenantId
var project = await _projectRepository.GetByIdAsync(notification.ProjectId, cancellationToken);
if (project == null)
{
_logger.LogWarning("Project {ProjectId} not found for archive notification", notification.ProjectId);
return;
}
await _notificationService.NotifyProjectArchived(
project.TenantId.Value,
notification.ProjectId.Value);
_logger.LogInformation("SignalR notification sent for archived project {ProjectId}", notification.ProjectId);
}
}

View File

@@ -0,0 +1,43 @@
using MediatR;
using Microsoft.Extensions.Logging;
using ColaFlow.Modules.ProjectManagement.Domain.Events;
using ColaFlow.Modules.ProjectManagement.Application.Services;
namespace ColaFlow.Modules.ProjectManagement.Application.EventHandlers;
/// <summary>
/// Handler for ProjectCreatedEvent - sends SignalR notification
/// </summary>
public class ProjectCreatedEventHandler : INotificationHandler<ProjectCreatedEvent>
{
private readonly IProjectNotificationService _notificationService;
private readonly ILogger<ProjectCreatedEventHandler> _logger;
public ProjectCreatedEventHandler(
IProjectNotificationService notificationService,
ILogger<ProjectCreatedEventHandler> logger)
{
_notificationService = notificationService;
_logger = logger;
}
public async Task Handle(ProjectCreatedEvent notification, CancellationToken cancellationToken)
{
_logger.LogInformation("Handling ProjectCreatedEvent for project {ProjectId}", notification.ProjectId);
var projectData = new
{
Id = notification.ProjectId.Value,
Name = notification.ProjectName,
CreatedBy = notification.CreatedBy.Value,
CreatedAt = DateTime.UtcNow
};
await _notificationService.NotifyProjectCreated(
notification.TenantId.Value,
notification.ProjectId.Value,
projectData);
_logger.LogInformation("SignalR notification sent for project {ProjectId}", notification.ProjectId);
}
}

View File

@@ -0,0 +1,55 @@
using MediatR;
using Microsoft.Extensions.Logging;
using ColaFlow.Modules.ProjectManagement.Domain.Events;
using ColaFlow.Modules.ProjectManagement.Domain.Repositories;
using ColaFlow.Modules.ProjectManagement.Application.Services;
namespace ColaFlow.Modules.ProjectManagement.Application.EventHandlers;
/// <summary>
/// Handler for ProjectUpdatedEvent - sends SignalR notification
/// </summary>
public class ProjectUpdatedEventHandler : INotificationHandler<ProjectUpdatedEvent>
{
private readonly IProjectNotificationService _notificationService;
private readonly IProjectRepository _projectRepository;
private readonly ILogger<ProjectUpdatedEventHandler> _logger;
public ProjectUpdatedEventHandler(
IProjectNotificationService notificationService,
IProjectRepository projectRepository,
ILogger<ProjectUpdatedEventHandler> logger)
{
_notificationService = notificationService;
_projectRepository = projectRepository;
_logger = logger;
}
public async Task Handle(ProjectUpdatedEvent notification, CancellationToken cancellationToken)
{
_logger.LogInformation("Handling ProjectUpdatedEvent for project {ProjectId}", notification.ProjectId);
// Get full project to obtain TenantId
var project = await _projectRepository.GetByIdAsync(notification.ProjectId, cancellationToken);
if (project == null)
{
_logger.LogWarning("Project {ProjectId} not found for update notification", notification.ProjectId);
return;
}
var projectData = new
{
Id = notification.ProjectId.Value,
Name = notification.Name,
Description = notification.Description,
UpdatedAt = DateTime.UtcNow
};
await _notificationService.NotifyProjectUpdated(
project.TenantId.Value,
notification.ProjectId.Value,
projectData);
_logger.LogInformation("SignalR notification sent for updated project {ProjectId}", notification.ProjectId);
}
}

View File

@@ -0,0 +1,11 @@
namespace ColaFlow.Modules.ProjectManagement.Application.Services;
/// <summary>
/// Service for sending project-related notifications (abstraction for SignalR)
/// </summary>
public interface IProjectNotificationService
{
Task NotifyProjectCreated(Guid tenantId, Guid projectId, object project);
Task NotifyProjectUpdated(Guid tenantId, Guid projectId, object project);
Task NotifyProjectArchived(Guid tenantId, Guid projectId);
}

View File

@@ -13,6 +13,7 @@ namespace ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate;
public class Project : AggregateRoot
{
public new ProjectId Id { get; private set; }
public TenantId TenantId { get; private set; }
public string Name { get; private set; }
public string Description { get; private set; }
public ProjectKey Key { get; private set; }
@@ -29,6 +30,7 @@ public class Project : AggregateRoot
private Project()
{
Id = null!;
TenantId = null!;
Name = null!;
Description = null!;
Key = null!;
@@ -37,7 +39,7 @@ public class Project : AggregateRoot
}
// Factory method
public static Project Create(string name, string description, string key, UserId ownerId)
public static Project Create(TenantId tenantId, string name, string description, string key, UserId ownerId)
{
// Validation
if (string.IsNullOrWhiteSpace(name))
@@ -49,6 +51,7 @@ public class Project : AggregateRoot
var project = new Project
{
Id = ProjectId.Create(),
TenantId = tenantId,
Name = name,
Description = description ?? string.Empty,
Key = ProjectKey.Create(key),
@@ -58,7 +61,7 @@ public class Project : AggregateRoot
};
// Raise domain event
project.AddDomainEvent(new ProjectCreatedEvent(project.Id, project.Name, ownerId));
project.AddDomainEvent(new ProjectCreatedEvent(project.Id, project.TenantId, project.Name, ownerId));
return project;
}

View File

@@ -8,6 +8,7 @@ namespace ColaFlow.Modules.ProjectManagement.Domain.Events;
/// </summary>
public sealed record ProjectCreatedEvent(
ProjectId ProjectId,
TenantId TenantId,
string ProjectName,
UserId CreatedBy
) : DomainEvent;

View File

@@ -0,0 +1,29 @@
using ColaFlow.Shared.Kernel.Common;
namespace ColaFlow.Modules.ProjectManagement.Domain.ValueObjects;
/// <summary>
/// TenantId Value Object (strongly-typed ID)
/// </summary>
public sealed class TenantId : ValueObject
{
public Guid Value { get; private set; }
private TenantId(Guid value)
{
if (value == Guid.Empty)
throw new ArgumentException("TenantId cannot be empty", nameof(value));
Value = value;
}
public static TenantId Create(Guid value) => new TenantId(value);
public static TenantId From(Guid value) => new TenantId(value);
protected override IEnumerable<object> GetAtomicValues()
{
yield return Value;
}
public override string ToString() => Value.ToString();
}

View File

@@ -0,0 +1,304 @@
// <auto-generated />
using System;
using ColaFlow.Modules.ProjectManagement.Infrastructure.Persistence;
using Microsoft.EntityFrameworkCore;
using Microsoft.EntityFrameworkCore.Infrastructure;
using Microsoft.EntityFrameworkCore.Migrations;
using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
using Npgsql.EntityFrameworkCore.PostgreSQL.Metadata;
#nullable disable
namespace ColaFlow.Modules.ProjectManagement.Infrastructure.Migrations
{
[DbContext(typeof(PMDbContext))]
[Migration("20251104092845_AddTenantIdToProject")]
partial class AddTenantIdToProject
{
/// <inheritdoc />
protected override void BuildTargetModel(ModelBuilder modelBuilder)
{
#pragma warning disable 612, 618
modelBuilder
.HasDefaultSchema("project_management")
.HasAnnotation("ProductVersion", "9.0.10")
.HasAnnotation("Relational:MaxIdentifierLength", 63);
NpgsqlModelBuilderExtensions.UseIdentityByDefaultColumns(modelBuilder);
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Epic", b =>
{
b.Property<Guid>("Id")
.HasColumnType("uuid");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone");
b.Property<Guid>("CreatedBy")
.HasColumnType("uuid");
b.Property<string>("Description")
.IsRequired()
.HasMaxLength(2000)
.HasColumnType("character varying(2000)");
b.Property<string>("Name")
.IsRequired()
.HasMaxLength(200)
.HasColumnType("character varying(200)");
b.Property<string>("Priority")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)");
b.Property<Guid>("ProjectId")
.HasColumnType("uuid");
b.Property<string>("Status")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)");
b.Property<DateTime?>("UpdatedAt")
.HasColumnType("timestamp with time zone");
b.HasKey("Id");
b.HasIndex("CreatedAt");
b.HasIndex("ProjectId");
b.ToTable("Epics", "project_management");
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Project", b =>
{
b.Property<Guid>("Id")
.HasColumnType("uuid");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone");
b.Property<string>("Description")
.IsRequired()
.HasMaxLength(2000)
.HasColumnType("character varying(2000)");
b.Property<string>("Name")
.IsRequired()
.HasMaxLength(200)
.HasColumnType("character varying(200)");
b.Property<Guid>("OwnerId")
.HasColumnType("uuid");
b.Property<string>("Status")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)");
b.Property<Guid>("TenantId")
.HasColumnType("uuid");
b.Property<DateTime?>("UpdatedAt")
.HasColumnType("timestamp with time zone");
b.HasKey("Id");
b.HasIndex("CreatedAt");
b.HasIndex("OwnerId");
b.HasIndex("TenantId");
b.ToTable("Projects", "project_management");
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Story", b =>
{
b.Property<Guid>("Id")
.HasColumnType("uuid");
b.Property<decimal?>("ActualHours")
.HasColumnType("numeric");
b.Property<Guid?>("AssigneeId")
.HasColumnType("uuid");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone");
b.Property<Guid>("CreatedBy")
.HasColumnType("uuid");
b.Property<string>("Description")
.IsRequired()
.HasMaxLength(4000)
.HasColumnType("character varying(4000)");
b.Property<Guid>("EpicId")
.HasColumnType("uuid");
b.Property<decimal?>("EstimatedHours")
.HasColumnType("numeric");
b.Property<string>("Priority")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)");
b.Property<string>("Status")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)");
b.Property<string>("Title")
.IsRequired()
.HasMaxLength(200)
.HasColumnType("character varying(200)");
b.Property<DateTime?>("UpdatedAt")
.HasColumnType("timestamp with time zone");
b.HasKey("Id");
b.HasIndex("AssigneeId");
b.HasIndex("CreatedAt");
b.HasIndex("EpicId");
b.ToTable("Stories", "project_management");
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.WorkTask", b =>
{
b.Property<Guid>("Id")
.HasColumnType("uuid");
b.Property<decimal?>("ActualHours")
.HasColumnType("numeric");
b.Property<Guid?>("AssigneeId")
.HasColumnType("uuid");
b.Property<DateTime>("CreatedAt")
.HasColumnType("timestamp with time zone");
b.Property<Guid>("CreatedBy")
.HasColumnType("uuid");
b.Property<string>("Description")
.IsRequired()
.HasMaxLength(4000)
.HasColumnType("character varying(4000)");
b.Property<decimal?>("EstimatedHours")
.HasColumnType("numeric");
b.Property<string>("Priority")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)");
b.Property<string>("Status")
.IsRequired()
.HasMaxLength(50)
.HasColumnType("character varying(50)");
b.Property<Guid>("StoryId")
.HasColumnType("uuid");
b.Property<string>("Title")
.IsRequired()
.HasMaxLength(200)
.HasColumnType("character varying(200)");
b.Property<DateTime?>("UpdatedAt")
.HasColumnType("timestamp with time zone");
b.HasKey("Id");
b.HasIndex("AssigneeId");
b.HasIndex("CreatedAt");
b.HasIndex("StoryId");
b.ToTable("Tasks", "project_management");
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Epic", b =>
{
b.HasOne("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Project", null)
.WithMany("Epics")
.HasForeignKey("ProjectId")
.OnDelete(DeleteBehavior.Cascade)
.IsRequired();
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Project", b =>
{
b.OwnsOne("ColaFlow.Modules.ProjectManagement.Domain.ValueObjects.ProjectKey", "Key", b1 =>
{
b1.Property<Guid>("ProjectId")
.HasColumnType("uuid");
b1.Property<string>("Value")
.IsRequired()
.HasMaxLength(20)
.HasColumnType("character varying(20)")
.HasColumnName("Key");
b1.HasKey("ProjectId");
b1.HasIndex("Value")
.IsUnique();
b1.ToTable("Projects", "project_management");
b1.WithOwner()
.HasForeignKey("ProjectId");
});
b.Navigation("Key")
.IsRequired();
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Story", b =>
{
b.HasOne("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Epic", null)
.WithMany("Stories")
.HasForeignKey("EpicId")
.OnDelete(DeleteBehavior.Cascade)
.IsRequired();
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.WorkTask", b =>
{
b.HasOne("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Story", null)
.WithMany("Tasks")
.HasForeignKey("StoryId")
.OnDelete(DeleteBehavior.Cascade)
.IsRequired();
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Epic", b =>
{
b.Navigation("Stories");
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Project", b =>
{
b.Navigation("Epics");
});
modelBuilder.Entity("ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate.Story", b =>
{
b.Navigation("Tasks");
});
#pragma warning restore 612, 618
}
}
}

View File

@@ -0,0 +1,43 @@
using System;
using Microsoft.EntityFrameworkCore.Migrations;
#nullable disable
namespace ColaFlow.Modules.ProjectManagement.Infrastructure.Migrations
{
/// <inheritdoc />
public partial class AddTenantIdToProject : Migration
{
/// <inheritdoc />
protected override void Up(MigrationBuilder migrationBuilder)
{
migrationBuilder.AddColumn<Guid>(
name: "TenantId",
schema: "project_management",
table: "Projects",
type: "uuid",
nullable: false,
defaultValue: new Guid("00000000-0000-0000-0000-000000000000"));
migrationBuilder.CreateIndex(
name: "IX_Projects_TenantId",
schema: "project_management",
table: "Projects",
column: "TenantId");
}
/// <inheritdoc />
protected override void Down(MigrationBuilder migrationBuilder)
{
migrationBuilder.DropIndex(
name: "IX_Projects_TenantId",
schema: "project_management",
table: "Projects");
migrationBuilder.DropColumn(
name: "TenantId",
schema: "project_management",
table: "Projects");
}
}
}

View File

@@ -18,7 +18,7 @@ namespace ColaFlow.Modules.ProjectManagement.Infrastructure.Migrations
#pragma warning disable 612, 618
modelBuilder
.HasDefaultSchema("project_management")
.HasAnnotation("ProductVersion", "9.0.0")
.HasAnnotation("ProductVersion", "9.0.10")
.HasAnnotation("Relational:MaxIdentifierLength", 63);
NpgsqlModelBuilderExtensions.UseIdentityByDefaultColumns(modelBuilder);
@@ -95,6 +95,9 @@ namespace ColaFlow.Modules.ProjectManagement.Infrastructure.Migrations
.HasMaxLength(50)
.HasColumnType("character varying(50)");
b.Property<Guid>("TenantId")
.HasColumnType("uuid");
b.Property<DateTime?>("UpdatedAt")
.HasColumnType("timestamp with time zone");
@@ -104,6 +107,8 @@ namespace ColaFlow.Modules.ProjectManagement.Infrastructure.Migrations
b.HasIndex("OwnerId");
b.HasIndex("TenantId");
b.ToTable("Projects", "project_management");
});

View File

@@ -26,6 +26,13 @@ public class ProjectConfiguration : IEntityTypeConfiguration<Project>
.IsRequired()
.ValueGeneratedNever();
// TenantId conversion
builder.Property(p => p.TenantId)
.HasConversion(
id => id.Value,
value => TenantId.From(value))
.IsRequired();
// Basic properties
builder.Property(p => p.Name)
.HasMaxLength(200)
@@ -77,6 +84,7 @@ public class ProjectConfiguration : IEntityTypeConfiguration<Project>
// Indexes for performance
builder.HasIndex(p => p.CreatedAt);
builder.HasIndex(p => p.OwnerId);
builder.HasIndex(p => p.TenantId);
// Ignore DomainEvents (handled separately)
builder.Ignore(p => p.DomainEvents);

View File

@@ -1,14 +1,24 @@
using System.Reflection;
using Microsoft.AspNetCore.Http;
using Microsoft.EntityFrameworkCore;
using ColaFlow.Modules.ProjectManagement.Domain.Aggregates.ProjectAggregate;
using ColaFlow.Modules.ProjectManagement.Domain.ValueObjects;
namespace ColaFlow.Modules.ProjectManagement.Infrastructure.Persistence;
/// <summary>
/// Project Management Module DbContext
/// </summary>
public class PMDbContext(DbContextOptions<PMDbContext> options) : DbContext(options)
public class PMDbContext : DbContext
{
private readonly IHttpContextAccessor _httpContextAccessor;
public PMDbContext(DbContextOptions<PMDbContext> options, IHttpContextAccessor httpContextAccessor)
: base(options)
{
_httpContextAccessor = httpContextAccessor;
}
public DbSet<Project> Projects => Set<Project>();
public DbSet<Epic> Epics => Set<Epic>();
public DbSet<Story> Stories => Set<Story>();
@@ -23,5 +33,24 @@ public class PMDbContext(DbContextOptions<PMDbContext> options) : DbContext(opti
// Apply all entity configurations from this assembly
modelBuilder.ApplyConfigurationsFromAssembly(Assembly.GetExecutingAssembly());
// Multi-tenant Global Query Filter for Project
modelBuilder.Entity<Project>().HasQueryFilter(p =>
p.TenantId == GetCurrentTenantId());
}
private TenantId GetCurrentTenantId()
{
var tenantIdClaim = _httpContextAccessor?.HttpContext?.User
.FindFirst("tenant_id")?.Value;
if (Guid.TryParse(tenantIdClaim, out var tenantId) && tenantId != Guid.Empty)
{
return TenantId.From(tenantId);
}
// Return a dummy value for queries outside HTTP context (e.g., migrations)
// These will return no results due to the filter
return TenantId.From(Guid.Empty);
}
}

View File

@@ -32,6 +32,9 @@ public class ProjectManagementModule : IModule
services.AddScoped<IProjectRepository, ProjectRepository>();
services.AddScoped<IUnitOfWork, UnitOfWork>();
// Note: IProjectNotificationService is registered in the API layer (Program.cs)
// as it depends on IRealtimeNotificationService which is API-specific
// Register MediatR handlers from Application assembly
services.AddMediatR(cfg =>
{

View File

@@ -0,0 +1,268 @@
# Test script for ColaFlow Project Management API
# Day 12 - Complete CRUD + Multi-Tenant + SignalR Integration
$baseUrl = "http://localhost:5167"
$ErrorActionPreference = "Stop"
Write-Host "========================================" -ForegroundColor Cyan
Write-Host "ColaFlow Project API Test" -ForegroundColor Cyan
Write-Host "========================================" -ForegroundColor Cyan
Write-Host ""
# Step 1: Register a new tenant and get access token
Write-Host "[1] Registering new tenant..." -ForegroundColor Yellow
$tenantSlug = "test-project-corp-$(Get-Random -Minimum 1000 -Maximum 9999)"
$registerBody = @{
tenantName = "Test Project Corp"
tenantSlug = $tenantSlug
subscriptionPlan = "Professional"
adminEmail = "admin@$tenantSlug.com"
adminPassword = "Admin@1234"
adminFullName = "Project Admin"
} | ConvertTo-Json
try {
$registerResponse = Invoke-RestMethod -Uri "$baseUrl/api/tenants/register" `
-Method Post `
-ContentType "application/json" `
-Body $registerBody
$token = $registerResponse.accessToken
$tenantId = $registerResponse.tenant.id
$userId = $registerResponse.user.id
Write-Host "✓ Tenant registered successfully" -ForegroundColor Green
Write-Host " Tenant ID: $tenantId" -ForegroundColor Gray
Write-Host " User ID: $userId" -ForegroundColor Gray
Write-Host " Token: $($token.Substring(0, 30))..." -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "✗ Failed to register tenant" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
exit 1
}
$headers = @{
"Authorization" = "Bearer $token"
"Content-Type" = "application/json"
}
# Step 2: Create a project
Write-Host "[2] Creating project..." -ForegroundColor Yellow
$createProjectBody = @{
name = "ColaFlow v2.0"
description = "Next generation project management system with AI integration"
key = "COLA"
} | ConvertTo-Json
try {
$project = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Post `
-Headers $headers `
-Body $createProjectBody
$projectId = $project.id
Write-Host "✓ Project created successfully" -ForegroundColor Green
Write-Host " Project ID: $projectId" -ForegroundColor Gray
Write-Host " Name: $($project.name)" -ForegroundColor Gray
Write-Host " Key: $($project.key)" -ForegroundColor Gray
Write-Host " Status: $($project.status)" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "✗ Failed to create project" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
if ($_.ErrorDetails) {
Write-Host $_.ErrorDetails.Message -ForegroundColor Red
}
exit 1
}
# Step 3: Get all projects
Write-Host "[3] Listing all projects..." -ForegroundColor Yellow
try {
$projects = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Get `
-Headers $headers
Write-Host "✓ Retrieved projects successfully" -ForegroundColor Green
Write-Host " Total projects: $($projects.Count)" -ForegroundColor Gray
foreach ($p in $projects) {
Write-Host " - $($p.name) [$($p.key)] - Status: $($p.status)" -ForegroundColor Gray
}
Write-Host ""
} catch {
Write-Host "✗ Failed to list projects" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
exit 1
}
# Step 4: Get specific project by ID
Write-Host "[4] Getting project by ID..." -ForegroundColor Yellow
try {
$retrievedProject = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects/$projectId" `
-Method Get `
-Headers $headers
Write-Host "✓ Retrieved project successfully" -ForegroundColor Green
Write-Host " ID: $($retrievedProject.id)" -ForegroundColor Gray
Write-Host " Name: $($retrievedProject.name)" -ForegroundColor Gray
Write-Host " Description: $($retrievedProject.description)" -ForegroundColor Gray
Write-Host " Status: $($retrievedProject.status)" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "✗ Failed to get project" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
exit 1
}
# Step 5: Update project
Write-Host "[5] Updating project..." -ForegroundColor Yellow
$updateProjectBody = @{
name = "ColaFlow v2.0 - Updated"
description = "Next generation project management system with AI integration - Now with enhanced features"
} | ConvertTo-Json
try {
$updatedProject = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects/$projectId" `
-Method Put `
-Headers $headers `
-Body $updateProjectBody
Write-Host "✓ Project updated successfully" -ForegroundColor Green
Write-Host " New Name: $($updatedProject.name)" -ForegroundColor Gray
Write-Host " New Description: $($updatedProject.description)" -ForegroundColor Gray
Write-Host " Updated At: $($updatedProject.updatedAt)" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "✗ Failed to update project" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
if ($_.ErrorDetails) {
Write-Host $_.ErrorDetails.Message -ForegroundColor Red
}
exit 1
}
# Step 6: Create another project to test multi-tenant isolation
Write-Host "[6] Creating second project..." -ForegroundColor Yellow
$createProject2Body = @{
name = "Internal Tools"
description = "Internal tooling and automation"
key = "TOOLS"
} | ConvertTo-Json
try {
$project2 = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Post `
-Headers $headers `
-Body $createProject2Body
Write-Host "✓ Second project created successfully" -ForegroundColor Green
Write-Host " Project ID: $($project2.id)" -ForegroundColor Gray
Write-Host " Name: $($project2.name)" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "✗ Failed to create second project" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
exit 1
}
# Step 7: Verify both projects are visible
Write-Host "[7] Verifying tenant isolation (listing projects)..." -ForegroundColor Yellow
try {
$allProjects = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Get `
-Headers $headers
Write-Host "✓ Retrieved all tenant projects" -ForegroundColor Green
Write-Host " Total projects: $($allProjects.Count)" -ForegroundColor Gray
if ($allProjects.Count -eq 2) {
Write-Host " ✓ Multi-tenant isolation working correctly" -ForegroundColor Green
} else {
Write-Host " ⚠ Expected 2 projects, got $($allProjects.Count)" -ForegroundColor Yellow
}
Write-Host ""
} catch {
Write-Host "✗ Failed to verify tenant isolation" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
exit 1
}
# Step 8: Archive first project
Write-Host "[8] Archiving project..." -ForegroundColor Yellow
try {
Invoke-RestMethod -Uri "$baseUrl/api/v1/projects/$projectId" `
-Method Delete `
-Headers $headers
Write-Host "✓ Project archived successfully" -ForegroundColor Green
Write-Host ""
} catch {
Write-Host "✗ Failed to archive project" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
if ($_.ErrorDetails) {
Write-Host $_.ErrorDetails.Message -ForegroundColor Red
}
exit 1
}
# Step 9: Verify archived project is no longer in active list
Write-Host "[9] Verifying project archival..." -ForegroundColor Yellow
try {
$archivedProject = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects/$projectId" `
-Method Get `
-Headers $headers
Write-Host "✓ Retrieved archived project" -ForegroundColor Green
Write-Host " Status: $($archivedProject.status)" -ForegroundColor Gray
if ($archivedProject.status -eq "Archived") {
Write-Host " ✓ Project successfully archived" -ForegroundColor Green
} else {
Write-Host " ⚠ Expected status 'Archived', got '$($archivedProject.status)'" -ForegroundColor Yellow
}
Write-Host ""
} catch {
Write-Host "✗ Failed to verify archival" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
exit 1
}
# Step 10: Test unauthorized access (try without token)
Write-Host "[10] Testing authorization (should fail without token)..." -ForegroundColor Yellow
try {
$noAuthHeaders = @{ "Content-Type" = "application/json" }
$response = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Get `
-Headers $noAuthHeaders `
-ErrorAction Stop
Write-Host "X SECURITY ISSUE: Endpoint accessible without authorization!" -ForegroundColor Red
Write-Host ""
} catch {
if ($_.Exception.Response.StatusCode.value__ -eq 401) {
Write-Host "Success: Authorization working correctly (401 Unauthorized)" -ForegroundColor Green
Write-Host ""
} else {
Write-Host "Warning: Unexpected error" -ForegroundColor Yellow
Write-Host ""
}
}
# Summary
Write-Host "========================================" -ForegroundColor Cyan
Write-Host "Test Summary" -ForegroundColor Cyan
Write-Host "========================================" -ForegroundColor Cyan
Write-Host "Success: All tests passed successfully!" -ForegroundColor Green
Write-Host ""
Write-Host "Implemented Features:" -ForegroundColor Cyan
Write-Host " - Complete CRUD operations (Create, Read, Update, Archive)" -ForegroundColor Green
Write-Host " - Multi-tenant isolation (Global Query Filter)" -ForegroundColor Green
Write-Host " - JWT-based authorization" -ForegroundColor Green
Write-Host " - Domain Events (ProjectCreated, ProjectUpdated, ProjectArchived)" -ForegroundColor Green
Write-Host " - SignalR integration ready (Event Handlers registered)" -ForegroundColor Green
Write-Host ""
Write-Host "Project Management Module - Day 12 Complete!" -ForegroundColor Green
Write-Host "========================================" -ForegroundColor Cyan

View File

@@ -0,0 +1,69 @@
# Debug script for Project API
$baseUrl = "http://localhost:5167"
# Register tenant
$tenantSlug = "test-project-$(Get-Random -Minimum 1000 -Maximum 9999)"
$registerBody = @{
tenantName = "Test Project Corp"
tenantSlug = $tenantSlug
subscriptionPlan = "Professional"
adminEmail = "admin@$tenantSlug.com"
adminPassword = "Admin@1234"
adminFullName = "Project Admin"
} | ConvertTo-Json
Write-Host "Registering tenant..." -ForegroundColor Yellow
$registerResponse = Invoke-RestMethod -Uri "$baseUrl/api/tenants/register" `
-Method Post `
-ContentType "application/json" `
-Body $registerBody
$token = $registerResponse.accessToken
Write-Host "Token obtained" -ForegroundColor Green
Write-Host ""
$headers = @{
"Authorization" = "Bearer $token"
"Content-Type" = "application/json"
}
# Try to create project with detailed error handling
$createProjectBody = @{
name = "ColaFlow v2.0"
description = "Test project"
key = "COLA"
} | ConvertTo-Json
Write-Host "Request Body:" -ForegroundColor Cyan
Write-Host $createProjectBody
Write-Host ""
Write-Host "Creating project..." -ForegroundColor Yellow
try {
$project = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Post `
-Headers $headers `
-Body $createProjectBody
Write-Host "SUCCESS!" -ForegroundColor Green
Write-Host $project | ConvertTo-Json
} catch {
Write-Host "ERROR:" -ForegroundColor Red
Write-Host "Status Code: $($_.Exception.Response.StatusCode.value__)"
Write-Host "Message: $($_.Exception.Message)"
if ($_.ErrorDetails) {
Write-Host "Details:" -ForegroundColor Yellow
Write-Host $_.ErrorDetails.Message
}
# Try to read response body
if ($_.Exception.Response) {
$reader = New-Object System.IO.StreamReader($_.Exception.Response.GetResponseStream())
$reader.BaseStream.Position = 0
$reader.DiscardBufferedData()
$responseBody = $reader.ReadToEnd()
Write-Host "Response Body:" -ForegroundColor Yellow
Write-Host $responseBody
}
}

View File

@@ -0,0 +1,219 @@
# Test script for ColaFlow Project Management API
# Day 12 - Complete CRUD + Multi-Tenant + SignalR Integration
$baseUrl = "http://localhost:5167"
$ErrorActionPreference = "Continue"
Write-Host "========================================" -ForegroundColor Cyan
Write-Host "ColaFlow Project API Test" -ForegroundColor Cyan
Write-Host "========================================" -ForegroundColor Cyan
Write-Host ""
# Step 1: Register a new tenant and get access token
Write-Host "[1] Registering new tenant..." -ForegroundColor Yellow
$tenantSlug = "test-project-corp-$(Get-Random -Minimum 1000 -Maximum 9999)"
$registerBody = @{
tenantName = "Test Project Corp"
tenantSlug = $tenantSlug
subscriptionPlan = "Professional"
adminEmail = "admin@$tenantSlug.com"
adminPassword = "Admin@1234"
adminFullName = "Project Admin"
} | ConvertTo-Json
try {
$registerResponse = Invoke-RestMethod -Uri "$baseUrl/api/tenants/register" `
-Method Post `
-ContentType "application/json" `
-Body $registerBody
$token = $registerResponse.accessToken
$tenantId = $registerResponse.tenant.id
$userId = $registerResponse.user.id
Write-Host "[SUCCESS] Tenant registered" -ForegroundColor Green
Write-Host " Tenant ID: $tenantId" -ForegroundColor Gray
Write-Host " User ID: $userId" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "[FAILED] Failed to register tenant" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
exit 1
}
$headers = @{
"Authorization" = "Bearer $token"
"Content-Type" = "application/json"
}
# Step 2: Create a project
Write-Host "[2] Creating project..." -ForegroundColor Yellow
$projectKey = "PRJ$(Get-Random -Minimum 100 -Maximum 999)"
$createProjectBody = @{
name = "ColaFlow v2.0"
description = "Next generation project management system with AI integration"
key = $projectKey
} | ConvertTo-Json
try {
$project = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Post `
-Headers $headers `
-Body $createProjectBody
$projectId = $project.id
Write-Host "[SUCCESS] Project created" -ForegroundColor Green
Write-Host " Project ID: $projectId" -ForegroundColor Gray
Write-Host " Name: $($project.name)" -ForegroundColor Gray
Write-Host " Key: $($project.key)" -ForegroundColor Gray
Write-Host " Status: $($project.status)" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "[FAILED] Failed to create project" -ForegroundColor Red
Write-Host $_.Exception.Message -ForegroundColor Red
exit 1
}
# Step 3: Get all projects
Write-Host "[3] Listing all projects..." -ForegroundColor Yellow
try {
$projects = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Get `
-Headers $headers
Write-Host "[SUCCESS] Retrieved projects" -ForegroundColor Green
Write-Host " Total projects: $($projects.Count)" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "[FAILED] Failed to list projects" -ForegroundColor Red
exit 1
}
# Step 4: Get specific project by ID
Write-Host "[4] Getting project by ID..." -ForegroundColor Yellow
try {
$retrievedProject = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects/$projectId" `
-Method Get `
-Headers $headers
Write-Host "[SUCCESS] Retrieved project" -ForegroundColor Green
Write-Host " Name: $($retrievedProject.name)" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "[FAILED] Failed to get project" -ForegroundColor Red
exit 1
}
# Step 5: Update project
Write-Host "[5] Updating project..." -ForegroundColor Yellow
$updateProjectBody = @{
name = "ColaFlow v2.0 - Updated"
description = "Next generation project management system - Enhanced"
} | ConvertTo-Json
try {
$updatedProject = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects/$projectId" `
-Method Put `
-Headers $headers `
-Body $updateProjectBody
Write-Host "[SUCCESS] Project updated" -ForegroundColor Green
Write-Host " New Name: $($updatedProject.name)" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "[FAILED] Failed to update project" -ForegroundColor Red
exit 1
}
# Step 6: Create another project
Write-Host "[6] Creating second project..." -ForegroundColor Yellow
$projectKey2 = "TOOL$(Get-Random -Minimum 100 -Maximum 999)"
$createProject2Body = @{
name = "Internal Tools"
description = "Internal tooling and automation"
key = $projectKey2
} | ConvertTo-Json
try {
$project2 = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Post `
-Headers $headers `
-Body $createProject2Body
Write-Host "[SUCCESS] Second project created" -ForegroundColor Green
Write-Host " Name: $($project2.name)" -ForegroundColor Gray
Write-Host ""
} catch {
Write-Host "[FAILED] Failed to create second project" -ForegroundColor Red
exit 1
}
# Step 7: Verify both projects are visible
Write-Host "[7] Verifying tenant isolation..." -ForegroundColor Yellow
try {
$allProjects = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects" `
-Method Get `
-Headers $headers
Write-Host "[SUCCESS] Retrieved all tenant projects" -ForegroundColor Green
Write-Host " Total projects: $($allProjects.Count)" -ForegroundColor Gray
if ($allProjects.Count -eq 2) {
Write-Host " [OK] Multi-tenant isolation working" -ForegroundColor Green
}
Write-Host ""
} catch {
Write-Host "[FAILED] Failed to verify tenant isolation" -ForegroundColor Red
exit 1
}
# Step 8: Archive first project
Write-Host "[8] Archiving project..." -ForegroundColor Yellow
try {
Invoke-RestMethod -Uri "$baseUrl/api/v1/projects/$projectId" `
-Method Delete `
-Headers $headers
Write-Host "[SUCCESS] Project archived" -ForegroundColor Green
Write-Host ""
} catch {
Write-Host "[FAILED] Failed to archive project" -ForegroundColor Red
exit 1
}
# Step 9: Verify archived project
Write-Host "[9] Verifying project archival..." -ForegroundColor Yellow
try {
$archivedProject = Invoke-RestMethod -Uri "$baseUrl/api/v1/projects/$projectId" `
-Method Get `
-Headers $headers
Write-Host "[SUCCESS] Retrieved archived project" -ForegroundColor Green
Write-Host " Status: $($archivedProject.status)" -ForegroundColor Gray
if ($archivedProject.status -eq "Archived") {
Write-Host " [OK] Project successfully archived" -ForegroundColor Green
}
Write-Host ""
} catch {
Write-Host "[FAILED] Failed to verify archival" -ForegroundColor Red
exit 1
}
# Summary
Write-Host "========================================" -ForegroundColor Cyan
Write-Host "Test Summary" -ForegroundColor Cyan
Write-Host "========================================" -ForegroundColor Cyan
Write-Host ""
Write-Host "All tests passed successfully!" -ForegroundColor Green
Write-Host ""
Write-Host "Implemented Features:" -ForegroundColor Cyan
Write-Host " - Complete CRUD operations" -ForegroundColor Green
Write-Host " - Multi-tenant isolation with Global Query Filter" -ForegroundColor Green
Write-Host " - JWT-based authorization" -ForegroundColor Green
Write-Host " - Domain Events (ProjectCreated, Updated, Archived)" -ForegroundColor Green
Write-Host " - SignalR integration ready" -ForegroundColor Green
Write-Host ""
Write-Host "Project Management Module - Day 12 Complete!" -ForegroundColor Green
Write-Host "========================================" -ForegroundColor Cyan

View File

@@ -0,0 +1,35 @@
# Test Tenant Registration API
$body = @{
tenantName = "Test Corp"
tenantSlug = "test-corp-$(Get-Random)"
subscriptionPlan = "Professional"
adminEmail = "admin@testcorp.com"
adminPassword = "Admin@1234"
adminFullName = "Test Admin"
} | ConvertTo-Json
Write-Host "Testing Tenant Registration API..." -ForegroundColor Cyan
Write-Host "Request Body:" -ForegroundColor Yellow
Write-Host $body
try {
$response = Invoke-RestMethod -Uri "http://localhost:5167/api/tenants/register" `
-Method Post `
-ContentType "application/json" `
-Body $body
Write-Host ""
Write-Host "SUCCESS! Registration completed successfully!" -ForegroundColor Green
Write-Host ""
Write-Host "Response:" -ForegroundColor Yellow
Write-Host ($response | ConvertTo-Json -Depth 10)
Write-Host ""
Write-Host "Access Token:" -ForegroundColor Cyan
Write-Host $response.accessToken
} catch {
Write-Host ""
Write-Host "FAILED! Registration failed!" -ForegroundColor Red
Write-Host "Error: $($_.Exception.Message)" -ForegroundColor Red
Write-Host "Details: $($_.ErrorDetails.Message)" -ForegroundColor Red
}

View File

@@ -27,7 +27,7 @@ public class AssignStoryCommandHandlerTests
// Arrange
var userId = UserId.Create();
var assigneeId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Description", TaskPriority.Medium, userId);
@@ -83,7 +83,7 @@ public class AssignStoryCommandHandlerTests
var userId = UserId.Create();
var firstAssignee = UserId.Create();
var secondAssignee = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Description", TaskPriority.Medium, userId);

View File

@@ -26,7 +26,7 @@ public class CreateStoryCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var epicId = epic.Id;
@@ -94,7 +94,7 @@ public class CreateStoryCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
_projectRepositoryMock

View File

@@ -26,7 +26,7 @@ public class CreateTaskCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var storyId = story.Id;
@@ -95,7 +95,7 @@ public class CreateTaskCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);

View File

@@ -26,7 +26,7 @@ public class DeleteStoryCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Story to Delete", "Description", TaskPriority.Medium, userId);
var storyId = story.Id;
@@ -70,7 +70,7 @@ public class DeleteStoryCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Story with Tasks", "Description", TaskPriority.Medium, userId);

View File

@@ -26,7 +26,7 @@ public class DeleteTaskCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Task to Delete", "Description", TaskPriority.Medium, userId);

View File

@@ -26,7 +26,7 @@ public class UpdateStoryCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Original Title", "Original Description", TaskPriority.Low, userId);
var storyId = story.Id;
@@ -89,7 +89,7 @@ public class UpdateStoryCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Original", "Original", TaskPriority.Low, userId);

View File

@@ -29,7 +29,7 @@ public class UpdateTaskStatusCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Test Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Test Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.Medium, userId);
@@ -63,7 +63,7 @@ public class UpdateTaskStatusCommandHandlerTests
{
// Arrange - This tests the bug fix for accepting "InProgress" (without space)
var userId = UserId.Create();
var project = Project.Create("Test Project", "Test Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Test Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.Medium, userId);
@@ -94,7 +94,7 @@ public class UpdateTaskStatusCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Test Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Test Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.Medium, userId);
@@ -124,7 +124,7 @@ public class UpdateTaskStatusCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Test Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Test Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.Medium, userId);
@@ -154,7 +154,7 @@ public class UpdateTaskStatusCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Test Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Test Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.Medium, userId);
@@ -207,7 +207,7 @@ public class UpdateTaskStatusCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Test Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Test Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.Medium, userId);
@@ -239,7 +239,7 @@ public class UpdateTaskStatusCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Test Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Test Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.Medium, userId);
@@ -272,7 +272,7 @@ public class UpdateTaskStatusCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Test Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Test Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.Medium, userId);
@@ -305,7 +305,7 @@ public class UpdateTaskStatusCommandHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Test Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Test Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.Medium, userId);

View File

@@ -24,7 +24,7 @@ public class GetStoryByIdQueryHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.High, userId);
var task1 = story.CreateTask("Task 1", "Description 1", TaskPriority.Medium, userId);

View File

@@ -24,7 +24,7 @@ public class GetTaskByIdQueryHandlerTests
{
// Arrange
var userId = UserId.Create();
var project = Project.Create("Test Project", "Description", "TST", userId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TST", userId);
var epic = project.CreateEpic("Test Epic", "Epic Description", userId);
var story = epic.CreateStory("Test Story", "Story Description", TaskPriority.Medium, userId);
var task = story.CreateTask("Test Task", "Task Description", TaskPriority.High, userId);

View File

@@ -23,7 +23,7 @@ public class ProjectTests
var ownerId = UserId.Create();
// Act
var project = Project.Create(name, description, key, ownerId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), name, description, key, ownerId);
// Assert
project.Should().NotBeNull();
@@ -47,7 +47,7 @@ public class ProjectTests
var ownerId = UserId.Create();
// Act
var project = Project.Create(name, description, key, ownerId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), name, description, key, ownerId);
// Assert
project.DomainEvents.Should().ContainSingle();
@@ -70,7 +70,7 @@ public class ProjectTests
var ownerId = UserId.Create();
// Act
var project = Project.Create(name, description!, key, ownerId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), name, description!, key, ownerId);
// Assert
project.Should().NotBeNull();
@@ -88,7 +88,7 @@ public class ProjectTests
var ownerId = UserId.Create();
// Act
Action act = () => Project.Create(invalidName, "Description", key, ownerId);
Action act = () => Project.Create(TenantId.Create(Guid.NewGuid()), invalidName, "Description", key, ownerId);
// Assert
act.Should().Throw<DomainException>()
@@ -104,7 +104,7 @@ public class ProjectTests
var ownerId = UserId.Create();
// Act
Action act = () => Project.Create(name, "Description", key, ownerId);
Action act = () => Project.Create(TenantId.Create(Guid.NewGuid()), name, "Description", key, ownerId);
// Assert
act.Should().Throw<DomainException>()
@@ -120,7 +120,7 @@ public class ProjectTests
var ownerId = UserId.Create();
// Act
var project = Project.Create(name, "Description", key, ownerId);
var project = Project.Create(TenantId.Create(Guid.NewGuid()), name, "Description", key, ownerId);
// Assert
project.Should().NotBeNull();
@@ -135,7 +135,7 @@ public class ProjectTests
public void UpdateDetails_WithValidData_ShouldUpdateProject()
{
// Arrange
var project = Project.Create("Original Name", "Original Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Original Name", "Original Description", "TEST", UserId.Create());
var originalCreatedAt = project.CreatedAt;
var newName = "Updated Name";
var newDescription = "Updated Description";
@@ -155,7 +155,7 @@ public class ProjectTests
public void UpdateDetails_WhenCalled_ShouldRaiseProjectUpdatedEvent()
{
// Arrange
var project = Project.Create("Original Name", "Original Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Original Name", "Original Description", "TEST", UserId.Create());
project.ClearDomainEvents(); // Clear creation event
var newName = "Updated Name";
var newDescription = "Updated Description";
@@ -178,7 +178,7 @@ public class ProjectTests
public void UpdateDetails_WithNullDescription_ShouldSetEmptyDescription()
{
// Arrange
var project = Project.Create("Original Name", "Original Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Original Name", "Original Description", "TEST", UserId.Create());
// Act
project.UpdateDetails("Updated Name", null!);
@@ -194,7 +194,7 @@ public class ProjectTests
public void UpdateDetails_WithEmptyName_ShouldThrowDomainException(string invalidName)
{
// Arrange
var project = Project.Create("Original Name", "Original Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Original Name", "Original Description", "TEST", UserId.Create());
// Act
Action act = () => project.UpdateDetails(invalidName, "Updated Description");
@@ -208,7 +208,7 @@ public class ProjectTests
public void UpdateDetails_WithNameExceeding200Characters_ShouldThrowDomainException()
{
// Arrange
var project = Project.Create("Original Name", "Original Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Original Name", "Original Description", "TEST", UserId.Create());
var name = new string('A', 201);
// Act
@@ -227,7 +227,7 @@ public class ProjectTests
public void CreateEpic_WithValidData_ShouldCreateEpic()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
project.ClearDomainEvents();
var epicName = "Epic 1";
var epicDescription = "Epic Description";
@@ -250,7 +250,7 @@ public class ProjectTests
public void CreateEpic_WhenCalled_ShouldRaiseEpicCreatedEvent()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
project.ClearDomainEvents();
var epicName = "Epic 1";
var createdBy = UserId.Create();
@@ -273,7 +273,7 @@ public class ProjectTests
public void CreateEpic_InArchivedProject_ShouldThrowDomainException()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
project.Archive();
var createdBy = UserId.Create();
@@ -289,7 +289,7 @@ public class ProjectTests
public void CreateEpic_MultipleEpics_ShouldAddToCollection()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
var createdBy = UserId.Create();
// Act
@@ -310,7 +310,7 @@ public class ProjectTests
public void Archive_ActiveProject_ShouldArchiveProject()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
// Act
project.Archive();
@@ -325,7 +325,7 @@ public class ProjectTests
public void Archive_WhenCalled_ShouldRaiseProjectArchivedEvent()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
project.ClearDomainEvents();
// Act
@@ -344,7 +344,7 @@ public class ProjectTests
public void Archive_AlreadyArchivedProject_ShouldThrowDomainException()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
project.Archive();
// Act
@@ -363,7 +363,7 @@ public class ProjectTests
public void Activate_ArchivedProject_ShouldActivateProject()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
project.Archive();
// Act
@@ -379,7 +379,7 @@ public class ProjectTests
public void Activate_AlreadyActiveProject_ShouldThrowDomainException()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
// Act
Action act = () => project.Activate();
@@ -393,7 +393,7 @@ public class ProjectTests
public void Activate_ArchivedProjectWithEpics_ShouldActivateSuccessfully()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
project.CreateEpic("Epic 1", "Description", UserId.Create());
project.Archive();
@@ -413,7 +413,7 @@ public class ProjectTests
public void Epics_Collection_ShouldBeReadOnly()
{
// Arrange
var project = Project.Create("Test Project", "Description", "TEST", UserId.Create());
var project = Project.Create(TenantId.Create(Guid.NewGuid()), "Test Project", "Description", "TEST", UserId.Create());
// Act & Assert
project.Epics.Should().BeAssignableTo<IReadOnlyCollection<Epic>>();
@@ -423,8 +423,8 @@ public class ProjectTests
public void Project_ShouldHaveUniqueId()
{
// Arrange & Act
var project1 = Project.Create("Project 1", "Description", "PRJ1", UserId.Create());
var project2 = Project.Create("Project 2", "Description", "PRJ2", UserId.Create());
var project1 = Project.Create(TenantId.Create(Guid.NewGuid()), "Project 1", "Description", "PRJ1", UserId.Create());
var project2 = Project.Create(TenantId.Create(Guid.NewGuid()), "Project 2", "Description", "PRJ2", UserId.Create());
// Assert
project1.Id.Should().NotBe(project2.Id);

View File

@@ -20,7 +20,7 @@ public class DomainEventsTests
var createdBy = UserId.Create();
// Act
var @event = new ProjectCreatedEvent(projectId, projectName, createdBy);
var @event = new ProjectCreatedEvent(projectId, TenantId.Create(Guid.NewGuid()), projectName, createdBy);
// Assert
@event.ProjectId.Should().Be(projectId);
@@ -38,8 +38,9 @@ public class DomainEventsTests
var createdBy = UserId.Create();
// Act
var event1 = new ProjectCreatedEvent(projectId, projectName, createdBy);
var event2 = new ProjectCreatedEvent(projectId, projectName, createdBy);
var tenantId = TenantId.Create(Guid.NewGuid());
var event1 = new ProjectCreatedEvent(projectId, tenantId, projectName, createdBy);
var event2 = new ProjectCreatedEvent(projectId, tenantId, projectName, createdBy);
// Assert - Records with same values should be equal
event1.ProjectId.Should().Be(event2.ProjectId);
@@ -163,7 +164,7 @@ public class DomainEventsTests
public void DomainEvents_OccurredOn_ShouldBeUtcTime()
{
// Arrange & Act
var projectCreatedEvent = new ProjectCreatedEvent(ProjectId.Create(), "Test", UserId.Create());
var projectCreatedEvent = new ProjectCreatedEvent(ProjectId.Create(), TenantId.Create(Guid.NewGuid()), "Test", UserId.Create());
var projectUpdatedEvent = new ProjectUpdatedEvent(ProjectId.Create(), "Test", "Desc");
var projectArchivedEvent = new ProjectArchivedEvent(ProjectId.Create());
var epicCreatedEvent = new EpicCreatedEvent(EpicId.Create(), "Epic", ProjectId.Create());
@@ -182,7 +183,7 @@ public class DomainEventsTests
var beforeCreation = DateTime.UtcNow;
// Act
var @event = new ProjectCreatedEvent(ProjectId.Create(), "Test", UserId.Create());
var @event = new ProjectCreatedEvent(ProjectId.Create(), TenantId.Create(Guid.NewGuid()), "Test", UserId.Create());
// Assert
var afterCreation = DateTime.UtcNow;
@@ -203,7 +204,7 @@ public class DomainEventsTests
var createdBy = UserId.Create();
// Act
var @event = new ProjectCreatedEvent(projectId, projectName, createdBy);
var @event = new ProjectCreatedEvent(projectId, TenantId.Create(Guid.NewGuid()), projectName, createdBy);
var originalProjectId = @event.ProjectId;
var originalProjectName = @event.ProjectName;
var originalCreatedBy = @event.CreatedBy;

View File

@@ -5,6 +5,8 @@ using ColaFlow.Modules.Identity.Infrastructure.Services;
using FluentAssertions;
using MediatR;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Hosting;
using Microsoft.Extensions.Logging;
using Moq;
namespace ColaFlow.Modules.Identity.Infrastructure.Tests.Persistence;
@@ -13,6 +15,8 @@ public class GlobalQueryFilterTests : IDisposable
{
private readonly Mock<ITenantContext> _mockTenantContext;
private readonly Mock<IMediator> _mockMediator;
private readonly Mock<IHostEnvironment> _mockEnvironment;
private readonly Mock<ILogger<IdentityDbContext>> _mockLogger;
private readonly IdentityDbContext _context;
public GlobalQueryFilterTests()
@@ -23,7 +27,10 @@ public class GlobalQueryFilterTests : IDisposable
_mockTenantContext = new Mock<ITenantContext>();
_mockMediator = new Mock<IMediator>();
_context = new IdentityDbContext(options, _mockTenantContext.Object, _mockMediator.Object);
_mockEnvironment = new Mock<IHostEnvironment>();
_mockEnvironment.Setup(e => e.EnvironmentName).Returns("Development");
_mockLogger = new Mock<ILogger<IdentityDbContext>>();
_context = new IdentityDbContext(options, _mockTenantContext.Object, _mockMediator.Object, _mockEnvironment.Object, _mockLogger.Object);
}
[Fact]
@@ -43,7 +50,10 @@ public class GlobalQueryFilterTests : IDisposable
.Options;
var mockMediator = new Mock<IMediator>();
using var context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object);
var mockEnvironment = new Mock<IHostEnvironment>();
mockEnvironment.Setup(e => e.EnvironmentName).Returns("Development");
var mockLogger = new Mock<ILogger<IdentityDbContext>>();
using var context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object, mockEnvironment.Object, mockLogger.Object);
var user1 = User.CreateLocal(
tenant1Id,
@@ -86,7 +96,10 @@ public class GlobalQueryFilterTests : IDisposable
.Options;
var mockMediator = new Mock<IMediator>();
using var context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object);
var mockEnvironment = new Mock<IHostEnvironment>();
mockEnvironment.Setup(e => e.EnvironmentName).Returns("Development");
var mockLogger = new Mock<ILogger<IdentityDbContext>>();
using var context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object, mockEnvironment.Object, mockLogger.Object);
var user1 = User.CreateLocal(tenant1Id, Email.Create("admin@tenant1.com"), "pass", FullName.Create("Admin One"));
var user2 = User.CreateLocal(tenant2Id, Email.Create("admin@tenant2.com"), "pass", FullName.Create("Admin Two"));
@@ -117,7 +130,10 @@ public class GlobalQueryFilterTests : IDisposable
.Options;
var mockMediator = new Mock<IMediator>();
using var context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object);
var mockEnvironment = new Mock<IHostEnvironment>();
mockEnvironment.Setup(e => e.EnvironmentName).Returns("Development");
var mockLogger = new Mock<ILogger<IdentityDbContext>>();
using var context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object, mockEnvironment.Object, mockLogger.Object);
var user1 = User.CreateLocal(tenant1Id, Email.Create("user1@test.com"), "pass", FullName.Create("User One"));
var user2 = User.CreateLocal(tenant2Id, Email.Create("user2@test.com"), "pass", FullName.Create("User Two"));
@@ -149,7 +165,10 @@ public class GlobalQueryFilterTests : IDisposable
.Options;
var mockMediator = new Mock<IMediator>();
using var context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object);
var mockEnvironment = new Mock<IHostEnvironment>();
mockEnvironment.Setup(e => e.EnvironmentName).Returns("Development");
var mockLogger = new Mock<ILogger<IdentityDbContext>>();
using var context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object, mockEnvironment.Object, mockLogger.Object);
var user1 = User.CreateLocal(tenant1Id, Email.Create("john@tenant1.com"), "pass", FullName.Create("John Doe"));
var user2 = User.CreateLocal(tenant2Id, Email.Create("jane@tenant2.com"), "pass", FullName.Create("Jane Doe"));

View File

@@ -5,6 +5,8 @@ using ColaFlow.Modules.Identity.Infrastructure.Services;
using FluentAssertions;
using MediatR;
using Microsoft.EntityFrameworkCore;
using Microsoft.Extensions.Hosting;
using Microsoft.Extensions.Logging;
using Moq;
namespace ColaFlow.Modules.Identity.Infrastructure.Tests.Repositories;
@@ -25,7 +27,12 @@ public class TenantRepositoryTests : IDisposable
var mockMediator = new Mock<IMediator>();
_context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object);
var mockEnvironment = new Mock<IHostEnvironment>();
mockEnvironment.Setup(e => e.EnvironmentName).Returns("Development");
var mockLogger = new Mock<ILogger<IdentityDbContext>>();
_context = new IdentityDbContext(options, mockTenantContext.Object, mockMediator.Object, mockEnvironment.Object, mockLogger.Object);
_repository = new TenantRepository(_context);
}

File diff suppressed because it is too large Load Diff