astro-sdk icon indicating copy to clipboard operation
astro-sdk copied to clipboard

Adjust test for Airflow-2.9

Open pankajastro opened this issue 9 months ago • 3 comments

  • pin google-cloud-bigquery
  • Remove conditional query building for different Airflow version
  • Adjust dataset and cleanoperator test
  • clean checkoperator test

pankajastro avatar May 03 '24 09:05 pankajastro

pre-commit failure is looking like because of recent pre-commit hook update will handle that in separate PR

pankajastro avatar May 07 '24 15:05 pankajastro

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.12%. Comparing base (162b0ab) to head (aea0ff6). Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2149      +/-   ##
==========================================
+ Coverage   89.75%   91.12%   +1.37%     
==========================================
  Files          75       75              
  Lines        4322     4316       -6     
  Branches      539      538       -1     
==========================================
+ Hits         3879     3933      +54     
+ Misses        343      287      -56     
+ Partials      100       96       -4     
Flag Coverage Δ
PythonSDK 91.12% <100.00%> (+1.37%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 07 '24 15:05 codecov[bot]

Why this in review state since 5 days?

kaxil avatar May 14 '24 09:05 kaxil

Why this in review state since 5 days?

Will connect with the codeowners to conclude on this.

pankajastro avatar May 14 '24 11:05 pankajastro

some tests are failing because of Databricks related infra change

pankajastro avatar May 16 '24 10:05 pankajastro

some tests are failing because of Databricks related infra change

Fixed in PR https://github.com/astronomer/astro-sdk/pull/2161 and https://github.com/astronomer/astro-sdk/pull/2159

pankajastro avatar May 16 '24 12:05 pankajastro