search

coders-school / desktop-business-app

0 stars 4 forks source link

Chore/add automated tests #1

Closed automationdream closed 8 months ago

automationdream commented 9 months ago

Some first working unit tests for CI/CD development

Changes:

If something will fail in the Unit Tests, then entire flow would fail.

Covers:

ziobron commented 9 months ago

Thanks @Aleksiiej for Code Review. 🏅 1 XP granted

automationdream commented 9 months ago

Czy zamiast COPY src /app/src i COPY tests /app/tests bardziej uniwersalne byłoby COPY . /app od razu cały folder z projektem zostanie skopiowany

Moim zdaniem działająca aplikacja powinna zawierać jedynie plik binarny. Ogólnie copy folder, z tego co widziałem to nie jest praktyka, ponieważ powiększa rozmiar obrazu :) Tu co prawda mamy tylko hello world ale po co nam na przykład plik Readme do działania aplikacji?

automationdream commented 8 months ago

Czy zamiast COPY src /app/src i COPY tests /app/tests bardziej uniwersalne byłoby COPY . /app od razu cały folder z projektem zostanie skopiowany

Moim zdaniem działająca aplikacja powinna zawierać jedynie plik binarny. Ogólnie copy folder, z tego co widziałem to nie jest praktyka, ponieważ powiększa rozmiar obrazu :) Tu co prawda mamy tylko hello world ale po co nam na przykład plik Readme do działania aplikacji?

Ok, już rozumiem. Tak jak omówiliśmy na spotkaniu C++ Ninja w dniu dzisiejszym, jak mówimy o developmencie to Twoje podejście jest rzeczywiście wygodniejsze, natomiast jeśli chodzi o kod produkcyjny to powinno się optymalizować obraz.

Aleksiiej commented 8 months ago

Proposition - to use this CMake set command and create separate folder for binary files: set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${PROJECT_SOURCE_DIR}/bin)

ziobron commented 8 months ago

Thanks @fbtom for Code Review. 🏅 1 XP granted

ziobron commented 8 months ago

Thanks @BaqWin for Code Review. 🏅 1 XP granted

ziobron commented 8 months ago

Your PR was merged! 🏅 2 XP granted. Thanks @automationdream for making course materials better!