include flagTags in postEvaluation and postEvaluationBatch

[su5kk]

Motivation and Context

We require the flagTags field that includes all tags related to a particular flag in the response for our internal logic.

How Has This Been Tested?

• Added validation checks in step_11_test_tag_batch_evaluation to ensure batch evaluations return:
  • flagTags field presence
• Added validation checks in step_12_test_tag_operator_batch_evaluation to ensure:
  • flagTags field presence

Types of changes

• Bug fix (non-breaking change which fixes an issue) ?? maybe not exactly a bug-fix

Checklist:

• My code follows the code style of this project.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.50%. Comparing base (770461b) to head (473d3b1).
⚠️ Report is 10 commits behind head on main.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #623      +/-   ##
==========================================
- Coverage   81.19%   76.50%   -4.69%     
==========================================
  Files          28       30       +2     
  Lines        2271     2418     +147     
==========================================
+ Hits         1844     1850       +6     
- Misses        337      478     +141     
  Partials       90       90              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[su5kk]

@nothing0012 hello, can you please take a look?

[zhouzhuojie]

this test kind of wrong, the intention I think it's to test if flagTags being included in the response, however, the test was just using tags as evaluation context input.

I think you could remove this line and assert that the tags were there in the resp

[su5kk]

hi, thank you for the review

I've added correct assertions
473d3b1

[su5kk]

@zhouzhuojie @nothing0012 hello, can you please take a look once again?

[nothing0012]

LGTM