search

ckeditor / ckeditor4-angular

Official CKEditor 4 Angular component.
Other
55 stars 32 forks source link

Configure travis to run all the tests #25

Closed engineering-this closed 5 years ago

engineering-this commented 5 years ago

Fully enabled CI Travis testing.

E2E tests are run only for Chrome, because they use Selenium/WebDriver to control browser.

Closes #15

f1ames commented 5 years ago

Rebased on the latest master.

f1ames commented 5 years ago

Also added badges to README file.

engineering-this commented 5 years ago

I've added one dependency karma-spec-reporter and used it instead of progress reporter. Additionally I've turned off capturing browser logs by karma. Errors will be still captured by reporter itself. Only issue remaining is that test are run asynchronously and successful messages are printed right after test, so they are not correctly ordered, browsers are mixed, etc. To make it less noisy I added an option to not log ✅ tests for travis, but they are still visible when run locally.

Only visible tests comes from protractor e2e tests, but there are just 6 tests for one browser, so I think it can stay this way.

Screenshot 2019-05-29 at 14 05 08

I just turned off an option to build for pushed branches, we need only build for pushed PRs.

Screenshot 2019-05-29 at 16 46 02

Before that change we had two builds per push:

Screenshot 2019-05-29 at 16 44 47

f1ames commented 5 years ago

Rebased on the latest master.