Truncate timestamp to shift windowing in trade agg

[sydneynotthecity]

PR Checklist

PR Structure

• This PR has reasonably narrow scope (if not, break it down into smaller PRs).
• This PR avoids mixing refactoring changes with feature changes (split into two PRs
  otherwise).
• This PR's title starts with the jira ticket associated with the PR.

Thoroughness

• This PR adds tests for the most critical parts of the new functionality or fixes.
• I've updated the docs and README with the added features, breaking changes, new instructions on how to use the repository.

Release planning

• I've decided if this PR requires a new major/minor/patch version accordingly to
  semver, and I've changed the name of the BRANCH to release/* , feature/* or patch/* .

What

Truncate the timestamp on date filter to ensure that the full day of data is considered. The current code will only pull partial date for the current date, dependent on runtime

Why

Jenkins build pipeline is failing tests for trade_agg and int_trade_agg_day because no data is pulled for the 1 day aggregate. This is because the current filter expects data newer than current timestamp:

where ledger_closed_at < timestamp_add('{{ dbt_airflow_macros.ts() }}',interval 1 day )
and ledger_closed_at >= '{{ dbt_airflow_macros.ts() }}'

Added logic to truncate the timestamp to the beginning of the date, so that all data for the current run is pulled.

Known limitations

This is less of a problem in production because we pass Airflow variables. The jenkins build job relies on the default, current timestamp, instead of an env variable being passed in.

To make this fix permanent, we also need to pass current_date - 1 in via env var in the jenkins_lint_and_test build script.

[chowbao]

Just wondering. Is there an issue with using timezones=none? I'd assume that's the same as the default UTC

[sydneynotthecity]

there isn't, but it is the default. My thought was why specify the default