meilisearch/MeiliSearch

Add a test to export chat settings in dumps

Open

#5736 opened on Jul 7, 2025

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Rust (20,887 stars) (733 forks)batch import
good first issuemaintenance

Description

Hey, I overall it's nice, I think we should add a test in https://github.com/meilisearch/meilisearch/blob/fef089c7b6a47961c64f3851cce034e3202e011e/crates/dump/src/lib.rs#L546

But if we want to land the PR for the rc.0 IMO we merge it as-is and do the test later / open an issue

Originally posted by @irevoire in https://github.com/meilisearch/meilisearch/pull/5731#pullrequestreview-2992732964

Contributor guide

Tech stack
rust
Domain
backendtesting
Issue type
test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Rustcargo testdump module structure
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80
Research direction
Examine the suggested file at crates/dump/src/lib.rs around line 546 to understand the current export chat settings implementation. Look at the linked PR #5731 for context on what was merged. Write a test that verifies the export of chat settings in dumps, following existing test patterns in the same file or crate.