pytest supports providing full diff between dictionaries when higher
verbosity is specified. For details see
https://docs.pytest.org/en/7.1.x/how-to/output.html#verbosity.

Make use of max verbosity level in CI (-vv)

Dmitry Rodionov authored 1 year ago and Yaroslav Dynnikov committed 1 year ago

Since https://github.com/tarantool/tarantool/pull/8626 memtx_memory is
now rounded up to 64MB in case lower value is specified.

Anatoly Popov authored 1 year ago and Yaroslav Dynnikov committed 1 year ago

1. http-server:

    1. Get info about tiers from _pico_tier
    2. Get info about instance addresses from:
        - http: lua api
        - binary:  _pico_peer_addresses
    3. Switch from arrays to map where it makes sense
    4. Compose all the data to `cluster_state` object
    5. Get version information for lua api.

2. Tests:

    1. Add DeepDiff package to better diff for json
    2. Amend tests to new specification
    3. Get picodata version via lua api and use it in asserts

3. Chore:

    1. Add .envrc and .direnv to .gitingore (direnv specific files)
    2. Add .vscode to .gitignore.

Signed-off-by: Anatoly Popov <a.popov@picodata.io>

Having two different fixtures `binary_path` and
`path_to_binary_with_webui` results in weird failures in CI:

```
FAILED test/int/test_couple.py::test_exit_after_apply - conftest.TarantoolError:
    ('ER_NO_SUCH_PROC', "Procedure 'pico._inject_error' is not defined")
```

The root cause is that one `cargo build` overrides another with
different features enabled. To resolve the issue this patch
significantly refactors the test flow:

1. Web UI tests are skipped by default. One can enable it explicitly
   with `pytest --with-webui`

2. New fixture `cargo_build` behavior depends on environment:
   - In local dev environment it rebuilds the project with required features
   - In CI it skips rebuilding and runs tests against the existing build

3. The `path_to_binary_with_webui` fixture is removed.

See also:

- https://stackoverflow.com/a/55769818/1182787
- https://docs.pytest.org/en/7.4.x/how-to/writing_hook_functions.html
- https://docs.pytest.org/en/7.4.x/how-to/mark.html
- https://docs.pytest.org/en/7.4.x/example/simple.html#control-skipping-of-tests-according-to-command-line-option

Дмитрий Кольцов authored 1 year ago and Yaroslav Dynnikov committed 1 year ago

This commit introduces the first iteration of Picodata Web UI.
It lacks a lot of features and also indulges a lot of technically
imperfect decisions. Those are supposed to be addressed later.

Picodata treats absent hostname (`:8080`) as `localhost` by default,
which resolves to IPv6 on Mac. The test explicitly specified
inconsistent addresses, and that caused "connection refused" error.

Yaroslav Dynnikov authored 1 year ago and Yaroslav Dynnikov committed 1 year ago

Adopted styleguide is "actual == expected", not vice versa.