airflow-declarative icon indicating copy to clipboard operation
airflow-declarative copied to clipboard

Published 20 hours ago • verified publisher icon rambler-digital-solutions

Add tests for bugs

Open KostyaEsmukov opened this issue 5 years ago • 0 comments

This PR contains a couple of commits containing failing tests for the two bugs in Declarative I'm aware of:

  1. The do block strips newlines from the substituted jinja variables, which breaks bash commands
  2. The natural delta: 1h format for the TimeDeltaSensor is currently unsupported: it is getting passed as a string rather than a timedelta.

This PR shouldn't be merged as is. Instead, the fixes for these bugs should be done on top of the corresponding commits from this PR.

KostyaEsmukov avatar Jul 17 '19 07:07 KostyaEsmukov