Florent Viard authored May 25, 2021 and hydrargyrum committed Jun 05, 2021

Firefox, for example, could omit the "text" field of "content" in the
case of a redirection.

For some reason, pylint generates import errors like "cannot import
modules.xxx.browser" which created a compat/ dir but nothing was written
in it (apart from __init__.py).
Now, we only create the compat/ dir when about to write a real file in
it.

This allows to use a virtualenv instead of forcing the python
interpreter of the pylint command shebang.

Since py3.4, generating a TestResult out of a TestSuite removes all
TestCases, which prevents from getting info from them.
Since we want info on TestCases, we have to force unittest not to remove
them, by monkeypatching.

xunitparser should be patched but it's unmaintained now.

For user setups, a number of dirs are used, whose paths are
customizable.

For workdir, before:
- $WEBOOB_WORKDIR
- $XDG_CONFIG_HOME/weboob

and after:
- $WOOB_WORKDIR
- $WEBOOB_WORKDIR
- $XDG_CONFIG_HOME/woob
- $XDG_CONFIG_HOME/weboob

If the old name is found in $XDG_CONFIG_HOME, it will try to rename it.

For data dir, now:
- $WOOB_DATADIR
- $WOOB_WORKDIR
- $WEBOOB_DATADIR
- $WEBOOB_WORKDIR
- $XDG_DATA_HOME/woob
- $XDG_DATA_HOME/weboob

Also, $WOOB_BACKENDS then $WEBOOB_BACKENDS.

By forcing argv, "config update" was received as a command.

setup.py installs in a different way than pip.

Since the package was renamed, it seems to cause problems if "weboob" was
previously installed with setup.py. Installing "woob" with pip doesn't seem
to cause a conflict though, so let's switch to it.

Florent Viard authored Feb 22, 2021 and hydrargyrum committed Feb 26, 2021

The name part of the request that is extracted, could be very very long,
and then the har-to-old.py script will fail because the filename is
longer than what is allowed by the file system.

Florent Viard authored Nov 02, 2020 and hydrargyrum committed Nov 17, 2020

For a given web request, there use to be 2 metadata files generated:

    001-200-3-request.txt
    001-200-3-response.txt

But, there is a typo in the code, and instead the generated files are:

    001-200-3-request.txt
    001-200-3.response.txt

This change fixes the typo to get back the original behavior.

Florent Viard authored Oct 29, 2020 and hydrargyrum committed Nov 17, 2020

The previous syntax for har-to-old was:

    usage: har-to-old.py [-h] file prefix

Change it to:

    usage: har-to-old.py [-h] file [destdir]

- prefix renamed in destdir, as it was not really a prefix but the
name(/path) to the destination folder for extracted files
- automatically create the destdir folder if it does not exist
- if destdir is not provided, use the file name without extension

ntome authored Oct 12, 2020 and hydrargyrum committed Oct 28, 2020

But it's pep8 rule keeps application-imports in the same group as
relative-imports. Changing it is not a matter of configuration but defining
our own style, which is tedious as it would require a setuptools entrypoint
just for that, so we inject manually the style.

ntome authored Sep 21, 2020 and hydrargyrum committed Sep 23, 2020

Thanks to this tool, we will be able to drop dumping to the old, custom,
hard-to-parse format in favor of the new, ill-defined format: HAR!

Jerome Berthier authored Aug 19, 2020 and hydrargyrum committed Aug 26, 2020

Useful when using with a pre-push hook, it is useless to run flakes
on the whole code base every time you push.

Example of pre-push hook using this feature:

ref=$(git rev-list --boundary HEAD...master | grep "^-" | cut -c2-)
PYFILES="$(git diff ${ref} --name-only | grep -E "*.py$" | xargs)" $PWD/tools/pyflakes.sh
if [ "$?" -ne "0" ]; then
    echo >&2 "Flakes error"
    exit 1
fi

Augustin Bouverot authored Aug 20, 2020 and hydrargyrum committed Aug 26, 2020

We now get 'AwesomeModule' instead of 'Awesome_Module' when generating
'awesome_module'.

ntome authored Jul 09, 2020 and hydrargyrum committed Aug 26, 2020

Constructions like:

    a and b or c
    1 << 2 & 3

are discouraged because precedence isn't obvious at first sight.
Parentheses must be added to make code explicit.

Vincent Ardisson authored Aug 05, 2020 and hydrargyrum committed Aug 07, 2020

flake8 is capable of running checks in parallel if we give it multiple files.

A star-mapping in a dict returns a None key.