<feed xmlns='http://www.w3.org/2005/Atom'>
<title>FreshRSS (Customized)/tests, branch 1.17.0</title>
<subtitle>Customized version of FreshRSS, a self-hosted RSS feed aggregator</subtitle>
<id>https://git.rdnlsmith.com/fresh-rss-custom/atom?h=1.17.0</id>
<link rel='self' href='https://git.rdnlsmith.com/fresh-rss-custom/atom?h=1.17.0'/>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/'/>
<updated>2020-07-08T10:11:55+00:00</updated>
<entry>
<title>Add a migration system (#2760)</title>
<updated>2020-07-08T10:11:55+00:00</updated>
<author>
<name>Marien Fressinaud</name>
<email>dev@marienfressinaud.fr</email>
</author>
<published>2020-07-08T10:11:55+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=8619cf6fa65bbd90871e7b7fe29816092a9d6c33'/>
<id>urn:sha1:8619cf6fa65bbd90871e7b7fe29816092a9d6c33</id>
<content type='text'>
* Add a Minz_Migrator class

Until now, we updated the database structure somewhere in the code but
it wasn't always consistent and somehow complicated to find. Also, this
code was always checked for nothing.

The Migrator aims to improve and ease the creation of migrations. It
should improve the way we apply the updates, making the update server
almost useless.

References:

- example of migration (before Migrator): https://github.com/FreshRSS/FreshRSS/commit/cc0db9af4f980829faa4bf0960617807b32fb4fa#diff-11a53443fa81512b128c66b065df0679R10
- update server: https://github.com/FreshRSS/update.freshrss.org
- PR moving the code of the update server to the core: https://github.com/FreshRSS/FreshRSS/pull/1760

* Automatically apply migrations

For now, administrators are used to have nothing to do during an update
else than getting the new code. I suggest to keep this behaviour and
automatically apply migrations if we detect new ones.

Another solution would be to create a CLI command and ask admins to call
it after getting the new code. It could hide migrations errors to end
users, but admin can forget to apply migrations since there are not used
to it.

* Add documentation for Minz Migrator

* Execute migrations even if next ones are applied

* Change mechanism to prevent multiple update at once

* Use mkdir to create the lock and to test it exists

Reference: https://stackoverflow.com/a/731634

* Append .lock to applied_migrations_path

There are no needs to define another file to serve as a lock.

* Change migrations naming convention

* Apply suggestions from code review

Co-Authored-By: Alexandre Alapetite &lt;alexandre@alapetite.fr&gt;

* Perform a low-cost migration versions comparaison

* Clarify version numbers concerning the migration system

Co-authored-by: Alexandre Alapetite &lt;alexandre@alapetite.fr&gt;</content>
</entry>
<entry>
<title>gitignore phpunit.phar (#2868)</title>
<updated>2020-04-01T00:08:25+00:00</updated>
<author>
<name>Alexandre Alapetite</name>
<email>alexandre@alapetite.fr</email>
</author>
<published>2020-04-01T00:08:25+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=5b9f8f55ad2bedc6fded1b0bbf4c1cdaf29df93a'/>
<id>urn:sha1:5b9f8f55ad2bedc6fded1b0bbf4c1cdaf29df93a</id>
<content type='text'>
Used for tests
Follow-up of https://github.com/FreshRSS/FreshRSS/pull/2844</content>
</entry>
<entry>
<title>Improve login and registration pages (#2794)</title>
<updated>2020-03-07T23:14:29+00:00</updated>
<author>
<name>Marien Fressinaud</name>
<email>dev@marienfressinaud.fr</email>
</author>
<published>2020-03-07T23:14:29+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=51edbc1578fe49b281b39d91451d2b9df0092028'/>
<id>urn:sha1:51edbc1578fe49b281b39d91451d2b9df0092028</id>
<content type='text'>
* Keep the user on login page on failure

* Show an error if username already exists

* Check the password format in the backend

* Return a better message if username is invalid

* Add a title to the login page

* wip: Improve look of login and register pages

* Set a capital M in username help message

On the registration page, username tip started with a minuscule, while
the password tip started with a capital.

* Change message if username is taken</content>
</entry>
<entry>
<title>tec: Fix test suite (#2721)</title>
<updated>2019-12-23T10:00:22+00:00</updated>
<author>
<name>Marien Fressinaud</name>
<email>dev@marienfressinaud.fr</email>
</author>
<published>2019-12-23T10:00:22+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=8f5d8af72f91e3fa1d47717379570964fe970069'/>
<id>urn:sha1:8f5d8af72f91e3fa1d47717379570964fe970069</id>
<content type='text'>
* Rework the travis file to enable PHPUnit

I realized that unit tests weren't executed on Travis. While working on
this file to enable these tests, I started to think we could simplify
it.

I separated jobs so:

- PHP linter and tests are only performed on PHP 7.3
- Translations are tested separatly so they can fail (it was already the
  case but it was hard to understand the way it was done)
- PHP 5.6 only checks syntax issues
- the last job checks css, js, etc. (it didn't change)

PHPUnit is not executed on 5.6 because only the version 5 is available
while the latest version is the 8 (https://phpunit.de/supported-versions.html).

I think it's easier to read (each job is more explicit) but I'm not a
Travis expert so maybe there's some room for improvements.

* Fix failing tests

The category `_name` regression was introduced in commit b323ed084.

I wasn't able to understand when and why Search tests was wrong.

The rest is about upgrade of PHPUnit.
</content>
</entry>
<entry>
<title>tests: add shellchecks and shfmt to README (#2458)</title>
<updated>2019-07-24T03:26:51+00:00</updated>
<author>
<name>Frans de Jonge</name>
<email>fransdejonge@gmail.com</email>
</author>
<published>2019-07-24T03:26:51+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=b12d261407a5daccb9cbda1801a58fb9799d61a2'/>
<id>urn:sha1:b12d261407a5daccb9cbda1801a58fb9799d61a2</id>
<content type='text'>
See https://github.com/FreshRSS/FreshRSS/pull/2454#discussion_r306174996</content>
</entry>
<entry>
<title>[CI] Run shellcheck and shfmt (#2454)</title>
<updated>2019-07-22T14:07:40+00:00</updated>
<author>
<name>Frans de Jonge</name>
<email>fransdejonge@gmail.com</email>
</author>
<published>2019-07-22T14:07:40+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=ad92518fe44279b69308aaf0a757469f5bf42910'/>
<id>urn:sha1:ad92518fe44279b69308aaf0a757469f5bf42910</id>
<content type='text'>
* [CI] Run shellcheck and shfmt

Cf. https://github.com/FreshRSS/FreshRSS/pull/2436#discussion_r305640019

* rename

* no need for disable anymore

* also remove leftover indentation flags even if it makes no difference to syntax checking

* define colors and reset before exit for local use
</content>
</entry>
<entry>
<title>Minor changes (#1747)</title>
<updated>2017-12-21T16:20:52+00:00</updated>
<author>
<name>Nicola Spanti</name>
<email>dev@nicola-spanti.info</email>
</author>
<published>2017-12-21T16:20:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=942508d8798c67a129e488b73ceecd6f5116bd9e'/>
<id>urn:sha1:942508d8798c67a129e488b73ceecd6f5116bd9e</id>
<content type='text'>
</content>
</entry>
<entry>
<title>Use __DIR__ for relative include and require</title>
<updated>2017-12-11T20:11:34+00:00</updated>
<author>
<name>Alexandre Alapetite</name>
<email>alexandre@alapetite.fr</email>
</author>
<published>2017-12-11T20:11:34+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=52d09886558361bd818182da9ddc204e83f618ba'/>
<id>urn:sha1:52d09886558361bd818182da9ddc204e83f618ba</id>
<content type='text'>
For uniformity, and to avoid having PHP searching in include_path.
http://php.net/manual/function.include.php
https://github.com/FreshRSS/FreshRSS/pull/1715
https://github.com/FreshRSS/FreshRSS/pull/1711#issuecomment-350581350
</content>
</entry>
<entry>
<title>[ci] Add Travis (#1619)</title>
<updated>2017-09-22T10:13:46+00:00</updated>
<author>
<name>Frans de Jonge</name>
<email>fransdejonge@gmail.com</email>
</author>
<published>2017-09-22T10:13:46+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=4e174ed9dd84ce4f8de410baf6a6e0fde9160055'/>
<id>urn:sha1:4e174ed9dd84ce4f8de410baf6a6e0fde9160055</id>
<content type='text'>
* [ci] Add Travis

* Exclude some libs

* Semi-auto whitespace fixes

* line length in SQLite

* Exclude tests from line length

* Feed.php line length

* Feed.php: get rid of unnecessary concat

* Feed.php: line length

* bootstrap.php: no newline at end of file

* Allow concatenating across multiple lines

* Add Travis badge

* do-install line length

* update-or-create-user line length

* cli/create-user line length

* tests/app/Models/SearchTest.php fix indentation

* tests/app/Models/UserQueryTest.php fix indentation

* tests/app/Models/CategoryTest.php fix indentation

* [fix] PHP 5.3 on precise

* cli/do-install no spaces

* cli/list-users line length

* cli/reconfigure line length

* empty catch statements

* api/index line length nonsense

* spaces before semicolon

* app/Models/EntryDAO bunch of indentation

* extra blank lines

* spaces before comma in function call

* testing tabwidth

* increase to 10

* comment out tabwidth line

* try older phpcs version 3.0.0RC4

* line length exception for app/install.php

* proper spaces

* stray spaces in i18n

* Minz/ModelPdo line length

* Minz whitespace

* greader line length

* greader elseif placement

* app/Models/Feed.php spacing in function argument

* ignore php 5.3

* app/Models/ConfigurationSetter.php stray whitespace

* EntryDAOSQLite line length

* I vote for higher max line length =P

* ignore SQL

* remove classname complaint

* line length/more legible SQL

* ignore line length nonsense

* greader line length

* feedController issues

* uppercase TRUE, FALSE, NULL

* revert

* importExportController lowercase null

* Share.php default value not necessary because ! is_array () a few lines down

* CategoryDAO constants should be UPPERCASE

* EntryDAO reduce line length

* contentious autofix

* Allow failures on all versions of PHP except 7.1 because reasons
</content>
</entry>
<entry>
<title>Improve search: intitle, author, inurl</title>
<updated>2017-04-01T20:31:12+00:00</updated>
<author>
<name>Alexandre Alapetite</name>
<email>alexandre@alapetite.fr</email>
</author>
<published>2017-04-01T20:31:12+00:00</published>
<link rel='alternate' type='text/html' href='https://git.rdnlsmith.com/fresh-rss-custom/commit/?id=d9c0d25b85ef3df7ea2cdc261e274efcdd5cfce0'/>
<id>urn:sha1:d9c0d25b85ef3df7ea2cdc261e274efcdd5cfce0</id>
<content type='text'>
Allow multiple values of intitle: , author:, inurl:

Note: Tests for UserQueryTest are broken due to
https://github.com/sebastianbergmann/phpunit/wiki/Release-Announcement-for-PHPUnit-4.0.0#backwards-compatibility-issues
</content>
</entry>
</feed>
