PersonType ids update

[leszko11]

No description provided.

[gagyn]

Mógłbyś jeszcze przejrzeć testy, które sprawdzają pobieranie odpowiednich osób? Z tego co pamiętam, były testy które sprawdzały czy pobiera odpowiednie osoby, więc może trzeba tam coś teraz zmienić

[leszko11]

Postaram się to sprawdzić

[gagyn]

https://github.com/nickofc/FilmWeb-API/tree/master/tests/FilmWebAPI.Tests/NewFeaturesTests.cs

[gagyn]

Naprawiłem testy które nie przechodziły oraz usunąłem dwie nie działające funkcje, które musiały przestać działać z powodu jakichś zmian w API.
@nickofc Nie działają teraz dwie funkcje: GetAllCinemas oraz GetFilmVideoUrl. Na razie zostawiłem te klasy, ale usunąłem wzmianki o nich w klasie FilmWeb. Nie wiem czy jest sens próbować naprawiać.

[gagyn]

@leszko11 Jeszcze może warto żebyś zrobił merge z aktualnymi zmianami do swojego brancha, to będziesz mieć poprawione testy i CI będzie na zielono

[leszko11]

Jasne, zrobię to :)

[nickofc]

@nickofc Nie działają teraz dwie funkcje: GetAllCinemas oraz GetFilmVideoUrl. Na razie zostawiłem te klasy, ale usunąłem wzmianki o nich w klasie FilmWeb. Nie wiem czy jest sens próbować naprawiać.

@gagyn
Na jakiej zasadzie nie działają?
Wiesz może czy są jakieś metody które można jeszcze zaimplementować?

[gagyn]

@nickofc Nie działają teraz dwie funkcje: GetAllCinemas oraz GetFilmVideoUrl. Na razie zostawiłem te klasy, ale usunąłem wzmianki o nich w klasie FilmWeb. Nie wiem czy jest sens próbować naprawiać.

@gagyn
Na jakiej zasadzie nie działają?
Wiesz może czy są jakieś metody które można jeszcze zaimplementować?

W jednym przypadku, zwracany przez API ciąg zawiera tylko "ok", a w drugim oprócz "ok" jeszcze jedną liczbę.
W jakim sensie czy są jeszcze metody które można zaimplementować? Na razie zrobionych metod jest z 20%

[leszko11]

Patrząc na wyniki testów zacząłem się zastanawiać czy czegoś nie pomieszałem. Typy sprawdzałem wykorzystując LiveSearch.
Pierwszy test failuje przy szukaniu autora zdjęć filmu o id 810167. Test oczekuje Lawrence Sher, ale dostaje Hildur Guðnadóttir, która jest autorką muzyki.
Poprzednia implementacja zakłada

        Muzyka = 3,
        Zdjecia = 4,

Testując jednak wyniki z LiveSearch, dla Hildur zwrotka jest taka
p\c1673378\c/33/78/1673378/451301.2.jpg\cHildur Guðnadóttir\c1\c4\c
Czyli typ z id 4

a dla Shera
p\c68413\c/84/13/68413/337049.2.jpg\cLawrence Sher\c2\c3\c
typ z id 3

[gagyn]

Powiem tak, trzeba mieć na względzie również to, że API jest nieoficjalne, więc tak do końca to nie wiemy co to robi, chyba nie ma żadnej dokumentacji, chyba, że się mylę.
Ja ostatnio kiedy nad tym siedziałem, miałem już trochę dosyć dociekania które id do jakiej osoby. Jeśli ty masz siłę aby znaleźć odpowiedni układ, to bardzo fajnie

Czyli rozumiem, że wyszło Ci, że zależnie od narzędzia są różne wyniki zwracane przez API?

[gagyn]

Swoją drogą, początkowo IDs wziąłem z tego repo https://github.com/b44x/filmweb-api/blob/master/src/nSolutions/Filmweb.php

[leszko11]

@gagyn
Sprawdzałem to bo jestem trakcie przepisywania tego na Swift(iOS). Zacząłem od obsługi wyszukiwarki. Przy mapowaniu odpowiedzi serwera chciałem uzupełnić brakujące typy profesji.

Teraz jeszcze zwróciłem uwagę, że tu
https://bitbucket.org/varabi/filmweb-api/src/master/src/main/java/info/talacha/filmweb/search/models/PersonSearchResult.java

https://bitbucket.org/varabi/filmweb-api/src/master/src/main/java/info/talacha/filmweb/models/Profession.java

Profesje są rozdzielone w zależności od metody szukania. Co za bzdura 😅

[leszko11]

W wolnej chwili posprawdzam to jeszcze i poprawię

[leszko11]

Dobra, finalnie niewiele dodałem. Testując kilka filmów zawsze dla id powyżej 13 już wali NullPointerException

[gagyn]

Spoko, zawsze coś do przodu