[Nofenak]

Czy taki test sprawdzający czy bilet nie jest zarezerwowany przez więcej niż jednego usera jest ok? Miejsce blokuje pesymistycznie na poziomie bazy.

@test
void ticket_is_booked_only_by_one_user() {
        //given
        var film = addFilm();
        var hall = addHall();
        var screening = addScreening(hall, film);
        var seat = addSeat(screening, hall.getSeats().getFirst());
        var users = addUsers();
        var bookTicketDto = new BookTicketDto(screening.getId(), List.of(seat.getId()));

        //when
        try (var executorService = Executors.newFixedThreadPool(3)) {
            executorService.submit(() -> bookTicket(users.getFirst().getMail(), bookTicketDto));
            executorService.submit(() -> bookTicket(users.get(1).getMail(), bookTicketDto));
            executorService.submit(() -> bookTicket(users.get(2).getMail(), bookTicketDto));
            executorService.shutdown();
        }

        //then
        Assertions.assertThat(ticketRepository.getAll()).hasSize(1);
    }

#java #naukaprogramowania #programowanie #programista15k

[lubaxd]

Test zakłada, że metoda bookTicket jest idempotentna, tzn. wielokrotne wywołanie tej metody z tymi samymi argumentami nie zmieni stanu systemu po pierwszym wywołaniu. Jeśli tak nie jest, test może nie działać poprawnie.

[5da4266d3de6dbaf425a2d4fc16225d0]

@Nofenak: Moim zdaniem to bardzo słaby test. Np. nie wiadomo z niego "która" rezerwacja przeszła. Moze np. kolejne nadpisują pierwszą? Już więcej sensu miałoby:

1. Zrób rezerwacje
2. Spróbuj zrobić drugą i sprawdź że tym razem API odpowiedziało błędem (np. REST rzucił jakieś 409)

Nie do końca rozumiem też po co ten executor tam.

[nad__czlowiek]

Nie do końca rozumiem też po co ten executor tam.

@5da4266d3de6dbaf425a2d4fc16225d0: no bo on próbuje sprawdzić czy jak kilka osób kupuje w tym czasie to tylko jedna osoba może kupić

[5da4266d3de6dbaf425a2d4fc16225d0]

@nad__czlowiek: No dobra ale jakbym chciał sprawdzić race-condition to bym to puścił ze 100 wątków co najmniej a nie z 3, i na większej liczbie rezerwacji i może jeszcze w jakiejś pęlti. Teraz to jest jakiś żart a nie testowanie tego race.

[nad__czlowiek]

@5da4266d3de6dbaf425a2d4fc16225d0: no ale ja się z tym nie kłócę ¯\(ツ)/¯

[Nofenak]

@nad__czlowiek: Dokładnie
@5da4266d3de6dbaf425a2d4fc16225d0 Jak nie zablokuje miejsca, to zawsze się wywala na tych 3 wątkach

[5da4266d3de6dbaf425a2d4fc16225d0]

Jak nie zablokuje miejsca, to zawsze się wywala na tych 3 wątkach

@Nofenak: No ale pewnie wywala się tez bez żadnych wątków, jak po prostu wywołasz to bookTicket jedno po drugim.

[lubaxd]

@Nofenak:

@test
void ticketisbookedonlybyoneuser() {
//given
var film = addFilm();
var hall = addHall();
var screening = addScreening(hall, film);
var seat = addSeat(screening, hall.getSeats().getFirst());
var users = addUsers(100);
var bookTicketDto = new BookTicketDto(screening.getId(), List.of(seat.getId()));
var executorService = Executors.newFixedThreadPool(100);

//when
for (var user : users) {
executorService.submit(() -> bookTicket(user.getMail(), bookTicketDto));
}
executorService.shutdown();
try {
executorService.awaitTermination(1, TimeUnit.MINUTES);
} catch (InterruptedException e) {
e.printStackTrace();
}

//then
Assertions.assertThat(ticketRepository.getAll()).hasSize(1);
}

[NieBendePrasowac]

jak po prostu wywołasz to bookTicket jedno po drugim.

@Nofenak: jesli tak jest to wiesz już od czego zacząć poprawki. Race condition też, ale w drugiej kolejności.

Ten test uruchamia 100 wątków, z których każdy próbuje zarezerwować ten sam bilet.

@lubaxd: dużo lepiej, ale dalej assert nie sprawdza który user zrobił rezerwację.

[Saly]

dużo lepiej, ale dalej assert nie sprawdza który user zrobił rezerwację.

@NieBendePrasowac: jak masz to przetestować jak to nie jest określone. Któremuś się uda, ale nie wiadomo któremu