search

python-dirbtuves / it-brandos-egzaminai

Informacinių technologijų brandos egzaminų, programavimo praktinių užduočių sprendimai
GNU Affero General Public License v3.0
13 stars 13 forks source link

2006/u2 and 2007/u2 simple/advanced solutions #18

Closed EdvardasB closed 5 years ago

EdvardasB commented 6 years ago

The 2006 tests.py throws an exception, problem with characters. I used Lithuanian symbols, didn't check if that's the cause though. Need to review the testing process later.

sirex commented 6 years ago

Testai nepraeina, testus galima paleisti taip:

./test

Kelias iki katalogo kuriame laikomas duomenų ir rezultatų failas turi būti preduodamas kaip argumentas main funkcijai. Automatinių testų metu, kiekvienam programos iškvietimui, su skirtingais įvesties duomenimis sukuriamas laikinas katalogas ir kelias iki to katalogo perduodamas main funkcijai.

Konstantų naudojimas yra stipriai perteklinis. Konstantas reikėtų naudoti tik tada, kai reikšė kartojasi daugelyje vietų ir yra tikimybė, kad ta reikšmė, kada nors pasikeis.

EdvardasB commented 6 years ago

Pataisius main funkciją, kad gautų direktorijos kelią argumentą, tenko susidurti su OS (Windows 7) bėdą

File "C:\Users\HOME\PycharmProjects\it-brandos-egzaminai\exams\2006\pagrindinis\u2\u2.py", line 12, in nuskaityti_duomenis_iš_failo with open(duomenu_failas, SKAITYMO_REŽIMAS, encoding=TEKSTO_KODUOTĖ) as failas: PermissionError: [Errno 13] Permission denied: 'C:\Users\HOME\AppData\Local\Temp\pytest-of-HOME\pytest-18\test0'

Nors security pridėjau ir Everyone -> Full control. Bet paleidžiant test su venv, toks pranešimas ties pirmu testu. Kol kas pagal pastabas sutvarkysiu kodą. O po to galėsi pasakyti ar pas tave suveikia testai.

sirex commented 6 years ago

Sveikas, buvau šiek tiek primiršęs apie šitą pull request. Įmečiau į master šaką Travis CI, gal galėtum rebasinti ant master šakos ir pataisyti klaidas, kurias randa Travis?

EdvardasB commented 6 years ago

Sveikas, ok. Pats buvau užsiėmęs kitais darbais, tai per daug ir nesukau galvos. Reiks dar peržiūrėti kodą, gal kils naujų minčių.

sirex commented 5 years ago

Nematau, kad būtų padarytas rebase ant master šakos. @EdvardasB padaryk rebase ant master, kad būtų galima žiūrėti toliau.

EdvardasB commented 5 years ago

@sirex done.

sirex commented 5 years ago

Kažkas negerai su rebasu, matau visus savo pakeitimus, bet jie sukomitinti tavo vardu, dėl to atsiranda konfliktas su master šaka, nes pakeitimai sudubliuoti.

Gal būt galėtum padaryti normalų rebase ir panaikinti besidubliuojančius komitus, kad nebūtų merge conflict?

EdvardasB commented 5 years ago

OK, dar pagalvojau, kad geriau naudoti wget o ne curl. Šiaip jei neįdiegti python3-dev, tai ir pip instaliacija failins iš get-pip.py reiks bent perspėjimą įdėti.