Fix get_niggli_reduced_lattice if entering A1 case

[packer-jp]

Summary

Major changes:

• fix 1: I found a bug at pymatgen/core/lattice.py::Lattice::get_niggli_reduced_lattice.
  • After updating lattice parameters at A1, variables A, B, ... also need to be updated.
  • Otherwise, condition checking would be wrong at A2 or later.
  • Therefore, I added a line for updating A, B, C, E, N, Y.
    • This is not minimal, but the simplest I think.
  • Also, added a test that fails before the fix and succeeds after the fix.
    • Correct values were calculated by spglib.

Todos

If this is work in progress, what else needs to be done?

Checklist

• Google format doc strings added. Check with ruff.
• Type annotations included. Check with mypy.
• Tests added for new features/fixes.
• If applicable, new classes/functions/modules have duecredit @due.dcite decorators to reference relevant papers by DOI (example)

Tip: Install pre-commit hooks to auto-check types and linting before every commit:

pip install -U pre-commit
pre-commit install

[janosh]

thanks @packer-jp, esp. for including a new test right away! 👍