If an error is returned by the server when loading folders, the animated...
If an error is returned by the server when loading folders, the animated spinner in the sidebar will disappear.
View ArticleGreatly simplified acceptance testing of infinite scrolling.
The new version is less exhaustive but it should be enough. Test time was extremely long, this should help cutting it down to a reasonable length.
View ArticleDo not start a rails server in CI, selenium takes care of that. Also store...
Do not start a rails server in CI, selenium takes care of that. Also store Sauce Labs credentials as encrypted env variables.
View ArticleMore polishing of CI tests config.
Ensure sauce labs cloud is used only in Travis CI, not in other CI environments. Max-duration value was too high, set it to 1 hour.
View ArticleMax duration for SauceLabs tests is 2 hours, instead of 1.
It's still an extremely long length. Probably a lot of tests have to be pruned. But right now the priority is to get tests to pass consistently.
View ArticleUse rspec-retry gem to retry failed tests up to 3 times.
Specially useful for intermittent failures in acceptance test. Selenium in particular sometimes fails with a Net::ReadTimeout error caused by a bug in selenium itself, usually retrying makes the test...
View ArticleInclude saucelabs gems only in bundle only in Travis CI.
Otherwise just having sauce-connect in the bundle demands having env variables with the username and API key for saucelabs, which makes no sense if we're not conna use the saucelabs cloud.
View ArticleSpeed up test by looking at some of the entries that have to be present in th...
Speed up test by looking at some of the entries that have to be present in the page, instead of checking all of them. This particular test is painfully slow (more than 5 minutes!). This commit makes it...
View ArticleAnother approach to have saucelabs gems in the bundle without causing problem...
Another approach to have saucelabs gems in the bundle without causing problems in environments other than travis ci.
View ArticleFix reporting to coveralls from the CI environment.
Also the rake task to run tests is now "rake spec:ci" instead of "rake spec:travisci". We've switched from travis to semaphore.
View ArticleFix error that caused & chars in feed and entry URLs become escaped into &
This was caused by the sanitizer, which assumes input is HTML and escapes any entities it finds. This behavior cannot be avoided, so the output from the sanitizer is passed to CGI.unescapeHTML() so...
View ArticleDo not escape HTML entities (e.g. & characters) in feed URLs when sending the...
Do not escape HTML entities (e.g. & characters) in feed URLs when sending them to the client as JSON. This fixes a problem where feeds with "&" characters in their URL had the wrong URL in...
View ArticleWhen updating a feed, if sanitization would turn the title into a blank...
When updating a feed, if sanitization would turn the title into a blank string, keep the old value instead. This is intended for feeds which have e.g. something that looks like an HTML comment for a...
View ArticleWhen autodiscovering a feed, if the autodiscovered url is relative and does...
When autodiscovering a feed, if the autodiscovered url is relative and does not begin with '/', prepend a '/' before converting it to an absolute url. This fixes error autodiscovering feeds with...
View ArticleAdded "published" attribute to EntryState model.
The idea is to denormalize the database so that the queries that retrieve paginated entries only have to follow the EntryState table and indexes, resulting in faster response time.
View ArticleAdded not null validation to "published" attribute of Entry model.
This should have been added long ago, it somehow slipped my radar.
View ArticleRemoved test, a test for the default value of "published" attribute of Entry...
Removed test, a test for the default value of "published" attribute of Entry model was already implemented.
View ArticleRepeat failed tests up to 3 times in CI environment only. Other environments...
Repeat failed tests up to 3 times in CI environment only. Other environments do not repeat failed tests. Trying to debug a failed test that is repeated is annoying.
View ArticlePublished attribute of EntryState model has its value fixed to the "published...
Published attribute of EntryState model has its value fixed to the "published" attribute of the associated Entry instance. No matter what value other code tries to set, it is guaranteed that an...
View Article