-
Notifications
You must be signed in to change notification settings - Fork 34
logbook
TODO: docker builds and travis ci
15.04.2017
html5validator has started to fail with This document appears to be written in Danish but the "html" start tag has "lang="en" error since 0.27 version. Workaround has been added by ignoring such warnings.
TODO: remove this workaround when false positive will be fixed in the upstream
Issue: https://github.com/svenkreiss/html5validator/issues/32
08.04.2017
At present we have 4 tests that fail time to time. Until we fix them completely, they might fail builds and show PRs as having errors. Because flaky tests don't relate to code from PRs, this fact could frustrate contributors. To improve this situation I added a special tag unstable that can be added to flaky tests to ignore their failures.
Commits: 56e848f73
06.04.2017
Thymeleaf Tooglz dialect doesn't have an easy way to set an attribute value depends on the feature state. I had to use 2 tags with different togglz:active/togglz:inactive attributes.
UPDATE(19.04.2017): I decided to not use Togglz for an attribute but move it to the controller. Approach with 2 tags didn't work because integration tests were failing (they were expecting a new URL but server was returning an old one).
Issue: https://github.com/php-coder/mystamps/issues/387
Issue: https://github.com/heneke/thymeleaf-extras-togglz/issues/9
05.04.2017
danger has started to fail with 'find_merge_base': Cannot find a merge base between danger_base and danger_head on pull requests. To workaround this error I rolled back to last working version 4.3.1
Issue: https://github.com/danger/danger/issues/768
14.03.2017
John Shkarin has reported that checkstyle on Windows complains about a lot of errors. We suspect that this happens because src/main/config/checkstyle-suppressions.xml file isn't taking into account at all or some of the suppressions are ignored (probably because of / vs \). We tried to follow the instructions from this blog but it didn't help.
TODO: fix it later Report: https://github.com/php-coder/mystamps/pull/547#issuecomment-286550307
01.03.2017
Input Text keyword from Selenium2Library doesn't support Russian characters in the report. I've created a bug report.
Issue: https://github.com/MarkusBernhardt/robotframework-selenium2library-java/issues/90
27.02.2017
Selenium2Library shows poor error message when we'are trying to interact with invisible element: You may only interact with visible elements. It doesn't contain locator of this element, for example. I've created an issue for that in upstream but author said that it should be implemented in the Selenium itself.
Issue: https://github.com/robotframework/Selenium2Library/issues/769
UPDATED(26.04.2017): I've created this in the Selenium: https://github.com/SeleniumHQ/selenium/issues/3917
24.02.2017
sed (from Ubuntu 12.04) that is installed in Travis CI doesn't support -z option. I had to rewrite script on Perl as a workaround.
TODO: revert this change back after updating to Ubuntu 16.04
Commits: feea131a3, aab6c7d97, 54152eb22
26.01.2017
When Selenium2Library and htmlunit are used together, the latter prints a lot of messages to the console. I asked upstream but didn't get an answer. To workaround this, I created a script that is filtering out these noise messages from the output.
Issue: https://github.com/MarkusBernhardt/robotframework-selenium2library-java/issues/85
Issue: https://github.com/php-coder/mystamps/issues/538
06.01.2017
I was needed to render categories with its subcategories. Database is returning data in the form (sub category name + category name or null). It turns out that it's impossible to render this data with Thymeleaf as a <select> with <optgroup> elements. I had to do a data transformation inside of a service method.
Commit: https://github.com/php-coder/mystamps/commit/b7fe1dac7a7f0bc09e9e153a8408687fd899c79f
05.10.2016
Because codenarc-maven-plugin doesn't pick up codenarc.properties file, I had to use @SuppressWarnings for ignoring some rules.
TODO: these workarounds should be removed when we'll find a way of using codenarc.properties
Commits: 24c47c2d6
Issue: https://github.com/gleclaire/codenarc-maven-plugin/issues/14
Issue: https://github.com/php-coder/mystamps/issues/539
05.10.2016
CodeNarc produces false positive on Spock tests that uses || as a delimiter for test data. I've reported about it to the upstream.
TODO: remove these workarounds when the issue will be fixed in the upstream
Commits: 24c47c2d6
Issue: https://github.com/CodeNarc/CodeNarc/issues/176
30.09.2016
Deploy to prod with ansible has started to fail with {"assertion": "war_file.stat.exists", "changed": false, "evaluated_to": false, "failed": true} To workaround this error I rolled back to last working version 2.1.1.0
Commit: fc72d3b93
Issue: https://github.com/ansible/ansible-modules-core/issues/5115
28.09.2016
Because Travis CI has MySQL version 5.5.53 (that is less than 5.6.5 but the latest for Ubuntu 12.04), we couldn't create a column with a NOW() as a default value. We did the same in 3 steps to support this old version.
30.01.2016
html5validator complained about An "img" element must have an "alt" attribute. I've suppressed this error until #314 will be fixed.
TODO: revert this commit when #314 will be fixed
Commit: ec40a6ad43