ARUP-CAS / aiscr-webamcr

Archeologická mapa České republiky
https://amcr-info.aiscr.cz/
GNU General Public License v3.0
5 stars 0 forks source link

Převod automatických testů na selenium #1170

Open motyc opened 1 year ago

motyc commented 1 year ago
pesikj commented 11 months ago

@motyc Jak už jsme spolu probírali, tyto automatické testy již nejsou použitelné, protože z GitHubu nemáme připojení do repozitáře Fedory. Testy jsem tedy odstranil.

motyc commented 11 months ago

@pesikj Rozumím. Jen jsem koukal, že jste odstranil pouze danou workflow. Bylo by myslím dobré dát pryč i související kód, protože stejně zastaral a jen nám bude zatěžovat repozitář (např. i při hledání výskytů různých funkcí atd.). Pokud tedy není nějaký dobrý důvod, proč kód testů zachovat.

pesikj commented 11 months ago

@motyc Podle mě by bylo dobré to ještě projít a podívat se, které situace máme naprogramované, aby to případně mohlo být přepsáno do Seleinum testů.

motyc commented 11 months ago

@pesikj Souhlas, lze to odložit a řešit to časem.

motyc commented 11 months ago

Základ ok, nechávám otevřené kvůli tomu vytěžení existujících testů.

motyc commented 5 months ago

@jhavrlant Aktualizoval jsem první komentář tak, aby shrnoval úkoly, které je třeba dodělat. Zároveň zde odpovídám na dva reportované problémy:

testovací scénář 2.3 Zapsání projektu (negativní scénář 1) – projde uložení https://aiscr-webamcr.readthedocs.io/cs/latest/automaticke_testy.html#zapsani-projektu-negativni-scenar-1

Podle mě ten scénář nikdy nemohl fungovat, protože žádné pravidlo pro zápis plánovaného zahájení v tomto smyslu není. Nenapadá @pesikj, z čeho jste vycházel?

negativnu testovací scénář Scenar_39 nefunguje – uloží se https://docs.google.com/document/d/1qJArruTG-iUNKQkuxtgB2Bz4zzfZYVCKtMhJBKyYP8E/edit#heading=h.xdbhlgqp4ewa

Chyba. Založeno issue #1754

motyc commented 4 months ago
jhavrlant commented 4 months ago
  • dokument.tests.test_selenium.AkceDokumenty.test_066_odeslani_dokumentu_p_001

@motyc chyba bude v testu. Musím se na něj podívat.