fix sqs dev endpoint to show invisible fifo messages correctly

[thrau]

Motivation

This fixes a long-standing issue of the /_aws/sqs/messages endpoint not showing invisible messages correctly of FIFO queues. The issue was that we still made the assumption that queue.inflight holds all inflight messages. This is no longer true since fix message group id behavior in sqs fifo queues, since we reworked the way fifo queues keep track of inflight messages in #8238.

I slightly change how the messages are collected for fifo queues, and now it works.

Changes

• the /_aws/sqs/messages endpoint now correctly lists in-flight messages for FIFO queues

[github-actions]

Test Results - Preflight, Unit

22 277 tests  ±0   20 536 ✅ ±0   15m 26s ⏱️ -20s
     1 suites ±0    1 741 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit c55ddfe. ± Comparison against base commit f93ee95.

♻️ This comment has been updated with latest results.

[github-actions]

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 23s ⏱️ +5s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit c55ddfe. ± Comparison against base commit f93ee95.

♻️ This comment has been updated with latest results.

[github-actions]

LocalStack Community integration with Pro

    2 files  ±    0      2 suites  ±0   1h 2m 49s ⏱️ - 55m 58s
3 193 tests  - 1 589  3 058 ✅  - 1 388  135 💤  - 201  0 ❌ ±0 
3 195 runs   - 1 589  3 058 ✅  - 1 388  137 💤  - 201  0 ❌ ±0 

Results for commit c55ddfe. ± Comparison against base commit f93ee95.

This pull request removes 1592 and adds 3 tests. Note that renamed tests count towards both.

tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_lambda_dynamodb
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_opensearch_crud
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_search_books
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_setup
tests.aws.scenario.kinesis_firehose.test_kinesis_firehose.TestKinesisFirehoseScenario ‑ test_kinesis_firehose_s3
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_destination_sns
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_infra
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_prefill_dynamodb_table
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input0-SUCCEEDED]
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input1-SUCCEEDED]
…

tests.aws.services.sqs.test_sqs_backdoor.TestSqsDeveloperEndpoints ‑ test_fifo_list_messages_with_invisible_messages[domain]
tests.aws.services.sqs.test_sqs_backdoor.TestSqsDeveloperEndpoints ‑ test_fifo_list_messages_with_invisible_messages[path]
tests.aws.services.sqs.test_sqs_backdoor.TestSqsDeveloperEndpoints ‑ test_fifo_list_messages_with_invisible_messages[standard]

♻️ This comment has been updated with latest results.

[github-actions]

Test Results (amd64) - Integration, Bootstrap

    5 files  ±    0      5 suites  ±0   1h 32m 6s ⏱️ - 1h 6m 25s
3 217 tests  - 1 939  3 085 ✅  - 1 575  132 💤  - 364  0 ❌ ±0 
3 223 runs   - 1 939  3 085 ✅  - 1 575  138 💤  - 364  0 ❌ ±0 

Results for commit c55ddfe. ± Comparison against base commit f93ee95.

This pull request removes 1942 and adds 3 tests. Note that renamed tests count towards both.

tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_lambda_dynamodb
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_opensearch_crud
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_search_books
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_setup
tests.aws.scenario.kinesis_firehose.test_kinesis_firehose.TestKinesisFirehoseScenario ‑ test_kinesis_firehose_s3
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_destination_sns
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_infra
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_prefill_dynamodb_table
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input0-SUCCEEDED]
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input1-SUCCEEDED]
…

tests.aws.services.sqs.test_sqs_backdoor.TestSqsDeveloperEndpoints ‑ test_fifo_list_messages_with_invisible_messages[domain]
tests.aws.services.sqs.test_sqs_backdoor.TestSqsDeveloperEndpoints ‑ test_fifo_list_messages_with_invisible_messages[path]
tests.aws.services.sqs.test_sqs_backdoor.TestSqsDeveloperEndpoints ‑ test_fifo_list_messages_with_invisible_messages[standard]

♻️ This comment has been updated with latest results.

[baermat]

Thanks for tackling this! A go from my side, once my question is addressed