优化

warehouse/src/test/java/com/baiyue/warehouse/repository/SupplierSqlContractTest.java

@@ -13,6 +13,7 @@ class SupplierSqlContractTest {
             assertThat(stream).isNotNull();
             String sql = new String(stream.readAllBytes(), StandardCharsets.UTF_8);
             assertThat(sql).contains("CREATE TABLE IF NOT EXISTS wh_supplier", "tenant_id BIGINT UNSIGNED NOT NULL DEFAULT 0",
+                "chk_wh_supplier_tenant CHECK (tenant_id = 0)",
                 "category='BOOKING' AND scope IN ('DOMESTIC','INTERNATIONAL')", "license_file_id VARCHAR(128)",
                 "MONTHLY_30','MONTHLY_45','MONTHLY_60','MONTHLY_90','PER_TRANSACTION','IMMEDIATE",
                 "created_by BIGINT UNSIGNED", "updated_by BIGINT UNSIGNED",

warehouse/src/test/java/com/baiyue/warehouse/service/SupplierServiceTest.java

@@ -2,6 +2,8 @@ package com.baiyue.warehouse.service;
 
 import com.baiyue.common.core.exception.BizException;
 import com.baiyue.warehouse.dto.SupplierDtos.SaveRequest;
+import com.baiyue.warehouse.dto.SupplierDtos.PageQuery;
+import com.baiyue.warehouse.dto.SupplierDtos.StatusRequest;
 import com.baiyue.warehouse.entity.SupplierEntity;
 import com.baiyue.warehouse.repository.SupplierRepository;
 import org.junit.jupiter.api.BeforeEach;
@@ -13,6 +15,7 @@ import org.mockito.junit.jupiter.MockitoExtension;
 
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
@@ -30,20 +33,40 @@ class SupplierServiceTest {
 
     @Test void createsPlatformPublicBookingSupplier() {
         when(repository.existsName(0L, "测试供应商", null)).thenReturn(false);
-        service.create(booking);
+        service.create(101L, booking);
         verify(repository).insert(any(SupplierEntity.class));
     }
 
     @Test void rejectsScopeForNonBookingSupplier() {
         SaveRequest customs = new SaveRequest("报关供应商", "CUSTOMS", "DOMESTIC", "宁波", "法人", "联系人",
             "13800000000", null, null, null, null, "ACTIVE", "PER_TRANSACTION", null);
-        assertThatThrownBy(() -> service.create(customs)).isInstanceOf(BizException.class)
+        assertThatThrownBy(() -> service.create(101L, customs)).isInstanceOf(BizException.class)
             .hasMessageContaining("仅适用于订舱供应商");
     }
 
     @Test void requiresScopeForBookingSupplier() {
         SaveRequest missingScope = new SaveRequest("订舱供应商", "BOOKING", null, "宁波", "法人", "联系人",
             "13800000000", null, null, null, null, "ACTIVE", "PER_TRANSACTION", null);
-        assertThatThrownBy(() -> service.create(missingScope)).isInstanceOf(BizException.class);
+        assertThatThrownBy(() -> service.create(101L, missingScope)).isInstanceOf(BizException.class);
+    }
+
+    @Test void rejectsScopeFilterWithoutBookingCategory() {
+        assertThatThrownBy(() -> service.page(new PageQuery(1, 20, "CUSTOMS", "DOMESTIC", null, null, null)))
+            .isInstanceOf(BizException.class).hasMessageContaining("scope仅适用于订舱供应商");
+    }
+
+    @Test void repeatedStatusChangeIsIdempotent() {
+        SupplierEntity entity = new SupplierEntity(); entity.setId(101L); entity.setTenantId(0L); entity.setStatus("ACTIVE");
+        when(repository.find(0L, 101L)).thenReturn(entity);
+        service.changeStatus(101L, new StatusRequest("ACTIVE"));
+        verify(repository, never()).save(any());
+    }
+
+    @Test void validatesInternalSupplierSnapshot() {
+        SupplierEntity entity = new SupplierEntity(); entity.setId(101L); entity.setTenantId(0L); entity.setName("测试供应商");
+        entity.setCategory("BOOKING"); entity.setScope("DOMESTIC"); entity.setStatus("ACTIVE");
+        when(repository.find(0L, 101L)).thenReturn(entity);
+        var snapshot = service.internalSnapshot(101L, true, "BOOKING", "DOMESTIC");
+        org.assertj.core.api.Assertions.assertThat(snapshot.supplierId()).isEqualTo(101L);
     }
 }