Page MenuHomePhabricator

Add tests for automation
ClosedPublic

Authored by natekupp on Jul 13 2019, 1:50 AM.

Details

Summary

This gets automation test coverage from 0% to > 90%

Test Plan

unit

Diff Detail

Repository
R1 dagster
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

natekupp created this revision.Jul 13 2019, 1:50 AM
natekupp updated this revision to Diff 2868.Jul 13 2019, 1:53 AM

add to buildkite

natekupp updated this revision to Diff 2873.Jul 13 2019, 6:47 PM

snapshot tests for dataproc parsing

natekupp edited the summary of this revision. (Show Details)Jul 13 2019, 11:11 PM
natekupp added reviewers: max, schrockn.
alangenfeld accepted this revision.Jul 15 2019, 4:01 PM
alangenfeld added inline comments.
.buildkite/pipeline.py
161

do we really care these scripts run in every python?

python_modules/automation/automation_tests/files/spark_confs.md
1 ↗(On Diff #2879)

this could go in a __snapshots__ directory (causing them to be hidden in phab)

This revision is now accepted and ready to land.Jul 15 2019, 4:01 PM
natekupp updated this revision to Diff 2880.Jul 15 2019, 4:19 PM
natekupp edited the summary of this revision. (Show Details)

comments

This revision was automatically updated to reflect the committed changes.