Fix test assertions and improve test documentation based on code review

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Author: Copilot

packages/drivers/redis/test/index.test.ts

@@ -645,10 +645,11 @@ describe('RedisDriver', () => {
                 value: 'user'
             });
 
-            expect(departments).toHaveLength(2);
+            // Eve (user, IT), Bob (user, HR), David (user, Sales)
+            expect(departments).toHaveLength(3);
             expect(departments).toContain('HR');
             expect(departments).toContain('Sales');
-            expect(departments).not.toContain('IT'); // Admins are in IT, not users
+            expect(departments).toContain('IT'); // Eve is a user in IT
         });
 
         it('should handle distinct on numeric fields', async () => {

packages/protocols/graphql/src/integration.test.ts

@@ -144,11 +144,14 @@ describe('GraphQL Protocol Integration Tests', () => {
     describe('Apollo Server Startup', () => {
         it('should start Apollo Server successfully', () => {
             expect(plugin).toBeDefined();
+            // Note: Full HTTP endpoint testing would require supertest
+            // For now, we verify the plugin initialized successfully
         });
 
-        it('should expose GraphQL endpoint', async () => {
-            // Note: This is a basic check - full HTTP testing would require supertest
+        it('should be accessible at configured port', () => {
+            // Verify base URL is configured correctly
             expect(baseUrl).toBe('http://localhost:14000');
+            // Note: Actual HTTP request verification would require supertest
         });
     });
 

packages/protocols/odata-v4/src/integration.test.ts

@@ -115,14 +115,18 @@ describe('OData V4 Protocol Integration Tests', () => {
     });
 
     describe('OData V4 Standard Endpoints', () => {
-        it('should support service document endpoint', () => {
+        it('should initialize plugin successfully', () => {
             expect(plugin).toBeDefined();
+            // Note: Actual HTTP endpoint testing would require supertest
+            // For now, we verify the plugin initialized correctly
         });
 
-        it('should support metadata endpoint', () => {
+        it('should provide metadata access', () => {
             const objects = kernel.metadata.list('object');
             expect(objects).toBeDefined();
             expect(objects.length).toBeGreaterThan(0);
+            // Verifies metadata is accessible - actual $metadata endpoint
+            // would require HTTP testing with supertest
         });
     });