Skip to content

test(etl): reproduce invalid amount coercion on live develop - #309

Closed
seonghobae wants to merge 1 commit into
developfrom
repro/reject-invalid-amount-f340d37
Closed

test(etl): reproduce invalid amount coercion on live develop#309
seonghobae wants to merge 1 commit into
developfrom
repro/reject-invalid-amount-f340d37

Conversation

@seonghobae

Copy link
Copy Markdown
Collaborator

Rebuilds #198 test-first from exact protected develop@f340d37caba9df0d4aeb88e5d18b78bd8cca4b60 rather than rewriting stale/conflicting #199.

This is intentionally test-only RED at exact head 6e49df6a419f8b25501e074d06fb7e3dd50381f4. EtlServiceAmountIntegrityTest reaches the real EtlService.processData(...) boundary and requires malformed, blank, excessive-precision and extreme-scale AMOUNT input to fail as INVALID_RECORD before JDBC. It also requires one invalid amount in a multi-record request to abort the entire prevalidated batch before any write.

Protected production still maps these invalid inputs to valid-looking 0.00, so the expected current result is deterministic RED. After that causal proof, apply only the bounded production correction and then align current docs/CHANGELOG without overwriting unrelated protected changes. Old #199 checks/reviews/base snapshots remain historical and do not transfer.

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a0bfddc5-73b1-4d38-a5b3-cb24ae74ac86

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant