test(nlp-profile): accept source_record kwarg in map_file_transfer stubs (bugbot #304)

[divyasinghds]

Summary

Fixes a bugbot finding on the v0.3.12 release PR #304.

The #805 NLP text-profile tests in test_ingestor_base.py stubbed map_file_transfer with a 4-arg lambda (c, r, o, cfg=None), but production calls it with source_record as a keyword (base.py:456). The stub raised TypeError on the first file-bearing NLP record — which was swallowed by the per-record try/except, so the tests passed without actually exercising the file-transfer / profile-attachment path (false confidence).

Aligned the 3 stubs (lines ~1035/1059/1082) with the already-correct ones (568/597/623) by accepting source_record=None.

Note on the other bugbot finding (#304: "MLM tokenizer preflight removed")

That one is a false positive — commit 82a52bb (#805 / #299, "refactor(ingest): drop ingest-time tokenizer + fingerprint") intentionally removed the entire TokenizerValidator + fingerprint machinery from the ingest path (it deleted test_tokenizer_validator.py, test_tokenizer_fingerprint.py, and TokenizerValidator no longer exists anywhere). The MLM factory dropping the tokenizer preflight is the designed behavior of #805, not a regression — so no change is made for it.

Test plan

• pytest green locally — 1151 passed, 1 xfailed
• The 3 profile tests now exercise the real map_file_transfer(..., source_record=...) path instead of swallowing a TypeError

🤖 Generated with Claude Code

---

Note

Low Risk
Test-only stub signature changes; no production code paths are modified.

Overview
Updates three NLP text-profile tests in test_ingestor_base.py so their map_file_transfer patch stubs accept source_record=None, matching how BaseIngestor.ingest calls the real function.

Previously the four-argument lambdas raised TypeError on that keyword; the per-record handler swallowed it, so the tests still passed without running file transfer or the text-profile attachment path. The stubs now match the ones already used elsewhere in the same file (e.g. self-supervised ingest tests).

^{Reviewed by Cursor Bugbot for commit 576a3cd. Bugbot is set up for automated code reviews on this repo. Configure here.}

[LukasWodka]

👋 Heads-up — Code review queue is at 38 / 30

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

• .github#57 — fix(fr-gate): pass items at or beyond the required stage · author: @aptracebloc · no reviewer assigned
• .github#60 — feat(security): public-repo PII gate (block customer names + secrets in PRs) · author: @LukasWodka · no reviewer assigned
• .github#61 — fix(wip-limit): avoid sort|head SIGPIPE failing the WIP check when queue >10 · author: @shujaatTracebloc · reviewer: @saadqbal
• averaging-service#117 — refactor(averaging): per-framework weight-handling seam (WS-B B1) · author: @divyasinghds · no reviewer assigned
• averaging-service#118 — feat(averaging): trainable-only keyed-dict weight format (WS-C image_validator falsely rejects images when target_size matches exactly #104) · author: @divyasinghds · no reviewer assigned
• averaging-service#119 — fix(averaging): exact federated GaussianNB merge — usable + correct (WS-B B2, docs: fix staging-recipe link branch ref (main → develop) #112) · author: @divyasinghds · no reviewer assigned
• backend#815 — chore(deps): bump cryptography from 47.0.0 to 48.0.1 · author: @dependabot · no reviewer assigned
• backend#833 — fix(deploy): graceful shutdown + Dockerfile hygiene; drop dead EC2 script · author: @LukasWodka · no reviewer assigned
• backend#840 — feat(rfc-0001): namespace-slug rule + validation command (#837) · author: @LukasWodka · no reviewer assigned
• backend#843 — fix(test): connect local MySQL test DB over TCP + document mac test-tier setup · author: @LukasWodka · reviewer: @saadqbal

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)