2022-12-21 Meeting #7

Jaunos koda labojumus lūdzu liec failos ar prefix 8_

Review

1. Pirmais JS/TS call ārpus async funkcijām ir bez await, await var izmantot tikai async funkcijās un tikai tad, kad ir kāda darbība, kuras izpilde nenotiek uzreiz (saņem Promise objektu kā atbildi, async funkcijas arī ģenerē promise objektu)

2. Vai nu jāizmanto papildus mainīgie piemēram LOGIN_KRISTIANA_DEV_USERNAME vai arī indeksi 0, 1 utt jāatšifrē izmantojot konstantes piemēram EMAIL[IDX_KRISTIANA_DEV_USER], kur const IDX_KRISTIANA_DEV_USER = 1;

3. Webstorm iekrāso un funkciju beigās neliek klāt ;

4. jāsaliek vairāk console.log ar jēgpilniem tekstiņiem

5. Tā vietā, lai izmantotu uz spaces balstītu XPath izmantot [contains(text(),'Change Metrics')]

6. Bug - for loop, lai kontrolētu izpildi var lietot continue, kas pārleks pāri nākamajam ierakstam vai break, kas izies no loop, bet nevar lietot return. Katrai funkcijai jābūt vienam return beigās, ja funkcija kaut ko atgriež (visai lielai funkcijai kā change_metric_filtering)

7. Konstantes ar lieliem burtiem rakstam (šeit piemēri kodēšanas standartiem: https://evalds.notion.site/TypeScript-JavaScript-SQL-Coding-standard-a5726efa2ac54f20b248da9eb32cee5d)

8. For loop problēmas, sk. iepriekšējos komentārus

9. Mainīgie eksistē viena scope ietvaros tikai { }

10. Jābūt kādam safety check, piemēram, ja while atkārtojas 100 reizes tad iziet, lai neiekārtos mūžīgā ciklā

    

11. 7_test_tasks diezgan labs, bet tur pietrūkst vairākās vietās await funkcijām, kuras atgriež promise, rezultātā nav garentāts, ka poga būs nospiesta nākamajā rindiņā. pietrūkst arī logging informācija un nav iedots beigās testa rezultāts, vai bija veiksmīgs?

    

TODO

Pieinstalē jest bibliotēku pēc pamācības un sagatavo webstorm, ka 8_ testi dod konsolē lasāmu rezultātu: https://www.jetbrains.com/help/webstorm/running-unit-tests-on-jest.html#ws_jest_running_tests https://blog.jetbrains.com/webstorm/2018/10/testing-with-jest-in-webstorm/

Sagatavot testus, lai atapazītu iepriekš ziņotās sarunās:

1. Smieklus

2. Positive voice marķierus

3. Emocijas - happiness, anger, saddness, jo informāciju jāņem nevis no vizuāliem datiem, bet no conv dictionary, kurš ir pieejams lapā pēc ielādes (pauls var paskaidrot struktūru, bet doma būtu “vai 11sekundē ir pozitīva balss vai negatīva?”) let conv = await page.evaluate("conv");

    

Lai būtu redzami visi testi, kas veiksmīgi un neveiksmīgi un kāpēc

Issues

1. Tā kā var būt vairāki reprocessing runs, tad man ir jāpārbauda tikai 1. (pēdējais veiktais). Es mēģināju, ko šādu, bet tas nestrādāja:

   xxxxxxxxxx
   1
   1
   const [p_tag] = await page.$x('//*[@id="admin_panel"]/div//div[@class="side_panel_body"]//p[@class="api_tasks"]')[1];
   

   [1] daļa laikam radīja error

   

^ jā, jo tu jau lieto const [p_tag] tas ir JS dīvains pieraksts, kas ļauj atvērt masīvus, ja tu rakstītu ( await page.$x('//*[@id="admin_panel"]/div//div[@class="side_panel_body"]//p[@class="api_tasks"]'))[0] tad strādātu

Ievēro, ka es lietoju papildus iekavas, jo mums vajag await rezultātu nevis Promise pointeri pirms await

šeti vēl piemērs par masīvu atvēršanu

xxxxxxxxxx
7
1
const some = [1, 2, 3, 4];
2
const [a, b, c, d ] = some;
3
// tas pats kas
4
a = some[0]
5
b = some[1]
6
c = some[2]
7
d = some[3]

1. I can't get rid of this one bug:

   

Lai varētu izmantot vērtības, kuras tiek atgrieztas no funkcijas tas ir jāuzkodē, nevar izmantot citas funkcijas mainīgos, pat ja funkcija tikko izsaukta , visi mainīgie dzīvo savā scope { }

Tāpēc, ka p_tag ir elements, bet pats saturs ir iekš /text() ja izmanto ar XPath vai arī p_tag.text iekš JS