Uniezależnienie testów od aktaulnego stanu bazy.

[eengine-sync]

Trzeba na pewno zmienić testy rabatów (najlepiej nie tylko) w taki sposób, aby nie były związane z produktami.

Generalnie w ogóle najlepiej dodawać własne produkty do testów i na nich bazować.

Teraz np. wszystkie się wywaliły jak wpadły ceny netto :)

Usunąłem na masterze sprawdzanie order_total_price tam, gdzie nie było to konieczne, a testy rabatów oznaczyłem jako pending, bo są oparte o sprawdzanie order_total_price.

[eengine-sync]

➤ Rafał Roźniakowski commented:

https://github.com/eEngineSoftwareHouse/perseids-api/pull/185

[eengine-sync]

➤ Anna Walaszek commented:

@fizol poproszę o CR

[eengine-sync]

➤ Pawel Walaszek commented:

@rrozniakowski Niestety o CircleCI nie wiem nic poza tym, że służy do testów, a o elixirze mało. Zróbmy tak, że takie CR będziemy robić we dwóch. Będziesz opowiadał co było do zrobienia, co i jak zrobiłeś. Wiem, że to nie to samo, ale potencjalnie da podobny efekt, tj. Ty dzięki temu, że będziesz musiał opowiedzieć co zrobiłeś spojrzysz na to z boku i będzie to okazja do wyciągnięcia wniosków, a ja będę miał szerszy obraz i też może coś podpowiem, a na pewno będę zadawał dużo pytań :)

Jeżeli temat nie jest pilny, to wróćmy do niego w przyszłym tygodniu. Ustrzel mnie w poniedziałek/wtorek, godzina wedle uznania.

[eengine-sync]

➤ Rafał Roźniakowski commented:

@fizol ok :]

[eengine-sync]

➤ Pawel Walaszek commented:

Dodałem w konfiguracji domenę http://circlecibeta.manymornings.eengine.tech/, która wpuszcza do magento bez jakichkolwiek zabezpieczeń. Dodany został też nagłowek X-Robots-Tag: noindex dla wszystkich eengine.tech, aby w razie czego, ostatecznie i tak google nie indeksowało.

[eengine-sync]

➤ Pawel Walaszek commented:

@rrozniakowski Daj znać jak pociągniesz temat testów dalej. Jestem bardzo ciekaw, czy i jak to działa w praktyce.

[eengine-sync]

➤ Rafał Roźniakowski commented:

@fizol jeżeli chcę wejść na domenę która podałeś niżej nadal wymagana jest autoryzacja użytkownika jeżeli nie jesteś z IP eEngine.

[eengine-sync]

➤ Pawel Walaszek commented:

@rrozniakowski Sprawdzę to, ale poszukaj na stronie circleCi czy oni nie mają stałej jawnie podanej grupy IP, z których łączą się ich środowiska testowe, jeżeli tak to byśmy je zdefiniowali jako do przepuszczania przez GW, a jeżeli nie to spróbuje naprawić to co zrobiłem, ale jak widać nie działa poprawnie.

[eengine-sync]

➤ Rafał Roźniakowski commented:

@fizol -> https://discuss.circleci.com/t/circleci-source-ip/1202 => https://ip-ranges.amazonaws.com/ip-ranges.json Jeżeli Cie dobrze zrozumiałem to o to chodziło ?

[eengine-sync]

➤ Pawel Walaszek commented:

@rrozniakowski Przepuściłem po prostu domenę http://magento.manymornings.eengine.tech/ bez hasła i IPka, bo przecież i tak dodane są nagłówki X-Robots-Tag: noindex dla wszystkiego co idzie przez *.eengine.tech, więc nie ma ryzyka, że google wejdzie i zaindeksuje.