Skip to content
This repository has been archived by the owner on Jun 16, 2021. It is now read-only.

Hello integration testing #410

Merged
merged 4 commits into from Sep 28, 2018
Merged

Hello integration testing #410

merged 4 commits into from Sep 28, 2018

Conversation

msdundar
Copy link
Contributor

@msdundar msdundar commented Sep 27, 2018

Bu PR'in yaptığı işi/değişikliği ve bu işi/değişikliği neden yaptığını açıklayın:

  • Entegrasyon testleri için capybara ve selenium-webdriver'ı etkinleştirir.
  • Driver olarak selenium_chrome_headless'ı ayarlar.
  • En sık kullanılan ekran çözünürlüklerini esas alarak dört farklı ekran çözünürlüğü tanımlar.
  • Giriş Yap/Kayıt Ol ekranını tanımlanan çözünürlüklerde test eder. (Mobil cihazlarda kayıt ol paneli görünmüyor #409 sorununu bu test fark etti)
  • Kullanıcı girişi iş akışını tanımlanan çözünürlüklerde test eder.
  • Mobil cihazlarda görünmeyen 'Kayıt Ol' panelini görünür hale getirir.

İlgili iş kayıtları:

#409

Kapatılacak iş kayıtları:

Fixes #409

Veritabanına etkileri:

N/A

Sistem/Ops etkileri:

N/A

Ekran Görüntüleri:

headless olmayan selenium_chrome modunda, farklı çözünürlükler için 1) sayfa elemenlerini kontrol 2) login testi:

peek 2018-09-28 02-40

CI'da kullanacağımız için böyle olmayacak ve headless kullanacağız tabii ki. Ancak geliştiriciler debug etmek için bu moda geçebilir veya ENV development ise bunu kullan da yapabiliriz. Yalnız çok CPU tüketiyor baştan söyleyeyim, CPU kullanımını %80'lere çıkartıyor.

Kontrol listesi:

  • Açtığınız PR'in başlığı kurallara (sadece ilk harf büyük, emir kipinde problem cümlesi vb.) uygun mu?
  • Katkı sağlama dokümanını okudunuz mu?
  • Yaptığınız iş/değişikliği dokümante ettiniz mi?
  • Yaptığınız iş/değişikliğin testlerini yazdınız mı?
  • Test coverage oranını kontrol ettiniz mi?
  • Kod kalitesi (karma) ve test suite dahil olmak üzere tüm entegre kontroller başarıyla geçiyor mu?
  • Kendinizi bu PR'e assign ettiniz mi?
  • Yaptığınız iş/değişiklik ile ilgili tüm proje üyelerinden review talep ettiniz mi?
  • Gerekli etiketleri (bug, feature-step, help-wanted vb.) girdiniz mi?

@msdundar
Copy link
Contributor Author

msdundar commented Sep 27, 2018

Snyk, selenium-webdriver'ın bağımlılıklarında (rubyzip) güvenlik açığı buldu.

Selenium'u değiştirebilir miyiz diye baktım, pek alternatifi yok:

  • racktest: does not have any support for executing JavaScript.
  • capybara-webkit: depends on QtWebkit which went EOL quite some time ago.
  • poltergeist: depends on PhantomJS for which active development ended quite some time ago.

Yakın zamanda güncellemeyi yapmışlar, ancak yeni sürüm çıkmadıkları için rubygems'te hala eski paket var: SeleniumHQ/selenium#6330

Copy link
Member

@roktas roktas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Süper süper 👏

@roktas
Copy link
Member

roktas commented Sep 28, 2018

Snyk, selenium-webdriver'ın bağımlılıklarında (rubyzip) güvenlik açığı buldu.

Selenium'u değiştirebilir miyiz diye baktım, pek alternatifi yok:

* racktest: does not have any support for executing JavaScript.

* capybara-webkit: depends on QtWebkit which went EOL quite some time ago.

* poltergeist: depends on PhantomJS for which active development ended quite some time ago.

Yakın zamanda güncellemeyi yapmışlar, ancak yeni sürüm çıkmadıkları için rubygems'te hala eski paket var: SeleniumHQ/selenium#6330

Bu arkadaşlar sıkça release yapıyorlar, en kısa zaman düzeleceğine eminim. Synk'da .synk dosyası üzerinden (veya Synk'da admin gui ile) kısa süreli olarak ignore yapılabiliyor: https://snyk.io/docs/ignoring-issues Şimdilik expire olacak şekilde bir ignore koysak ve arada kontrol etsek?

Copy link
Member

@isubas isubas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eline sağlık 👍

@msdundar msdundar merged commit f5f7a55 into develop Sep 28, 2018
@msdundar msdundar deleted the initialize-integration-testing branch September 28, 2018 13:15
@msdundar
Copy link
Contributor Author

30 gün süreyle ignore ettim. 30 günün sonunda hala güncel sürüm yayınlanmadıysa tekrar uyarı verecek.

@msdundar msdundar mentioned this pull request Sep 28, 2018
9 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants