fix: correct Without validator failing when instance is absent (#241)

Author: yuchem2

packages/express-cargo/src/decorators/validators/crossField.ts

@@ -35,10 +35,8 @@ export function Without(fieldName: string, message?: cargoErrorMessage): Propert
             new ValidatorRule(
                 propertyKey,
                 'without',
-                (value: unknown, instance?: Record<string | symbol, any>) => {
-                    if (!instance) return false
-                    return !(!!value && !!instance?.[fieldName])
-                },
+                (value: unknown, instance?: Record<string | symbol, any>) =>
+                    !(!!value && !!instance?.[fieldName]),
                 message || `${String(propertyKey)} cannot exist with ${fieldName}`,
             ),
             { name: Without.name, category: 'validator', args: [fieldName] },

packages/express-cargo/tests/validator/without.test.ts

@@ -74,12 +74,12 @@ describe('Without decorator', () => {
         expect(result).toBeNull()
     })
 
-    it('Case 5: Should pass when instance is undefined', () => {
+    it('Case 5: Should pass when instance is undefined (related field is absent)', () => {
         const meta = classMeta.getFieldMetadata(TARGET_FIELD)
         const WithoutRule = meta.getValidators()?.find(v => v.type === 'without')
         const instance = undefined
         const result = WithoutRule?.validate('ValueA', instance)
 
-        expect(result).toBeInstanceOf(CargoFieldError)
+        expect(result).toBeNull()
     })
 })