Compare commits
7 Commits
172d0de1fe
...
6d2396f3c1
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
6d2396f3c1 | ||
|
|
ef409b8ba5 | ||
|
|
f21d9cd6d4 | ||
|
|
3232b70ecc | ||
|
|
9ada0cac4a | ||
|
|
3843d07577 | ||
|
|
5a1ad2eb97 |
@@ -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.
|
||||
|
||||
@@ -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": []
|
||||
|
||||
2253
colaflow-api/MCP-SERVER-ARCHITECTURE.md
Normal file
2253
colaflow-api/MCP-SERVER-ARCHITECTURE.md
Normal file
File diff suppressed because it is too large
Load Diff
314
colaflow-api/SIGNALR-IMPLEMENTATION.md
Normal file
314
colaflow-api/SIGNALR-IMPLEMENTATION.md
Normal 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
|
||||
@@ -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);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -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);
|
||||
70
colaflow-api/src/ColaFlow.API/Hubs/BaseHub.cs
Normal file
70
colaflow-api/src/ColaFlow.API/Hubs/BaseHub.cs
Normal 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);
|
||||
}
|
||||
}
|
||||
25
colaflow-api/src/ColaFlow.API/Hubs/NotificationHub.cs
Normal file
25
colaflow-api/src/ColaFlow.API/Hubs/NotificationHub.cs
Normal 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
|
||||
});
|
||||
}
|
||||
}
|
||||
73
colaflow-api/src/ColaFlow.API/Hubs/ProjectHub.cs
Normal file
73
colaflow-api/src/ColaFlow.API/Hubs/ProjectHub.cs
Normal 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}";
|
||||
}
|
||||
}
|
||||
@@ -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
|
||||
|
||||
@@ -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");
|
||||
}
|
||||
@@ -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);
|
||||
}
|
||||
}
|
||||
@@ -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
|
||||
});
|
||||
}
|
||||
}
|
||||
@@ -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);
|
||||
|
||||
@@ -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);
|
||||
|
||||
@@ -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;");
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -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
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -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");
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -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 =>
|
||||
@@ -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;");
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -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 =>
|
||||
|
||||
@@ -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>;
|
||||
@@ -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;
|
||||
}
|
||||
}
|
||||
@@ -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");
|
||||
}
|
||||
}
|
||||
@@ -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;
|
||||
|
||||
@@ -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,
|
||||
|
||||
@@ -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)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -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;
|
||||
}
|
||||
@@ -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>()
|
||||
};
|
||||
}
|
||||
}
|
||||
@@ -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");
|
||||
}
|
||||
}
|
||||
@@ -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);
|
||||
}
|
||||
}
|
||||
@@ -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);
|
||||
}
|
||||
}
|
||||
@@ -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);
|
||||
}
|
||||
}
|
||||
@@ -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);
|
||||
}
|
||||
@@ -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;
|
||||
}
|
||||
|
||||
@@ -8,6 +8,7 @@ namespace ColaFlow.Modules.ProjectManagement.Domain.Events;
|
||||
/// </summary>
|
||||
public sealed record ProjectCreatedEvent(
|
||||
ProjectId ProjectId,
|
||||
TenantId TenantId,
|
||||
string ProjectName,
|
||||
UserId CreatedBy
|
||||
) : DomainEvent;
|
||||
|
||||
@@ -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();
|
||||
}
|
||||
@@ -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
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -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");
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -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");
|
||||
});
|
||||
|
||||
|
||||
@@ -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);
|
||||
|
||||
@@ -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);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -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 =>
|
||||
{
|
||||
|
||||
268
colaflow-api/test-project-api.ps1
Normal file
268
colaflow-api/test-project-api.ps1
Normal 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
|
||||
69
colaflow-api/test-project-debug.ps1
Normal file
69
colaflow-api/test-project-debug.ps1
Normal 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
|
||||
}
|
||||
}
|
||||
219
colaflow-api/test-project-simple.ps1
Normal file
219
colaflow-api/test-project-simple.ps1
Normal 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
|
||||
35
colaflow-api/test-tenant-registration.ps1
Normal file
35
colaflow-api/test-tenant-registration.ps1
Normal 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
|
||||
}
|
||||
@@ -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);
|
||||
|
||||
|
||||
@@ -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
|
||||
|
||||
@@ -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);
|
||||
|
||||
|
||||
@@ -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);
|
||||
|
||||
|
||||
@@ -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);
|
||||
|
||||
@@ -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);
|
||||
|
||||
|
||||
@@ -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);
|
||||
|
||||
@@ -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);
|
||||
|
||||
@@ -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);
|
||||
|
||||
@@ -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);
|
||||
|
||||
@@ -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;
|
||||
|
||||
@@ -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"));
|
||||
|
||||
@@ -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);
|
||||
}
|
||||
|
||||
|
||||
2141
progress.md
2141
progress.md
File diff suppressed because it is too large
Load Diff
Reference in New Issue
Block a user