Код
public class Company
{
public string Name { get; set; }
}
public class Customer
{
public string Surname { get; set; }
public string Name { get; set; }
public Company Company { get; set; }
}
public class CustomerDto
{
public int Id { get; set; }
[Required]
public string Surname { get; set; }
[Required]
public string Name { get; set; }
public string SecondName { get; set; }
}
public class CustomerService
{
private readonly IGenericRepository<Customer> _customerRepository;
public CustomerService(IGenericRepository<Customer> customerRepository)
{
_customerRepository = customerRepository;
}
public CustomerDto Add(CustomerDto dto)
{
//Validation
if (dto == null)
{
throw new ArgumentNullException(nameof(dto));
}
var validContext = new System.ComponentModel.DataAnnotations.ValidationContext(dto);
Validator.ValidateObject(dto, validContext);
//Mapping by AutoMapper
Mapper.Initialize(cfg => cfg.CreateMap<CustomerDto, Customer>());
var entity = Mapper.Map<CustomerDto, Customer>(dto);
_customerRepository.Insert(entity);
_customerRepository.Save();
dto.Id = entity.Id;
return dto;
}
}
Юнит-тестирование CustomerService.Add()
Необходимо создать юнит-тест для метода Add()
. Изначально, вообще была попытка реализовать этот метод через TDD, т.е. сначала создать для него юнит-тест и только затем имлемантацию. Но я не смог этого сделать так как не разобрался за что конкретно отвечает этот метод и надо ли вообще писать для него тест.
Но два юнит-теста я сделал и вот как все это выглядит:
[TestClass]
public class Customer_Test
{
[TestMethod]
public void Add_Test()
{
//Arrange
//Use NSubstitude for mocking
var mockRepository = Substitute.For<IGenericRepository<Customer>>();
//Act
var service = new CustomerService(mockRepository);
var dto = new CustomerDto
{
Surname = "Shakhabov",
Name = "Adam"
};
var addedDto = service.Add(dto);
//Assert
Assert.IsNotNull(addedDto);
}
[TestMethod]
[ExpectedException(typeof(ValidationException))]
public void Add_TestException()
{
//Arrange
var mockRepository = Substitute.For<IGenericRepository<Customer>>();
//Act
var service = new CustomerService(mockRepository);
var dto = new CustomerDto
{
Surname = "Shakhabov",
};
var addedDto = service.Add(dto);
//Assert
//expectedException ValidationException
}
}
Вот что (цель) должен сделать метод: Добавить в базу данных объект
на основе полученного DTO-объекта
и затем обратно вернуть этот DTO-объект
, но уже с обновленным полем Id
, значение для которого сформировала СУБД.
Вот как (задачи) делает это метод:
- Получив DTO-объект, сразу верифицирует его;
- Используя библиотеку AutoMapper, проецирует DTO на сущность, которая в свою очередь создается с нуля;
- Используя объект-репозиторий, добавляет сущность в контекст.
- Сохраняет контекст в результате чего источник данных (СУБД) инициализирует поле Id у сущности (entity);
- Инициализирует поле dto.Id полем entity.Id и возвращает DTO.
Вопрос
Правильно ли были сформулированы требования для метода Add()
и соответственно построены юнит-тесты на их основе? Какие тесты (выражения Assert
) создаются для методов с подобной логикой, где целью является инициализировать свойство DTO-объекта
значением, получаемое сторонним сервисом (в данном случаи объект-репозиторий)?
dto.Id = dto.Id
. Да, это была опечатка. Обновил.Id
, ведь он происходит как бы из ни "откуда". Его (Id) не сервис и не репозиторий делает, а контекст. Без кода трудно понять, что ты предлагаешь говоря: конвертацию из DTO в сущность БД перенести внутрь DTOCustomerDto.ToCustomer()
увидеть на деле хотелось бы.