add adapter_response for test

[aezomz]

Add adapter_response for dbt test

resolves #2964
or partial resolve?

Description

Checklist

• I have read the contributing guide and understand what's expected of me
• I have signed the CLA
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
• I have run changie new to create a changelog entry

[cla-bot]

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @aezomz

[ChenyuLInx]

Thanks for contributing! This looks good! Can we also do similar thing for freshness task?
Also do you think you can look up functional tests that test those two functions and add some test there?(link for freshness, for test maybe add some additional check here?)
How to run functional tests(link). Once you setup you can also just do pytest testfilename.py

[aezomz]

Thanks for contributing! This looks good! Can we also do similar thing for freshness task? Also do you think you can look up functional tests that test those two functions and add some test there?(link for freshness, for test maybe add some additional check here?) How to run functional tests(link). Once you setup you can also just do pytest testfilename.py

Can u share where can I get the adapter response for freshness? I couldn't find it
https://github.com/dbt-labs/dbt-core/blob/main/core/dbt/adapters/base/impl.py#L1072

Big thank you for pointing me to the test that i should update with.

[jtcohen6]

Seconding, thanks for the contribution @aezomz!

Can u share where can I get the adapter response for freshness? I couldn't find it
https://github.com/dbt-labs/dbt-core/blob/main/core/dbt/adapters/base/impl.py#L1072

Check out this comment from a few months ago: #2964 (comment)

And this commit (though might need some rework given other changes): a066c1e

[aezomz]

@jtcohen6 , thanks for the guidance. I will never figure this out on my own.

There is some mypy errors that i suppressed since its returning AttrDict now, feel free to suggest a better way.
I did some pytest to ensure all is passing. But will see from CI if there are anything else breaking.

[ChenyuLInx]

@aezomz Thanks for adding this function to both test and freshness!!! Looks great to me

[elyobo]

🎉 thanks @aezomz!