chore(rpc-subscriptions): Remove timestamp rounding

[shruthilayaj]

With this change #6609 it's no longer
required to round the timestamps to get back a single time bucket.

[codecov]

❌ 1 Tests Failed:

Tests completed	Failed	Passed	Skipped
2465	1	2464	5

View the top 1 failed tests by shortest run time

tests.subscriptions.test_data.TestBuildRequest::test_conditions[RPC subscription]

Stack Traces | 0.318s run time

Traceback (most recent call last):
  File ".../tests/subscriptions/test_data.py", line 221, in test_conditions
    self.compare_conditions(subscription, exception, "count", expected_value)
  File ".../tests/subscriptions/test_data.py", line 208, in compare_conditions
    assert result.result["data"][0][aggregate] == value
AssertionError: assert 11.0 == 20.0

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

[shruthilayaj]

The BaseSubscriptionTest class that the test inherits from adds events to the previous hour. This test worked before because we were rounding timestamp to the nearest granularity.

The larger time window will catch the events I expect to catch.

[shruthilayaj]

adding a couple extra messages outside the time bounds of the subscription execution so I know those don't get included in the count