Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update functional correctness tests to support renamed CSV files #195

Closed
EnekoGonzalez3 opened this issue Sep 28, 2024 · 0 comments
Closed
Assignees
Milestone

Comments

@EnekoGonzalez3
Copy link
Contributor

EnekoGonzalez3 commented Sep 28, 2024

The current functional correctness tests are outdated due to changes in the naming of CSV files, making it impossible to continue the original verification process. A new test proposal is needed to ensure the following checks are performed:

  1. Message Storage Validation: Ensure that every message generated by the client is stored in the correct file, corresponding to its chronicle and story name.
  2. Timestamp Range Check: Verify that the timestamp of each message falls within the expected range of [story_chunk_start, storychunk_end(, ensuring proper data segmentation.
  3. Message Sorting by Timestamp: Ensure all messages within a file are correctly sorted by their timestamp.

To achieve this, the client_admin tool will be used for personalized real-time message sending, and the data string format will follow the pattern: chroniclename_storyname.

@EnekoGonzalez3 EnekoGonzalez3 self-assigned this Sep 28, 2024
@ibrodkin ibrodkin added this to the 2024-10-04 milestone Oct 15, 2024
@EnekoGonzalez3 EnekoGonzalez3 changed the title Update Fidelity Test to Reflect New CSV File Naming Convention Update functional correctness tests to support renamed CSV files Oct 24, 2024
@EnekoGonzalez3 EnekoGonzalez3 linked a pull request Nov 8, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment