Skip to content

Port tests 04-key-tests#25

Open
ninetteadhikari wants to merge 57 commits intoport-python-testsfrom
port-python-04-key-tests
Open

Port tests 04-key-tests#25
ninetteadhikari wants to merge 57 commits intoport-python-testsfrom
port-python-04-key-tests

Conversation

@ninetteadhikari
Copy link
Member

@ninetteadhikari ninetteadhikari commented Nov 18, 2025

Overview

Ports over python tests to elixir for 04-key-tests

Testing recommendations

Related Issues or Pull Requests

Checklist

  • Code is written and works correctly
  • Changes are covered by tests
  • Any new configurable parameters are documented in rel/overlay/etc/default.ini
  • Documentation changes were made in the src/docs folder
  • Documentation changes were backported (separated PR) to affected branches

@ninetteadhikari ninetteadhikari force-pushed the port-python-04-key-tests branch 2 times, most recently from 730c9c0 to 52e6d4a Compare November 18, 2025 13:19
@ninetteadhikari ninetteadhikari marked this pull request as ready for review November 18, 2025 13:19
@ninetteadhikari ninetteadhikari changed the title Port python 04 key tests Port tests 04-key-tests Nov 18, 2025
@ninetteadhikari ninetteadhikari force-pushed the port-python-04-key-tests branch from 52e6d4a to 93906bd Compare November 18, 2025 13:53
"utf8-3[]:number" => 9,
"utf8-3[]:null" => nil,
},
]
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
]
]

Copy link

@hulkoba hulkoba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left 2 minor nitpicky space suggestions.
Otherwise, this looks great, and I change the label 🚀

Don't forget to deprecate the python file :)

@hulkoba hulkoba force-pushed the port-python-tests branch 5 times, most recently from c413a13 to 25f1f55 Compare November 19, 2025 14:21
@ninetteadhikari ninetteadhikari force-pushed the port-python-04-key-tests branch 4 times, most recently from 71d5ab7 to 3e950f7 Compare November 21, 2025 10:59
janl and others added 30 commits December 6, 2025 15:01
This is in preparation for using parallel make features.

Since both .PHONY couch and src/couch exist, the parallising
of src/* would clobber the top level .PHONY couch target.
When called with -jN for N>1, make will run `rebar -r eunit` for
all src/* Erlang apps (without the default skipped ones) up to the
limit of N times in parallel.

This is best used with GNU Make, as it allows for controlling the
output of each subtask to be grouped. BSD Make has no such feature
and interleaves all parallel target’s output, making it hard to
read.

For example:

    gmake -j2 --output-sync=target

will run two test suites in parallel and keep their respective
outputs separated. It does this by buffering all output before a
task is done that means for the first few tests, you don’t see
output as you do with serial execution.

On my machine I can run up to -j6 relatively stable, making use
of all 14 cores. Beyond that, Spurious errors can occur. I’ll
file those separately.

-j2 shows an almost 2x speed improvement, as expected and things
scale relatively linarly up until ~2.5 minutes of runtime, which
seems to be a lower bound with all our various setup and wait
bits.

For comparison, -j1, that is serial execution takes about 10.5
minutes on this machine.

Use with care in CI, but definitely use on your local dev machine.
This is achived by configuring each couch_server launched
with separate etc, data and log directories. These directories
are cleaned up after successful test runs.
…sic-find

Port Python test to Elixir: `02-basic-find-test`
feat(dev/run): write couch_uri files so ports are discoverable with --auto-ports
When using the `simple` password scheme, the number of iterations
is `undefined`, so the user's password hash is updated every time
when a new request is made using user's authentication credentials.

Add a case statement to avoid this situation.
…dex-selection

Port Python test to Elixir: `05-index-selection-test‎`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants