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

Published 20 hours ago • verified publisher icon astronomer

Refactor integration tests to unit tests

Open phanikumv opened this issue 1 year ago • 2 comments

Describe the bug https://github.com/astronomer/astro-sdk/pull/1488/commits/67d557c1588779e964b5b29096940d670a9c1b5d —There was no need of integration tests for them; unit tests would be fine. Let’s please not use fixtures unnecessarily. Let’s identify such tests and refactor them when you get some time.

Version

  • Astro: [e.g. 0.6.0]
  • OS: [eg. Debian]

To Reproduce Steps to reproduce the behavior:

  1. Write the DAG '...'
  2. Create connection '....'
  3. Run using '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Additional context Add any other context about the problem here.

phanikumv avatar Jan 12 '23 10:01 phanikumv