linera-io/linera-protocol

Use `insta` for snapshot tests in `linera-witty-macros`

Closed

#1,761 opened on Mar 6, 2024

View on GitHubContributor guide
 (0 comments) (0 reactions) (0 assignees)Rust (2,354 forks)batch import
good first issuetesting

Repository metrics

Stars
 (32,149 stars)
PR merge metrics
 (Avg merge 2d 22h) (111 merged PRs in 30d)

Description

We currently have several snapshots embedded in the code of linera-witty-macros. insta is a tool for managing snapshots so that they don't clutter up the code and can be easily reviewed and regenerated that we are now using in linera-views-derive. It would be nice to use it for linera-witty-macros as well.

Contributor guide

Research direction
Examine the existing snapshot tests embedded in `linera witty macros` code, then look at how `linera views derive` uses `insta` as a reference. The task is to replace the inline snapshots with `insta` snapshot calls, ensuring tests still pass and snapshots are managed via the `insta` review workflow.
Tech stack
rust
Domain
testingdeveloper experience
Issue type
Refactor
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
Rust programminginsta snapshot testing tool
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70